[
https://issues.apache.org/jira/browse/WICKET-6264?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15621584#comment-15621584
]
ASF GitHub Bot commented on WICKET-6264:
----------------------------------------
GitHub user barney2k7 opened a pull request:
https://github.com/apache/wicket/pull/186
WICKET-6264: Form: improve error message for disabled/invisible IForm…
…SubmittingComponent
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/barney2k7/wicket WICKET-6264
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/wicket/pull/186.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #186
----
commit afe1e333c662d6e15842b8112eedd20ae2223646
Author: barney2k7 <[email protected]>
Date: 2016-10-31T08:15:43Z
WICKET-6264: Form: improve error message for disabled/invisible
IFormSubmittingComponent
----
> Form: improve error message for disabled/invisible IFormSubmittingComponent
> ---------------------------------------------------------------------------
>
> Key: WICKET-6264
> URL: https://issues.apache.org/jira/browse/WICKET-6264
> Project: Wicket
> Issue Type: Improvement
> Components: wicket
> Affects Versions: 7.5.0
> Reporter: Jonas
> Priority: Minor
>
> org.apache.wicket.markup.html.form.Form.onFormSubmitted(IFormSubmitter)
> throws WicketRuntimeExceptions in case the IFormSubmittingComponent has is
> not/no longer enabled or visible.
> A logged message looks like this:
> {noformat}
> 2016-10-28 11:49:27,289 ERROR [org.apache.wicket.DefaultExceptionMapper] -
> Unexpected error occurred
> org.apache.wicket.WicketRuntimeException: Submit Button ok (path=form:ok) is
> not enabled
> at
> org.apache.wicket.markup.html.form.Form.onFormSubmitted(Form.java:772)
> at
> org.apache.wicket.markup.html.form.Form.onFormSubmitted(Form.java:708)
> ...
> {noformat}
> This kind of problem is currently quite hard to fix, as neither the error
> message nor the stack trace contains any hint what page is affected.
> I would like to propose to add e.g. the pages' class to both error messages,
> e.g. "Submit Button ok (path=form:ok) on page com.foo.bar.WebPage is not
> enabled" and "...visible"
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
