ICEfaces
  1. ICEfaces
  2. ICE-11293

ace:dateTimeEntry - Converter not called once component is unrendered

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-3.3.0.GA_P04, 4.2
    • Fix Version/s: EE-4.2.0.GA, EE-3.3.0.GA_P05
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      All

      Description

      An ace:dateTimeEntry component(s) is setup to use a converter via the converter attribute. If the component is unrendered (rendered=false) and then rendered again (rendered=true) the converter is no longer called when the value is submitted.

        Activity

        Hide
        Arran Mccullough added a comment -

        Attached test case that reproduces this issue.

        Steps:

        • Load welcomeICEfaces.jsf
        • In each of the dateTimeEntry fields, enter in some invalid numbers (For Example: 123456). Upon focus lost, the converter is called and an error message is shown.
        • Click the 'Render Panel' button, the dateTimeEntry fields are no longer displayed.
        • Click the button again to display the components again.
        • Do the same test as before, the converter error messages are no longer shown.
        Show
        Arran Mccullough added a comment - Attached test case that reproduces this issue. Steps: Load welcomeICEfaces.jsf In each of the dateTimeEntry fields, enter in some invalid numbers (For Example: 123456). Upon focus lost, the converter is called and an error message is shown. Click the 'Render Panel' button, the dateTimeEntry fields are no longer displayed. Click the button again to display the components again. Do the same test as before, the converter error messages are no longer shown.
        Hide
        Mircea Toma added a comment - - edited

        Avoid short-cutting the widget setup when lazy loading to ensure that the event listeners are registered. Previously the event listeners on date time entry's input element were not registered on lazy loading.

        Show
        Mircea Toma added a comment - - edited Avoid short-cutting the widget setup when lazy loading to ensure that the event listeners are registered. Previously the event listeners on date time entry's input element were not registered on lazy loading.
        Hide
        Liana Munroe added a comment -

        Verified ICEfaces EE-3.3.0 maintenance branch r51480, Tomcat 8, MS Edge, IE 11, 10, 9, 8, FF 47, Chrome 57

        Show
        Liana Munroe added a comment - Verified ICEfaces EE-3.3.0 maintenance branch r51480, Tomcat 8, MS Edge, IE 11, 10, 9, 8, FF 47, Chrome 57
        Hide
        Liana Munroe added a comment -

        Reopened to add fix to ICEfaces 4 trunk.

        Show
        Liana Munroe added a comment - Reopened to add fix to ICEfaces 4 trunk.
        Hide
        Mircea Toma added a comment -

        Applied fix to ICEfaces 4 trunk as well.

        Show
        Mircea Toma added a comment - Applied fix to ICEfaces 4 trunk as well.
        Hide
        Liana Munroe added a comment -

        Verified ICEfaces 4 trunk r51493, Tomcat 8, MS Edge, IE 11, 10, 9, 8, FF 47, Chrome 57.
        New test app added:
        http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Manual/ICE-11293

        Show
        Liana Munroe added a comment - Verified ICEfaces 4 trunk r51493, Tomcat 8, MS Edge, IE 11, 10, 9, 8, FF 47, Chrome 57. New test app added: http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Manual/ICE-11293

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Arran Mccullough
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: