ICEfaces
  1. ICEfaces
  2. ICE-4827

selectInputDate "required" attribute not working when calendar popup is open

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.8.1
    • Fix Version/s: 1.8.2-RC1, 1.8.2
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      All

      Description

      This is a follow-on issue from ICE-2927 which was fixed for 1.7.2.


      Stuart Campbell added a comment - 17/Jul/09 02:36 AM
      required="true" isn't honoured when the popup is open. When a page is loaded, if the popup is opened, then a submit occurs, it will just submit null without complaint.

      [ Permlink | « Hide ]
      Joanne Bai added a comment - 21/Jul/09 03:42 PM - Visible to icesoft-internal-developers
      QA verified that when the calendar popup is open, with required="true", there's no validation on null date input.

      Verified on ICEfaces trunk subversion: 19088
      The test app can be found at repo\qa\trunk\Regression\ICE-2927

      To reproduce:
      - build the test app using ant clean tomcat6.0, and deploy it to tomcat6
      - in the second row, click on the calendar button to bring up the popup
      - click on the Submit button at the bottom. There's no validation for the second calendar
      - if click Submit while the popup is closed, the second calendar will have a validation

        Activity

        Hide
        yip.ng added a comment -

        This problem is actually the same as ICE-4584. I ran the regression test for ICE-2927 and traced some code. The problem as described by Joanne existed only before the fix for ICE-4584. After the fix for ICE-4584, the validation occurs correctly even for the popup.

        Show
        yip.ng added a comment - This problem is actually the same as ICE-4584 . I ran the regression test for ICE-2927 and traced some code. The problem as described by Joanne existed only before the fix for ICE-4584 . After the fix for ICE-4584 , the validation occurs correctly even for the popup.
        Hide
        Joanne Bai added a comment -

        QA verified it with success on trunk revision 19160 (use test case ICE-2927 to test)

        Show
        Joanne Bai added a comment - QA verified it with success on trunk revision 19160 (use test case ICE-2927 to test)

          People

          • Assignee:
            yip.ng
            Reporter:
            Ken Fyten
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: