Difference between revisions of "Medmasterpro API Review"
From OpenEMR Project Wiki
Bradymiller (talk | contribs) |
Bradymiller (talk | contribs) |
||
| Line 34: | Line 34: | ||
==addvisitvitals.php== | ==addvisitvitals.php== | ||
==classes.php== | ==classes.php== | ||
:*The site variable will need to be dealt with at some point. Can do this later int he review process after have a better idea of the code flow. | |||
==deleteappointment.php== | ==deleteappointment.php== | ||
==deletecontactgeneral.php== | ==deletecontactgeneral.php== | ||
Revision as of 23:53, 16 March 2013
Overview
- This is too review the Medmasterpro api code at http://github.com/medmasterpro/openemr . It gets it's own wiki page because it is an extensive and exciting ongoing project.
Functions
Overview
- These are all in the api directory.
Global Issues
- Change the 'push_notification' global to something more specific like 'device_push_notification_service'
Core functions/scripts in the includes directory
addappointment.php
- Surround the entire $device_token_badge with the 'push_notification' global switch.
- In $strQuery suery, need single quotes around the add_escape_custom($patientId)
- All the getUserData() function does is return two separate but identical variables with the getUsername() function. Please clean this up a bit, since it appears all you need is a $user = getUsername($userId) and no need for the other variables (emr/password/username).
addcheckout.php
addcontactgeneral.php
addfacility.php
addfeesheet.php
addinsurancecompany.php
addlist.php
addonotes.php
addpatientdocument.php
addpatientdocumentwithlink.php
addpatientnotes.php
addpatient.php
addpayment.php
addprescription.php
addresource.php
addresourcewithlink.php
addreviewofsystems.php
addroschecks.php
addsoap.php
addvisit.php
addvisitvitals.php
classes.php
- The site variable will need to be dealt with at some point. Can do this later int he review process after have a better idea of the code flow.
