[
https://issues.apache.org/jira/browse/ISIS-1083?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14510639#comment-14510639
]
ASF subversion and git services commented on ISIS-1083:
-------------------------------------------------------
Commit e3e1c7ac60e085219e70fcae2fddbe49565d932b in isis's branch
refs/heads/master from [~erikdehair]
[ https://git-wip-us.apache.org/repos/asf?p=isis.git;h=e3e1c7a ]
ISIS-1083 resetting dependent choice when selecting another referenced
choice.
> Resetting dependent choice
> --------------------------
>
> Key: ISIS-1083
> URL: https://issues.apache.org/jira/browse/ISIS-1083
> Project: Isis
> Issue Type: Improvement
> Components: Core: Viewer: Wicket
> Reporter: Erik de Hair
> Assignee: Dan Haywood
> Priority: Trivial
>
> As discussed on the Isis users list:
> ============================================
> Seems reasonable. Would you like to provide a pull request? [1]
> [1] http://isis.apache.org/contributors/contributing.html
> On 2 March 2015 at 16:20, Erik de Hair <e.deh...@pocos.nl> wrote:
> > Hi Dan,
> >
> >
> > On 03/02/2015 03:35 PM, Dan Haywood wrote:
> >
> >> Is there any use of setting the selected option to the first item of the
> >> choices? It seems a bit strange from the user's perspective.
> >>
> >>
> >> The rationale, IIRC, is that if the current value is not valid in the new
> >> set of choices --- if(!curr.containedIn(choiceMementos)) --- then we set
> >> the dependent choice to one of the valid set available as a convenience.
> >>
> >> I suppose, strictly speaking, that this should only be done if the
> >> dependent parameter is mandatory; otherwise a user might want to leave it
> >> as null.
> >>
> > I think, even when the parameter is mandatory, this should be handled by
> > validation. It would be even better to set it to null because the user is
> > forced to choose an option and would otherwise possibly submit a selected
> > value the user didn't want to set (and didn't see it changing).
> >
> >>
> >> I can reset it in my code but maybe this scenario isn't desirable for any
> >>> Isis application.
> >>>
> >>>
> >>> (Without trying it out for myself), I'm trying to think as to how you'd
> >> do
> >> that ... in the defaultNXxx() method, I am guessing?
> >>
> >> By copying the ReferencePanel-class to my app Just to try if it
> > works when setting it to null.
> >
> >>
> >> Cheers
> >> Dan
> >>
> >>
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
