-
Bug
-
Resolution: Unresolved
-
Minor
-
None
If a hudson.model.Descriptor.FormException is thrown then the resulting error shown to the user does not include the filed that caused the error.
this leads to either having to duplicate the field name in the message, or a page that is mostly useless to the user as they do not know which field the error refers to.
Because of this it is preferable to throw a different unchecked exception even though this is the expected exception for form binding errors, so that the details are at least present in the logs
reproduction steps
in a @DataboundConstructor throw a new FormException("can not be null", "theField")
Attempt to create a new instance of the describable
Expected Results
the page displayed to the user contains the error message "can not be null" as well as a reference to the field `theField`
Actual results
only the message is show
this is because the code ignores the field here
This is also worse since https://github.com/jenkinsci/jenkins/pull/9495 as the exception no longer triggers an angry Jenkins and prints the line in the logs to correlate.