Bug in New/Search popup window

tmccormi wrote on Wednesday, August 19, 2015:

The window will not page to the next 100 records when you click the >> link. Not easy to test on the pubic sites as there are never more than 100 patients on file.

Anyone else have this? I’ve tested it on versions 4.2 and devtip

fsgl wrote on Wednesday, August 19, 2015:

Forward does not work in New/Search, but Next works in Patients.

tmccormi wrote on Wednesday, August 19, 2015:

Since those are completely different programs it does not surprise me at all … one of the big issues with evolved software is the lack of shared code for common functions …

Tony

bradymiller wrote on Thursday, August 20, 2015:

one of the big issues with nonexistant software is the lack of any functions :slight_smile:

bradymiller wrote on Thursday, August 20, 2015:

Would be nice to have a testing server that brings in several thousand patients. If somebody creates this via Art’s package, could likely get it to import into one of the development demos.
-brady

fsgl wrote on Thursday, August 20, 2015:

Testing had been done in production copy ( which has more than 4 patients).

  1. png attached below to avoid breach of HIPAA.

growlingflea wrote on Thursday, August 20, 2015:

It looks like when the new window pops up, the top.restoreSession() loses scope. This was fixed by doing the following code change in interface/main/finder/patient_select.php

Here is the pull request.

bradymiller wrote on Saturday, August 22, 2015:

Hi Daniel,
Thanks for the fix. Just committed this fix to the codebase.
-brady
OpenEMR