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

Remove incomplete price set reports from 4.2 distribution

    Details

      Description

      These reports are "in development" and per Eileen's post below might be best accessed as extensions OR pushed into core at a later time.

      In addition, he Price Set Line Item report throws a fatal error on my updated 4.2 sandbox and the 4.2 demo:

      Unsupported operand types in /Users/dgg/svn/crm_v4.2/CRM/Report/Form/Price/Lineitem.php on line 53

      == from Eileen McN ===========
      I’ve been thinking about the price set reports I put into 4.2 and I’ve come to the conclusion I’d be happier moving the templates out of the 4.2 release and releasing them as a separate extension.

      I’ve had a few enquiries so I know there are people who want to be able to use them but I’m not feeling comfortable with some of the compromises between UI & configurability in the context of core. (this has been touched on in the broader discussion). The other issue being that I’ve been working a lot on tweaking things in it and I feel like I want the flexibility to take some of my ideas further and develop them – which I would be better able to do on github than in core. I forsee some changes coming up

      In practice this would mean removing the CRM_Report_Form_Extended.php & the CRM_Report_Form_Price folder along with the few lines of SQL that add the reports into core.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                dgg David Greenberg
                Reporter:
                dgg David Greenberg
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: