Uploaded image for project: 'Reporting Module'
  1. Reporting Module
  2. REPORT-719

Reporting module fails to load in 1.12 due to removal of deprecated reporting classes

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: TBD
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: Reporting 0.9.4
    • Component/s: None
    • Labels:
      None
    • Complexity:
      Undetermined

      Description

      Recently, work in TRUNK-4337 has been completed to remove all deprecated reporting code out of OpenMRS core and into the reportingcompatiblity module.

      Unfortunately, this has resulted in the reporting module failing to load in 1.12 and beyond, due to the presence of the DataExportDataSetDefinition / Evaluator / Persister classes.

      The persister and the evaluator are the primary problems, as they are loaded automatically by Spring due to the presence of the @Handler annotation, and reference classes in the "org.openmrs.reporting." package such as AbstractReportObject that no longer exist.

      Steps:

      1. Remove these classes from the reporting module

      2. Move these classes into reportingcompatibilty (if necessary - may want to poll community and determine if anyone actually uses this anywhere)

      • Introduce dependency on reporting within reportingcompatibility
      • Move the classes over and either keep in the same packages (if possible) or move to new packages within reportingcompatibility and add a liquibase migration script to convert any existing serialized definitions that refer DataExportDataSetDefinition classes to use new package.

        Attachments

          Activity

            People

            Assignee:
            kristopherschmidt Kristopher Schmidt
            Reporter:
            mseaton Mike Seaton
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: