openmrs/openmrs-module-reporting — pull request #16 by surangak (2013-01-17 17:22:03+00:00)
Title: Updates

Hi,

I've made the following changes to version 0.7.3.1 of the reporting module based on Jembi requirements. I'm making a pull request for these changes in case you consider them suitable for being merged into the module.

To explain my fix in laymens' terms,
If you look at the manageDefinitions.form page, you will see that there are three options for picking data definitions. You can select from either person data, patient data or encounter data.
Now we found out that even if we were to create data definitions using the PatientData list, they don't show up as an option in the data set definitions page. Only data definitions created from person data show up on this list.
So based on our requirements, we added functionality for a single data-set definition to contain stuff from both person data and patient data as well.
We tested this out extensively (also using reports created this way) and it works fine.

Hope that this will be useful to you...