New Report - CPT Finanancial

tmccormi wrote on Saturday, June 02, 2012:

This is a report contributed/designed by MI2 and written by Visolve.  It produces a summary of all financial activity by date-range, or provider.   A list_options table can be added to identify “important” codes which can either be shown exclusively to highlighted on the report.

=> Create a list_options with the name “CPT_report_codes” and enter some codes to differentiate the important Codes.
=> Administration -> lists-> New list ( with the name “CPT_report_codes”). Then in that list add codes in the “ID” field.

Will add the above to upgrade sql, etc so the list_option entry will exist before commit.

Please review.

https://github.com/tmccormi/openemr/tree/cpt_fin_report

-Tony

tmccormi wrote on Saturday, June 02, 2012:

We are working on some improvements and bug fixes to this too, so this is just a early look code review.  Right now it’s not summarizing the CPT codes, just listing each one separately… more SQL-fu needed…
-Tony

bradymiller wrote on Saturday, June 02, 2012:

Hi Tony,

New scripts should really be using the new security model (otherwise somebody else will need to covert them in the future):
http://open-emr.org/wiki/index.php/Codebase_Security#Plan

If Visolve if willing to learn and am happy to walk through it with them and show what needs to be changed. This is generally what I do for developers that do not know how to use the new security model yet.

-brady
OpenEMR

bradymiller wrote on Saturday, June 02, 2012:

Additionally,

Since we are now avoiding hard-coding of code type, would it be possible to:
1. Substitute CPT in labels with Procedure
2. In sql query, grather than hard-code the CPT4, grab all item that are set as ‘Fees’ in Adminisitration->Lists->Code Types (stored in the code_types table)

-brady
OpenEMR

visolveemr wrote on Monday, June 04, 2012:

Hi,

Thanks for notifying us on the security model and the suggestions.  While designing the CPT report, we had considered the existing sales report as the base.

- ViSolve

tmccormi wrote on Tuesday, June 05, 2012:

I don;t think using Procedure is a good at this time as the Orders system uses that term and it will be confusing.  Here’s what I have asked Visolve to do:
So:
- Menu - Financial Summary by Svc Code
- Use “Service Codes” in place of CPT in all labels
- LBF (list_options) title: “Service Codes for Financial Summary”  (no
underlines!)
- PHP pgm - “svc_code_financial_report.php”
- PHP pgm csv output name "svc_financial_report_YYYYMMDD-YYYYMMDD.csv

-Tony

bradymiller wrote on Tuesday, June 05, 2012:

Hi Tony,

Sounds good (I agree ‘Procedure’ is not a good idea).

-brady
OpenEMR

sunsetsystems wrote on Tuesday, June 05, 2012:

Added some commit comments.

Rod
www.sunsetsystems.com

tmccormi wrote on Wednesday, June 13, 2012:

We have incorporated the suggestions given and have committed the same to
our git
https://github.com/visolve-openemr/Openemr/commit/3d9b32218274d5659b721f064541245bc9178eb9

Please do review the same and share your views.

Thanks
DEVI.G

(posted by Tony)

bradymiller wrote on Wednesday, June 13, 2012:

Hi,

Placed a review on github.

thanks,
-brady
OpenEMR Project

weesnerkim wrote on Friday, June 29, 2012:

Hi

I like the looks of this one, I will review and comment. Good workguys.

Kim Weesner

bradymiller wrote on Tuesday, July 03, 2012:

Hi,

Just committed this to sourceforge (note the modifications I made on this listed in the commit comments).

-brady
OpenEMR