[jira] [Updated] (ISIS-1083) Resetting dependent choice

["Dan Haywood (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/ISIS-1083?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Dan Haywood updated ISIS-1083:
------------------------------
    Fix Version/s:     (was: 1.10.0)
                   1.9.0

> 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
>             Fix For: 1.9.0
>
>
> 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....@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)