ICEfaces
  1. ICEfaces
  2. ICE-2018

tree: partialSubmit in nested inputText components causes strange behaviour

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.6
          • Fix Version/s: 1.8.1
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            any

            Description

            use case:

            there's a simple tree containing an icon, label and inputText component. the tree has a root node with 3 children.
            the inputText component's partialSubmit attribute is set to true.
            when validation is used on the inputText component and a wrong value is entered into the textfield, a validation
            error message is shown and all inputText components are populated with this value.

            I just copied the code of the component showcase and modified it with a custom validator.
            1. Text File
              ICE-2018.patch
              0.7 kB
              Stefan Niederhauser

              Issue Links

              is duplicated by

              Bug - A problem which impairs or prevents the functions of the product. ICE-1409 Wrong Tree Node reference

              • Major - Major loss of function.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Hide
                Permalink
                Stefan Niederhauser added a comment -

                The problem is that when a validator fires, the update model phase is not executed.
                This causes the tree node's value not to be reset and therefore all nodes have the value of the last node.

                Solution: restore the state of the tree nodes when rendering the tree..

                Show
                Stefan Niederhauser added a comment - The problem is that when a validator fires, the update model phase is not executed. This causes the tree node's value not to be reset and therefore all nodes have the value of the last node. Solution: restore the state of the tree nodes when rendering the tree..
                Hide
                Permalink
                Stefan Niederhauser added a comment -

                This is the patch for the solution I described. (for icefaces version 1.8.0)

                Show
                Stefan Niederhauser added a comment - This is the patch for the solution I described. (for icefaces version 1.8.0)
                Hide
                Permalink
                Adnan Durrani added a comment -

                Fixed.

                Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\tree\Tree.java
                Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\tree\TreeRenderer.java
                Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\tree\Tree.java
                Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\tree\TreeRenderer.java
                Completed: At revision: 18876

                Show
                Adnan Durrani added a comment - Fixed. Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\tree\Tree.java Modified: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\tree\TreeRenderer.java Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\tree\Tree.java Sending content: D:\work\development\head\svn\ossrepo\icefaces\trunk\icefaces\component\src\com\icesoft\faces\component\tree\TreeRenderer.java Completed: At revision: 18876

                  People

                  • Assignee:
                    Adnan Durrani
                    Reporter:
                    Michael Thiem
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    2 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: