Code for reverse sorting of past appointments, written by Terry Hill

growlingflea wrote on Thursday, August 14, 2014:

Hi all. Tony wanted me to let the forum know about a helpful snippet of code written by Terry Hill. This code allows reverse sorting of past appointments if the user puts a negative number in the global field. The link to the repo with Terry’s addition can be found here:

The branch is called ‘terrysbranch’.

tmccormi wrote on Thursday, August 14, 2014:

Will port this to dev tip (it’s 412 based) and merge it later this evening if we hear no objections. Terry is working on some other cool stuff, like UB04 support as well.

PS: Daniel is my newest employee and is soaking up OpenEMR modalities rapidly…

Tony

sunsetsystems wrote on Thursday, August 14, 2014:

I put in 3 code review comments. Would like to see those addressed before pushing to SF. Otherwise looks good (didn’t test). Thanks.

Rod
http://www.sunsetsystems.com/

bradymiller wrote on Sunday, August 17, 2014:

Hi,
I also placed a review on github(agree with Rod and some minor translation comments).

Looking forward to more code from Daniel :slight_smile:

-brady
OpenEMR

tmccormi wrote on Monday, August 18, 2014:

Daniel will certainly be contributing code of his own soon enough… :slight_smile:

Tony McCormick, CTO
Medical Information Integration, LLC

Direct: 713-574-6709, cell 503-330-2239
Office: 866-735-0897
@MI2_OpenEMR

On Sun, Aug 17, 2014 at 1:27 AM, Brady Miller bradymiller@users.sf.net
wrote:

Hi,
I also placed a review on github(agree with Rod and some minor translation
comments).

Looking forward to more code from Daniel :slight_smile:

-brady
OpenEMR http://www.open-emr.org

Code for reverse sorting of past appointments, written by Terry Hill
https://sourceforge.net/p/openemr/discussion/202506/thread/9b50fe1d/?limit=25#e261

Sent from sourceforge.net because you indicated interest in
OpenEMR / Discussion / Developers

To unsubscribe from further messages, please visit
SourceForge.net: Log In to SourceForge.net


Please be aware that e-mail communication can be intercepted in
transmission or misdirected. Please consider communicating any sensitive
information by telephone. The information contained in this message may be
privileged and confidential. If you are NOT the intended recipient, please
notify the sender immediately with a copy to hipaa-security@mrsb-ltd.com and
destroy this message.

teryhill wrote on Monday, August 18, 2014:

Tony did you get the changes I sent you to address the issues that Rod and Brady had?

Thanks

Terry

bradymiller wrote on Wednesday, September 10, 2014:

Hi Terry,

I have committed your code to the codebase:
https://sourceforge.net/p/openemr/code/ci/8278e2cb08875734b5028a3ad5affe8b6ba95cd3

Thank you for the contribution!,
-brady
OpenEMR

mdsupport wrote on Wednesday, October 01, 2014:

Queries used for the appointment widgets do not consider recurring appointments accurately. In case anyone is interested, we use attached code based on appointments standard function. Create and test the patch for demographics.php as follows:

  1. Add the following line somewhere
    require_once("$srcdir/appointments.inc.php");
  2. Replace the code between the following lines by attached file
    // Show current and upcoming appointments.

    // END of past appointments

To keep appointments relevant, code looks for 3 years in past and 1 year in future. Obviously there are better solutions and different approaches. This is one of the possibilities.

Sorry, can’t provide a patch - severely diverged UI & functionality differences.