ICEfaces
  1. ICEfaces
  2. ICE-8467

REGRESSION: ACE:DateTimeEntry - Unable to clear popup calendar input

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.1, EE-3.0.0.GA_P01
    • Fix Version/s: EE-3.0.0.GA_P01, 3.2
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      icefaces-3.0.x-maintenance revision# 30501
      Last passing revision# 30469
      Browser: IE, Chrome, Firefox

      Description

      Clearing the dateTimeEntry input of a popup calendar and submitting the field causes the input to automatically repopulate it's value with the value that was present before clearing the field.

      Tests are located at: http://server.ice:8888/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Nightly/dateTimeEntry

      To reproduce:
      1. Build / deploy test app
      2. Navigate to '/calendarAttribute.jsf'
      3. Clear the dateTimeEntry input field and push enter
      Notice the value of the dateTimeEntry returns
      1. calendarAttribute.xhtml
        6 kB
        Ken Fyten
      1. screenshot-01.png
        203 kB
      2. screenshot-02.png
        204 kB
      3. screenshot-03.png
        152 kB

        Activity

        Hide
        yip.ng added a comment -

        Reverted adding value property to meta class. Caused regression in ICE-8508. Need to find another fix. Re-opened.

        Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\datetimeentry\DateTimeEntryMeta.java
        Completed: At revision: 30658

        Modified: C:\svn\ossrepo\icefaces3\branches\icefaces-3.0.x-maintenance\icefaces\ace\component\src\org\icefaces\ace\component\datetimeentry\DateTimeEntryMeta.java
        Completed: At revision: 30659

        Show
        yip.ng added a comment - Reverted adding value property to meta class. Caused regression in ICE-8508. Need to find another fix. Re-opened. Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\datetimeentry\DateTimeEntryMeta.java Completed: At revision: 30658 Modified: C:\svn\ossrepo\icefaces3\branches\icefaces-3.0.x-maintenance\icefaces\ace\component\src\org\icefaces\ace\component\datetimeentry\DateTimeEntryMeta.java Completed: At revision: 30659
        Hide
        yip.ng added a comment - - edited

        Not working as claimed at rev. 30469, neither trunk nor maintenance branch.

        Works if I minimalize the test page to just contain the calendar itself. See video: http://screencast.com/t/NvQggYPvn. Something else on the page (lots of other things on the page) is causing the problem?

        It's caused by another input field on the test page. See screenshot-03.png.

        Show
        yip.ng added a comment - - edited Not working as claimed at rev. 30469, neither trunk nor maintenance branch. Works if I minimalize the test page to just contain the calendar itself. See video: http://screencast.com/t/NvQggYPvn . Something else on the page (lots of other things on the page) is causing the problem? It's caused by another input field on the test page. See screenshot-03.png.
        Hide
        yip.ng added a comment -

        From Cruz:

        Behaviour In Firefox 3.6 at revision# 30469 with required = true:
        Clearing the calendar input field and pushing enter causes the required message to be displayed and the field to be cleared.

        Behaviour In Firefox 3.6 at revision# 30514 with required = true:
        Clearing the calendar input field and pushing enter does not cause the required message to be displayed because the field is not cleared.

        Show
        yip.ng added a comment - From Cruz: Behaviour In Firefox 3.6 at revision# 30469 with required = true: Clearing the calendar input field and pushing enter causes the required message to be displayed and the field to be cleared. Behaviour In Firefox 3.6 at revision# 30514 with required = true: Clearing the calendar input field and pushing enter does not cause the required message to be displayed because the field is not cleared.
        Hide
        Ken Fyten added a comment -

        Behaviour In Firefox 3.6 at revision# 30514 with required = true:
        Clearing the calendar input field and pushing enter does not cause the required message to be displayed because the field is not cleared.

        This is fine. It's not possible to remove a previously entered value in JSF on a "required" value.

        Does it clear okay if required=false?

        Show
        Ken Fyten added a comment - Behaviour In Firefox 3.6 at revision# 30514 with required = true: Clearing the calendar input field and pushing enter does not cause the required message to be displayed because the field is not cleared. This is fine. It's not possible to remove a previously entered value in JSF on a "required" value. Does it clear okay if required=false?
        Hide
        Ken Fyten added a comment -

        Attached fixed calendarAttribute.xhtml file.

        The fixes for this test are to include the "output" outputText component in the form with the ace:dateTimeEntry, and to limit the execution of the ace:dateTimeEntry on top to itself, to avoid constantly failing due to the empty "required=true" ace:dateTimeEntry at the bottom.

        Might want to consider removing the bottom ace:dateTimeEntry altogether, or moving into a page of it's own.

        Show
        Ken Fyten added a comment - Attached fixed calendarAttribute.xhtml file. The fixes for this test are to include the "output" outputText component in the form with the ace:dateTimeEntry, and to limit the execution of the ace:dateTimeEntry on top to itself, to avoid constantly failing due to the empty "required=true" ace:dateTimeEntry at the bottom. Might want to consider removing the bottom ace:dateTimeEntry altogether, or moving into a page of it's own.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: