ICEfaces
  1. ICEfaces
  2. ICE-3695

TreeNode do saveState restoreState

          Details

          • Type: New Feature New Feature
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.8DR#1
          • Fix Version/s: 1.8DR#2, 1.8
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            All

            Description

            The TreeNode component maintains references to its parent Tree component, and the current DefaultMutableTreeNode. It then tries to save and restore these fields. The Tree reference cannot be saved and restored, since upon restoration the TreeNode's parent would be a new Tree component, not the old one. So, it should probably just look up to its parent, and not even keep a reference. The DefaultMutableTreeNode reference is a hook into the entire tree data model, so we don't want to save this at all, especially since it comes from the bean anyway. The code should be refactored so that the DefaultMutableTreeNode reference is just transient, and not saved away.

              Issue Links

              blocks

              New Feature - A new feature of the product, which has yet to be developed. ICE-3696 Test component field state saving

              • Major - Major loss of function.
              • Closed

              Improvement - An improvement or enhancement to an existing feature or task. ICE-3442 Component saveState restoreState

              • Major - Major loss of function.
              • Closed

                Activity

                Hide
                Permalink
                yip.ng added a comment -

                getTree() changed to search for parent tree every time.

                defaultMutableTreeNode seems to be set every time by the renderer, which in turn gets it from a value binding, therefore no change in getter.

                Both fields marked transient.

                No fields to save anymore, therefore removed saveState() and restoreState().

                Show
                yip.ng added a comment - getTree() changed to search for parent tree every time. defaultMutableTreeNode seems to be set every time by the renderer, which in turn gets it from a value binding, therefore no change in getter. Both fields marked transient. No fields to save anymore, therefore removed saveState() and restoreState().

                  People

                  • Assignee:
                    Unassigned
                    Reporter:
                    Mark Collette
                  • Votes:
                    1 Vote for this issue
                    Watchers:
                    2 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: