ICEfaces
  1. ICEfaces
  2. ICE-11410

ace:simpleSelectMenu is not applying class 'ui-state-error' when JSF validator provides invalid state

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-3.3.0.GA_P05
    • Fix Version/s: 4.3, EE-3.3.0.GA_P06
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      ace JSF validator

      Description

      see attached test case. Tested with clients P04 patch as well as P05 release. This may also affect ICEfaces 4.2.
      the ace:textEntry does show the invalid css class when invalid (there are 2 input controls on the form). A validator is attached to both the ace:textEntry and the ace:simpleSelectMenu. The form gets submitted but the correct css is not being applied to the simpleSelectMenu component as it does for the ace:textEntry.
      1. SimpleSelectOneMenuValidator.mov
        2.00 MB
        Judy Guglielmin

        Activity

        Judy Guglielmin created issue -
        Judy Guglielmin made changes -
        Field Original Value New Value
        Attachment IceFaces-testcase-SimpleSelectOneMenu-validation.zip [ 22913 ]
        Judy Guglielmin made changes -
        Judy Guglielmin made changes -
        Attachment SimpleSelectOneMenuValidator.mov [ 22915 ]
        Judy Guglielmin made changes -
        Fix Version/s EE-3.3.0.GA_P06 [ 13114 ]
        Judy Guglielmin made changes -
        Assignee Arturo Zambrano [ artzambrano ]
        Ken Fyten made changes -
        Fix Version/s 4.3 [ 13096 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #52492 Fri Jun 22 14:10:40 MDT 2018 art.zambrano ICE-11410 added check to see if a custom validator set the component to invalid, in order to exit the validation process and thus avoid performing more validation logic that could set the component back to valid
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/selectmenu/SelectMenu.java
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/simpleselectonemenu/SimpleSelectOneMenu.java
        Hide
        Arturo Zambrano added a comment -

        r52491, r52492: added check to see if a custom validator set the component to invalid, in order to exit the validation process and thus avoid performing more validation logic that could set the component back to valid

        Committed to 4.3 trunk and EE 3.3 maintenance branch.

        The issue was present in ace:selectMenu as well.

        Show
        Arturo Zambrano added a comment - r52491, r52492: added check to see if a custom validator set the component to invalid, in order to exit the validation process and thus avoid performing more validation logic that could set the component back to valid Committed to 4.3 trunk and EE 3.3 maintenance branch. The issue was present in ace:selectMenu as well.
        Arturo Zambrano made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: