Details
-
Bug
-
Status: Closed
-
Must
-
Resolution: Fixed
-
OpenMRS 1.9.3
-
None
-
None
-
Medium
Description
To reproduce:
1) Create a simple concept, with a fully-specified name and a synonym
2) Now go back and blank out the synonym, but don't hit the "remove" button
3) Also make a change to the fully-specifed name
4) Submit the form
A couple things go wrong here:
1) You get an "ugly" error message: Property synonymsByLocale[ht][0].name threw exception; nested exception is java.lang.NullPointerException
2) If you look in the DB, you will see that the fully-specified name has been changed to the name you set in step #3 above. This should not have happened, because there was a validation failure. On top of this, the "name" column of a concept_name row should never be changed directly--whenever a concept name is changed, the old row should be voided and the new row should be created.
I'm not quite sure where the flush happens that causes this error, but note that the "onSubmit" method of the ConceptFormController is never reached. The ConceptFormValidator is called during the call to super.processFormSubmission(request, response, object, errors) which happens within ConceptFormController.processFormSubmission.
Gliffy Diagrams
Attachments
Issue Links
- is related to
-
TRUNK-2969 New visit save errors should not clear new encounter rows
-
- Closed
-
- testing discovered
-
TRUNK-3951 NotTransactional not working correctly
-
- Closed
-