Patient Report Improvement

athidevwork wrote on Wednesday, April 03, 2013:

Hi Everyone,

I just did a small update to  filter out the empty fields in patient report.

Problem Summary:
Patient report generation was printing all the demographics fields even if they were empty.

Description:
There are like around 50 fields in demographics and the patient report prints all the 50 fields with empty labels even if we fill just 3 or 4.

Fix:
I am filtering the empty fields not to be included for printing in the reports.

Link to code : http://github.com/bradymiller/openemr/commits/Athi-patient-report-improvement_1

tmccormi wrote on Monday, April 08, 2013:

I’ll be happy to take a look at this one.
–Tony

tmccormi wrote on Monday, April 08, 2013:

Quick test looks good, some fields from the example_patient_data.sql show up on the report as “blank”, this has more to do with bad data in the examples than the code though.

I think this is OK, with one thought. It maybe that some fields need to show up even when they are blank, like “ST:”, seems odd and not good practice to skip the state field in an address block. Might need/want to add two things:

  1. A show-all (even if blank) check box (or globals?)
  2. An attribute in the layout editor/table to force display even if blank

-Tony

bradymiller wrote on Sunday, April 28, 2013:

Hi,

Just tested this. Seems good enough to commit and I think Tony agrees. Adding the options to show if blank would be much more work and can’t really think of an example why that would be needed(For example showing a state label when there is no state doesn’t seem like a good example). Will plan to commit this unless anybody objects.

-brady
OpenEMR

bradymiller wrote on Monday, May 06, 2013:

Hi,
Just committed this to sourceforge.
-brady
OpenEMR