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. ICE-2018.patch
        0.7 kB
        Stefan Niederhauser

        Issue Links

          Activity

          Michael Thiem created issue -
          Michael Thiem made changes -
          Field Original Value New Value
          Attachment support-4249.war [ 10612 ]
          Michael Thiem made changes -
          Attachment support-4249-src.zip [ 10613 ]
          Stefan Zeller made changes -
          Salesforce Case [50070000008LSXB]
          Hide
          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
          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)
          Stefan Niederhauser made changes -
          Attachment ICE-2018.patch [ 11697 ]
          Joel Krebs made changes -
          Link This issue is duplicated by ICE-4324 [ ICE-4324 ]
          Ken Fyten made changes -
          Assignee Ken Fyten [ ken.fyten ]
          Ken Fyten made changes -
          Fix Version/s 1.8.1 [ 10170 ]
          Assignee Priority P3
          Assignee Ken Fyten [ ken.fyten ] Yip Ng [ yip.ng ]
          Ken Fyten made changes -
          Assignee Yip Ng [ yip.ng ] Adnan Durrani [ adnan.durrani ]
          Ken Fyten made changes -
          Link This issue is duplicated by ICE-4324 [ ICE-4324 ]
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #18876 Wed May 13 15:33:21 MDT 2009 adnan.durrani Fix for ICE-2018 (tree: partialSubmit in nested inputText components causes strange behaviour)
          Files Changed
          Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/tree/Tree.java
          Commit graph MODIFY /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/tree/TreeRenderer.java
          Hide
          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
          Adnan Durrani made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Assignee Priority P3
          Mark Collette made changes -
          Link This issue is duplicated by ICE-1409 [ ICE-1409 ]

            People

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

              Dates

              • Created:
                Updated:
                Resolved: