Uploaded image for project: 'CiviCRM'
  1. CiviCRM
  2. CRM-17010

Add data tags to price field options

    Details

    • Type: Bug
    • Status: Done/Fixed
    • Priority: Trivial
    • Resolution: Fixed/Completed
    • Affects Version/s: 4.6.6
    • Fix Version/s: 4.6.7
    • Component/s: None
    • Labels:
      None
    • Documentation Required?:
      None
    • Funding Source:
      Needs Funding

      Description

      Hi Coleman, I wanted to get your feedback on this, after briefly discussing with Tim on IRC. I don't deal with price sets via js much but when others do there always seems to be much cussing about classes & html structure. I just needed to do some & it seemed to me that adding data classes to the html was potentially a fix. In the case of the piece I was working on I want to know the membership_type_id & num_terms of the selected option - both values in the price_field_value table.

      I'll do a PR of what I think might improve this & you can feedback. The data in the table doesn't seem to be sensitive.

        Attachments

          Activity

            People

            • Assignee:
              monish.deb Monish Deb
              Reporter:
              eileen Eileen McNaughton
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: