Patient Statements PDF uploaded:code review

zhhealthcare wrote on Tuesday, May 18, 2010:

Hi

We have uploaded the patient statements download to PDF  in the following tracker.
https://sourceforge.net/tracker/?func=detail&atid=1245239&aid=3003355&group_id=60081

We have not yet come round to creating patches.  So please do the needful to review and append for common use.

Thanks
sam

stephen-smith wrote on Tuesday, May 18, 2010:

I commented on your tracker item.

zhhealthcare wrote on Wednesday, May 19, 2010:

Stephen
We uploaded a revised zip file.  See if you can unzip it this time :slight_smile:
Thanks
Sam

bradymiller wrote on Saturday, June 12, 2010:

hey,

I just committed this with some minor modifications. Check out commits on
github below branch after your commit there (removed a client specific
entry and also saved the temp file to where the text tmp file is saved):
http://github.com/bradymiller/openemr/tree/pdf_eob_4

Note that saving these tmp files in tmp dirctory is not secure. They need
to both be deleted after upload (like the backup script). Will issue this
bug in the tracker.

thanks for the contribution and please everybody test it out on the cvs demo
tomorrow(if it tests well, then will include it in next 3.2.0 patch). I tested it fully, but I’m not a billing expert…

-brady

zhhealthcare wrote on Monday, June 14, 2010:

Thank You for your modification.We changed our local code to reflect the
same, you mentioned at
http://github.com/bradymiller/openemr/tree/pdf_eob_4.

Thanks
Paul