ICEfaces
  1. ICEfaces
  2. ICE-7067

PushRenderer.addCurrentView(String) should be allowed to be invoked earlier in the lifecycle

          Details

          • Type: Improvement Improvement
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: EE-2.0.0.GA, 2.0.2
          • Fix Version/s: 2.1-Beta, 3.0, EE-2.0.0.GA_P01
          • Component/s: Framework
          • Labels:
            None
          • Environment:
            ICEfaces

            Description

            Currently when invoking PushRenderer.addCurrentView(String) early in the lifecycle, it results in the invocation of PushContext.addGroupMember with a pushID value of null. For instance with a ICEfaces component, places like in a setter's and the isInView methods of a UIComponentBase component, the pushID can not yet be found. In the encodeAll(...) method it can be found though. It would be nice if this could work earlier within the lifecycle somehow. Maybe even by caching earlier invocations if the addCurrentView(String) method.

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Jack Van Ooststroom added a comment -

              Assigning to Mircea for investigation.

              Show
              Jack Van Ooststroom added a comment - Assigning to Mircea for investigation.
              Hide
              Permalink
              Mircea Toma added a comment -

              Moved the assigning of viewId from the component tree building time to before restore view phase. This way the viewId is available when the PushRenderer.addCurrentView() method is called in a the constructor or the setter of a bean.

              Show
              Mircea Toma added a comment - Moved the assigning of viewId from the component tree building time to before restore view phase. This way the viewId is available when the PushRenderer.addCurrentView() method is called in a the constructor or the setter of a bean.

                People

                • Assignee:
                  Mircea Toma
                  Reporter:
                  Jack Van Ooststroom
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: