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

              Ascending order - Click to sort in descending order
              Arran Mccullough created issue -
              Hide
              Permalink
              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.
              Arran Mccullough made changes -
              Field Original Value New Value
              Attachment Case14087Example.war [ 22505 ]
              Attachment Case14087Example.zip [ 22506 ]
              Ken Fyten made changes -
              Assignee Mircea Toma [ mircea.toma ]
              Fix Version/s EE-4.2.0.GA [ 13071 ]
              Fix Version/s EE-3.3.0.GA_P05 [ 13082 ]
              Assignee Priority P1 [ 10010 ]
              Hide
              Permalink
              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.
              Mircea Toma made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              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
              Permalink
              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.
              Liana Munroe made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Hide
              Permalink
              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.
              Mircea Toma made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #51485 Mon May 08 16:50:27 MDT 2017 mircea.toma ICE-11293 Avoid short-cutting the widget setup when lazy loading to ensure that the event listeners are registered.
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/datetimeentry/datetimeentry.js
              Hide
              Permalink
              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
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: