ICEfaces
  1. ICEfaces
  2. ICE-10736

ace:simpleSelectOneMenu fails validation for noSelectionOption even when required="false"

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.0
    • Fix Version/s: 4.1
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      Any, tested with Emporium trunk
    • Assignee Priority:
      P2

      Description

      When using <f:selectItem noSelectionOption="true"> as a child of ace:simpleSelectOneMenu the validation will still fire (saying "Value is not valid") even if required="false" (or is not specified).

      This validation should ONLY fire when required="true", as per h:selectOneMenu.

        Activity

        Hide
        Arturo Zambrano added a comment -

        This is actually working correctly right now, the same way as h:selectOneMenu. The only difference is that you need to specify a value to the selectItem that has noSelectionOption="true". It can just be the empty String.

        This JIRA will be redirected to cover the case when selectItems using noSelectionOption="true" have no value.

        Show
        Arturo Zambrano added a comment - This is actually working correctly right now, the same way as h:selectOneMenu. The only difference is that you need to specify a value to the selectItem that has noSelectionOption="true". It can just be the empty String. This JIRA will be redirected to cover the case when selectItems using noSelectionOption="true" have no value.
        Hide
        Arturo Zambrano added a comment -

        r45099: added logic to cover the case when noSelectionOption="true" and no value is specified in the SelectItem

        Testing Notes: Please test that when selecting a SelectItem that has noSelectionOption="true" a validation error is only fired when the component is required. The validation error shouldn't fire when the component is not required, regardless of whether the SelectItem has a value specified or not.

        Show
        Arturo Zambrano added a comment - r45099: added logic to cover the case when noSelectionOption="true" and no value is specified in the SelectItem Testing Notes: Please test that when selecting a SelectItem that has noSelectionOption="true" a validation error is only fired when the component is required. The validation error shouldn't fire when the component is not required, regardless of whether the SelectItem has a value specified or not.
        Hide
        Carmen Cristurean added a comment -

        Verified with the QA test application, /noSelectionOptionTest.jsf with ICEfaces4 trunk revision 45251/ IE11, CHrome43, FF34.

        Show
        Carmen Cristurean added a comment - Verified with the QA test application, /noSelectionOptionTest.jsf with ICEfaces4 trunk revision 45251/ IE11, CHrome43, FF34.

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Carlo Guglielmin
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: