[
https://issues.apache.org/jira/browse/WICKET-6412?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16092068#comment-16092068
]
ASF subversion and git services commented on WICKET-6412:
---------------------------------------------------------
Commit 3e6aa9b9f8ae447a2f99e6192aebe45b05761a8f in wicket's branch
refs/heads/master from [~mgrigorov]
[ https://git-wip-us.apache.org/repos/asf?p=wicket.git;h=3e6aa9b ]
WICKET-6412 Model#orElse behavior is inconsistent!
Add javadoc to IModel#orElse() with possible usages.
> Model#orElse behavior is inconsistent!
> --------------------------------------
>
> Key: WICKET-6412
> URL: https://issues.apache.org/jira/browse/WICKET-6412
> Project: Wicket
> Issue Type: Improvement
> Affects Versions: 8.0.0-M6
> Reporter: Jarosław
> Assignee: Martin Grigorov
> Fix For: 8.0.0-M7
>
>
> Dear Wicket Devs,
> I was very confused by Model#orElse method.
> Model and Optional have similar methods and these methods behave the same:
> {code}
> Optional<Integer> _optional = Optional.ofNullable("1").map(Integer::parseInt);
> IModel<Integer> _model = Model.of("1").map(Integer::parseInt);
> {code}
> Unfortunately "orElse" behavior is totally confusing!
> {code}
> Integer integer = _optional.orElse(0);
> IModel<Integer> integerIModel = _model.orElse(0); //WRONG!
> {code}
> It should return a Model's Object instead of Model itself!
> This definitely break the least surprise rule, so can you please change that
> before Wicket8 final release?
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
