ICEfaces
  1. ICEfaces
  2. ICE-10751

ice:selectInputDate and ice:selectInputText do not work correctly within ace:panelStack

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: EE-3.3.0.GA_P04
          • Fix Version/s: EE-3.3.0.GA_P04
          • Component/s: ACE-Components, ICE-Components
          • Labels:
            None
          • Environment:
            3.3.0-mtce branch and custom patch

            Description

            deploy iceMock3.war and then proceed to Medium or Large button page.
            The icon for calendar is clicked on, and the request is sent but the popup does not appear.

              Issue Links

              depends on

              New Feature - A new feature of the product, which has yet to be developed. ICE-10638 New ace:panelStack with custom tag-handler rendering optimization and client capabilities

              • Major - Major loss of function.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Judy Guglielmin created issue -
                Judy Guglielmin made changes -
                Field Original Value New Value
                Assignee Mircea Toma [ mircea.toma ]
                Hide
                Permalink
                Judy Guglielmin added a comment -

                smb://iceads1.icesoft.domain/Public/Users/mircea/3.3.0_MtceBranch_iceMock3/iceMock3.war

                is location to the test project showing this problem and using the 3.3.0_Mtce branch libraries (same result as the latest build using rev45264 (which closed ICE-10638)

                Note, deploy this test app and choose "medium" then try to use the popup calendar icon to choose a date from the ice:selectInputDate component.

                Show
                Judy Guglielmin added a comment - smb://iceads1.icesoft.domain/Public/Users/mircea/3.3.0_MtceBranch_iceMock3/iceMock3.war is location to the test project showing this problem and using the 3.3.0_Mtce branch libraries (same result as the latest build using rev45264 (which closed ICE-10638 ) Note, deploy this test app and choose "medium" then try to use the popup calendar icon to choose a date from the ice:selectInputDate component.
                Judy Guglielmin made changes -
                Link This issue depends on ICE-10638 [ ICE-10638 ]
                Ken Fyten made changes -
                Fix Version/s 4.1 [ 11375 ]
                Fix Version/s EE-3.3.0.GA_P04 [ 12270 ]
                Assignee Priority P1 [ 10010 ]
                Ken Fyten made changes -
                Fix Version/s 4.1 [ 11375 ]
                Repository Revision Date User Message
                ICEsoft Public SVN Repository #45737 Wed Jul 08 18:27:09 MDT 2015 mircea.toma ICE-10751 Save and restore child state during JSF lifecycle (in case the child is re-instantiated) and in-between lifecycles. This ensures the stack pane's children have their state preserved in between post-backs.
                Files Changed
                Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/stackpane/StackPane.java
                Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/stackpane/StackPaneRenderer.java
                Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/fileentry/fileEntry.js
                Hide
                Permalink
                Mircea Toma added a comment -

                Save and restore child state during JSF lifecycle (in case the child is re-instantiated) and in-between lifecycles. This ensures the stack pane's children have their state preserved in between post-backs.

                Show
                Mircea Toma added a comment - Save and restore child state during JSF lifecycle (in case the child is re-instantiated) and in-between lifecycles. This ensures the stack pane's children have their state preserved in between post-backs.
                Mircea Toma made changes -
                Status Open [ 1 ] Resolved [ 5 ]
                Resolution Fixed [ 1 ]
                Hide
                Permalink
                Judy Guglielmin added a comment - - edited

                There is a serious regression with this last commit: The ace checkboxButton and comboBox no longer maintain their state with this revision in showcase example. The simple example you can click on checkboxButton and it will automatically deselect itself right after. If you try the panelSeries or database example, select query2, click on a checkbox, select a different query1 and then go back to query2, you will see that the checkbox has the original state (unchecked).

                Show
                Judy Guglielmin added a comment - - edited There is a serious regression with this last commit: The ace checkboxButton and comboBox no longer maintain their state with this revision in showcase example. The simple example you can click on checkboxButton and it will automatically deselect itself right after. If you try the panelSeries or database example, select query2, click on a checkbox, select a different query1 and then go back to query2, you will see that the checkbox has the original state (unchecked).
                Judy Guglielmin made changes -
                Resolution Fixed [ 1 ]
                Status Resolved [ 5 ] Reopened [ 4 ]
                Hide
                Permalink
                Liana Munroe added a comment - - edited

                Tested iceMock3.war with ICEfaces EE-3.3.0 maintenance branch r45737 libs.
                The issue appears to be fixed.

                The regression issue reported with showcase was also found when tested with FF 34, Chrome 43 and IE 11.

                Show
                Liana Munroe added a comment - - edited Tested iceMock3.war with ICEfaces EE-3.3.0 maintenance branch r45737 libs. The issue appears to be fixed. The regression issue reported with showcase was also found when tested with FF 34, Chrome 43 and IE 11.
                Repository Revision Date User Message
                ICEsoft Public SVN Repository #45744 Thu Jul 09 17:00:55 MDT 2015 mircea.toma ICE-10751 Make sure children are created before visiting them during decoding phases. Also save children's state after visiting them.
                Files Changed
                Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/stackpane/StackPane.java
                Hide
                Permalink
                Mircea Toma added a comment -

                Make sure children are created before visiting them during decoding phases. Also save children's state after visiting them.

                Show
                Mircea Toma added a comment - Make sure children are created before visiting them during decoding phases. Also save children's state after visiting them.
                Mircea Toma made changes -
                Status Reopened [ 4 ] Resolved [ 5 ]
                Resolution Fixed [ 1 ]
                Hide
                Permalink
                Liana Munroe added a comment -

                Verified showcase regression resolved ICEfaces4 trunk, EE-3.3.0 maintenance branch r45747. Tomcat 7, FF 34, Chrome 43.

                Show
                Liana Munroe added a comment - Verified showcase regression resolved ICEfaces4 trunk, EE-3.3.0 maintenance branch r45747. Tomcat 7, FF 34, Chrome 43.
                Repository Revision Date User Message
                ICEsoft Public SVN Repository #45751 Mon Jul 13 12:11:24 MDT 2015 mircea.toma ICE-10751 Discard view scoped beans before restore phase to avoid giving a chance to EL resolver to find them after navigation.
                Files Changed
                Commit graph MODIFY /icefaces4/trunk/icefaces/core/src/main/java/org/icefaces/impl/application/WindowScopeManager.java
                Ken Fyten made changes -
                Resolution Fixed [ 1 ]
                Status Resolved [ 5 ] Reopened [ 4 ]
                Hide
                Permalink
                Mircea Toma added a comment -

                The previous comment describes the related ICE-10749 issue.

                Show
                Mircea Toma added a comment - The previous comment describes the related ICE-10749 issue.
                Mircea Toma made changes -
                Status Reopened [ 4 ] Resolved [ 5 ]
                Resolution Fixed [ 1 ]
                Ken Fyten made changes -
                Status Resolved [ 5 ] Closed [ 6 ]

                  People

                  • Assignee:
                    Mircea Toma
                    Reporter:
                    Judy Guglielmin
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    4 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: