openmrs/openmrs-core — pull request #161 by gsluthra (2012-12-26 14:27:51+00:00)
Title: Trunk 2119 -- Update Exception Reporting Screen to ask for User's Name, Subject, Email and Recent Steps

Enhanced the exception reporting page to ask for User's Name, Subject, Email and Recent steps.
See JIRA for how the screen now looks.

Here are some of my comments on this pull request:

1. All code has been written using plain Javascript. jQuery wasn't available to page. Should I have instead used jQuery?
2. Double quotes in text area may break the javascript.
3. Since I wasn't sure how to control the JIRA variable reading, I concatenated user Name & Email and Recent steps to Stack Trace element, so it shows up nicely in Description section of the issue. And, I modified error summary element to contain user entered summary.
4. The email field has no email validation. Suggestions? Is it needed? It would be good to have I think.
5. Subject is mandatory. Other fields are optional.
6. Unit Tests? None have been written as of now.