Downloads Documentation Community Contribute Demo






Show Sidebar
Login | Register

Ticket #2069 (closed defect: fixed)

Opened 1 month ago

Last modified 3 days ago

nameLayout.jsp Error with duplicate variable 'status' preventing patient entry and admin tasks

Reported by: syhaas Assigned to: syhaas
Priority: major Milestone: OpenMRS 1.6
Component: OpenMRS Code Base Keywords:
Cc: djazayeri Introductory Ticket: 0
Code Review Status: Approved

Description (Last modified by bwolfe)

The error received is:

ERROR - errorhandler_jsp._jspService(72) |2010-02-02 14:36:43,187| Error on page /openmrs/WEB-INF/view/admin/patients/newPatientForm.jsp
javax.servlet.jsp.JspException: org.apache.jasper.JasperException: Unable to compile class for JSP:

An error occurred at line: 200 in the jsp file: /WEB-INF/view/portlets/nameLayout.jsp
Duplicate local variable status
197:
        <input type="hidden" name="_${status.expression}"/>
198:
        <input type="checkbox" name="${status.expression}"
199:
                   <c:if test="${status.value == true}">checked="checked"</c:if>

200:
                   onClick="toggleLayer('<spring:bind path="personNameId">voidReasonRow-${status.value}</spring:bind>'); if (voidedBoxClicked) voidedBoxClicked(this); "
201:
        />
202:
</spring:bind>


Potential fix: move the inner <spring:bind> out of the encompassing <spring:bind>

Attachments

2069.patch (0.9 kB) - added by syhaas on 02/04/10 20:19:06.
nameLayout.jsp

Change History

02/04/10 20:06:10 changed by syhaas

  • status changed from new to assigned.
  • type changed from task to defect.
  • description changed.

02/04/10 20:19:06 changed by syhaas

  • attachment 2069.patch added.

nameLayout.jsp

02/04/10 20:19:16 changed by syhaas

  • review_status set to Needs Review.

02/05/10 14:29:22 changed by bwolfe

  • status changed from assigned to closed.
  • review_status changed from Needs Review to Approved.
  • resolution set to fixed.
  • description changed.

Applied the patch to trunk in [11983] and to 1.6.x in [11984]. Thanks Sy!

03/11/10 19:20:07 changed by mharrison

  • status changed from closed to reopened.
  • resolution deleted.

This is a problem in the 1.5.x branch too. Ben can you apply the patch? I'm doing it locally so I can continue testing.

03/11/10 21:44:24 changed by bwolfe

  • cc set to djazayeri.

Darius, since you're in the backporting mood, want to do this one as well?

03/15/10 19:09:34 changed by djazayeri

  • status changed from reopened to closed.
  • resolution set to fixed.

Backported to 1.5.x in [12464].