I discovered there was another deprecated updatePatientProgram method that
appears to be getting called.  Deleting this method fixes both problems #2
and 3.  Why does the wrong method get called?

@Deprecated
    public void updatePatientProgram(Integer patientProgramId, String
enrollmentDateYmd, String completionDateYmd)
            throws ParseException {
        updatePatientProgram(patientProgramId, enrollmentDateYmd,
completionDateYmd, null);
    }

On Fri, Aug 5, 2011 at 1:11 AM, Greg Warren <[email protected]>wrote:

> Hi, I'm working on 
> TRUNK-2329<https://tickets.openmrs.org/browse/TRUNK-2329>and have several 
> issues when clicking on the link to edit a patient
> program.  I have latest trunk code.
>
> 1)  I get "A javascript error has occurred: *dateToFormat.getDate is not a
> function"* when the popup opens. I think this is because there are two
> formatDate methods:
>
>   patientPrograms.jsp: function formatDate(ymd)
>   patientGraphs.jsp function formatDate(dateToFormat)
>
> It seems the wrong one from patientGraphs is being called.  Renaming one of
> them seems to fix that problem.
>
> 2)  The save button does save, but the callback method does not seem to be
> running so the popup does not close and the page does not refresh.  I think
> the delete callback was broken too.
>
> 3)  Maybe this is related to #2, but when saving the locationId is always
> null.  The javascript variable locationId is set to a string value, but the
> locationId parameter in DWRProgramWorkflowService.updatePatientProgram
> method is always null.
>
> Can anyone else reproduce these issues?
>

_________________________________________

To unsubscribe from OpenMRS Developers' mailing list, send an e-mail to 
[email protected] with "SIGNOFF openmrs-devel-l" in the  body (not 
the subject) of your e-mail.

[mailto:[email protected]?body=SIGNOFF%20openmrs-devel-l]

Reply via email to