[
https://issues.apache.org/jira/browse/WICKET-6325?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Sven Meier resolved WICKET-6325.
--------------------------------
Resolution: Fixed
Fix Version/s: 8.0.0-M4
> AbstractTree#updateBranch() and #updateNode() make AjaxRequestTarget mandatory
> ------------------------------------------------------------------------------
>
> Key: WICKET-6325
> URL: https://issues.apache.org/jira/browse/WICKET-6325
> Project: Wicket
> Issue Type: Bug
> Components: wicket-extensions
> Affects Versions: 8.0.0-M3
> Reporter: Sven Meier
> Assignee: Sven Meier
> Priority: Minor
> Fix For: 8.0.0-M4
>
>
> With the new Optional<AjaxRequestTarget> spreading over Wickets code base,
> AbstractTree#updateBranch() and #updateNode() were changed (by me) to get an
> optional ART.
> Further examination shows, that this was a wrong decision:
> - most Java experts argue against optional method parameters
> - there are cases now where an existing ART must be wrapped in an optional
> just to pass it into the methods.
> The former solution to check the parameter against null isn't feasible now
> either, since you have to use optional#orElse(null) if you happen to have an
> Optional<ART> only.
> Thus it is better to just make the ART parameter mandatory.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
