ICEfaces
  1. ICEfaces
  2. ICE-877

Support Facelets dynamic ui:include

          Details

          • Type: New Feature New Feature
          • Status: Closed
          • Priority: Critical Critical
          • Resolution: Fixed
          • Affects Version/s: 1.5
          • Fix Version/s: 1.6DR#2, 1.6
          • Component/s: Framework
          • Labels:
            None
          • Environment:
            Operating System: All
            Platform: All
          • Affects:
            Documentation (User Guide, Ref. Guide, etc.), Compatibility/Configuration

            Description

            Right now we only support a static UIComponent tree, but stock Facelets should
            support a dynamic UIComponent tree via their handler classes, whose apply(-)
            methods could create varying numbers of UIComponent children. This has
            ramifications with the StateManager code, and the transient and non-transient
            components.

              Activity

              Descending order - Click to sort in ascending order
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]
              Assignee Priority P1
              Assignee Mark Collette [ mark.collette ]
              Ken Fyten made changes -
              Fix Version/s 1.6 [ 10031 ]
              Ken Fyten made changes -
              Summary ICEfaces doesn't support Facelets dynamic ui:include Support Facelets dynamic ui:include
              Affects [Documentation (User Guide, Ref. Guide, etc.), Compatibility/Configuration]
              Ken Fyten made changes -
              Issue Type Bug [ 1 ] New Feature [ 2 ]
              Mark Collette made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Fix Version/s 1.6DR#2 [ 10040 ]
              Fix Version/s 1.6 [ 10031 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Mark Collette added a comment -

              Subversion 13493 : Fixed Facelets' FacetHandler so it preserves facet components, and their ids.

              3rd\facelets-1.1\src\java\com\sun\facelets\tag\jsf\core\FacetHandler.java

              Subversion 13391 : Made dynamic ui:include work. Augmented duplicate id reporting. Improved UIComponent tree debugging.

              icefaces\core\src\com\icesoft\faces\application\D2DViewHandler.java
              icefaces\core\src\com\icesoft\faces\facelets\D2DFaceletViewHandler.java
              icefaces\lib\icefaces-facelets.jar

              Subversion 13392 : Changed ICEfaces + Facelets version of component-showcase to use dynamic ui:include

              icefaces\samples\component-showcase\src\com\icesoft\icefaces\samples\showcase\navigation\PageContentBean.java
              icefaces\samples\component-showcase\src\com\icesoft\icefaces\samples\showcase\navigation\TreeNavigation.java
              icefaces\samples\component-showcase\web\inc\content-facelet.jspx

              Show
              Mark Collette added a comment - Subversion 13493 : Fixed Facelets' FacetHandler so it preserves facet components, and their ids. 3rd\facelets-1.1\src\java\com\sun\facelets\tag\jsf\core\FacetHandler.java Subversion 13391 : Made dynamic ui:include work. Augmented duplicate id reporting. Improved UIComponent tree debugging. icefaces\core\src\com\icesoft\faces\application\D2DViewHandler.java icefaces\core\src\com\icesoft\faces\facelets\D2DFaceletViewHandler.java icefaces\lib\icefaces-facelets.jar Subversion 13392 : Changed ICEfaces + Facelets version of component-showcase to use dynamic ui:include icefaces\samples\component-showcase\src\com\icesoft\icefaces\samples\showcase\navigation\PageContentBean.java icefaces\samples\component-showcase\src\com\icesoft\icefaces\samples\showcase\navigation\TreeNavigation.java icefaces\samples\component-showcase\web\inc\content-facelet.jspx
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #13392 Wed Mar 14 13:48:10 MDT 2007 mark.collette ICE-877 : ICEfaces doesn't support Facelets dynamic ui:include.
              Changed ICEfaces + Facelets version of component-showcase to use dynamic ui:include
              Files Changed
              Commit graph MODIFY /icefaces/trunk/icefaces/samples/component-showcase/src/com/icesoft/icefaces/samples/showcase/navigation/TreeNavigation.java
              Commit graph MODIFY /icefaces/trunk/icefaces/samples/component-showcase/src/com/icesoft/icefaces/samples/showcase/navigation/PageContentBean.java
              Commit graph MODIFY /icefaces/trunk/icefaces/samples/component-showcase/web/inc/content-facelet.jspx
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #13391 Wed Mar 14 13:43:51 MDT 2007 mark.collette ICE-877 : ICEfaces doesn't support Facelets dynamic ui:include.
              Made dynamic ui:include work. Augmented duplicate id reporting. Improved UIComponent tree debugging.
              Files Changed
              Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/application/D2DViewHandler.java
              Commit graph MODIFY /icefaces/trunk/icefaces/lib/icefaces-facelets.jar
              Commit graph MODIFY /icefaces/trunk/icefaces/core/src/com/icesoft/faces/facelets/D2DFaceletViewHandler.java
              Ken Fyten made changes -
              Fix Version/s 1.6 [ 10031 ]
              Fix Version/s 1.6DR#2 [ 10040 ]
              Ken Fyten made changes -
              Estimated Complexity Medium
              Fix Version/s 1.6DR#2 [ 10040 ]
              Fix Version/s 1.6 [ 10031 ]
              Assignee Priority P1
              Ken Fyten made changes -
              Priority Major [ 3 ] Critical [ 2 ]
              Ken Fyten made changes -
              Affects Version/s 1.5 [ 10027 ]
              Affects Version/s 1.0.1 [ 10025 ]
              Icefaces Administrator made changes -
              Field Original Value New Value
              issue.field.bugzillaimportkey 895 12146
              Hide
              Permalink
              Ken Fyten added a comment -

              Raised priority. This is a pre-requisite for improved Seam support.

              Show
              Ken Fyten added a comment - Raised priority. This is a pre-requisite for improved Seam support.
              Hide
              Permalink
              Mark Collette added a comment -

              Should be doable in January.

              Show
              Mark Collette added a comment - Should be doable in January.
              Hide
              Permalink
              Ken Fyten added a comment -

              Target v1.6.

              Mark, please comment on feasibility, etc.

              Show
              Ken Fyten added a comment - Target v1.6. Mark, please comment on feasibility, etc.
              Hide
              Permalink
              Ted Goddard added a comment -

              Worthwhile investigating since it's an important customer feature and potentially low risk because it's
              confined to Facelets.

              Show
              Ted Goddard added a comment - Worthwhile investigating since it's an important customer feature and potentially low risk because it's confined to Facelets.
              Mark Collette created issue -

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: