ICEfaces
  1. ICEfaces
  2. ICE-5970

dynamic ui:includes not working with ICEfaces 2

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Cannot Reproduce
          • Affects Version/s: 2.0-Beta1
          • Fix Version/s: 2.0-Beta2, 2.0.0
          • Component/s: Framework, ICE-Components
          • Labels:
            None
          • Environment:
            ICEfaces 2 + Mojarra 2

            Description

            Users are migrating ICEfaces 1.8.x applications to ICEfaces 2, and finding that dynamic ui:includes are not working. We need to investigate why this is, and come up with either code changes, or a new technique that developers should use. This should be documented in the wiki, in its own page, and link to by: http://wiki.icefaces.org/display/ICE/Compatibility

              Activity

              Ascending order - Click to sort in descending order
              Mark Collette created issue -
              Mark Collette made changes -
              Field Original Value New Value
              Salesforce Case []
              Affects [Documentation (User Guide, Ref. Guide, etc.)]
              Assignee Deryk Sinotte [ deryk.sinotte ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #22113 Thu Aug 05 13:26:31 MDT 2010 deryk.sinotte ICE-5970: expanded test case for checking dynamic includes in stock JSF 2 and ICEfaces (both plain and compat modes)
              Files Changed
              Commit graph DEL /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/webapp/ice/input.xhtml
              Commit graph DEL /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/webapp/stock/output.xhtml
              Commit graph ADD /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/webapp/ice/start.xhtml
              Commit graph DEL /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/webapp/stock/sample.xhtml
              Commit graph DEL /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/java/com/x/jsf
              Commit graph ADD /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/webapp/stock/page03.xhtml
              Commit graph DEL /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/webapp/stock/input.xhtml
              Commit graph ADD /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/webapp/ice/page03.xhtml
              Commit graph ADD /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/webapp/stock/page02.xhtml
              Commit graph MODIFY /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/webapp/index.jsp
              Commit graph ADD /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/webapp/ice/page02.xhtml
              Commit graph ADD /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/webapp/stock/start.xhtml
              Commit graph DEL /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/webapp/ice/output.xhtml
              Commit graph ADD /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/webapp/stock/page01.xhtml
              Commit graph DEL /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/webapp/ice/sample.xhtml
              Commit graph ADD /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/webapp/ice/page01.xhtml
              Hide
              Permalink
              Deryk Sinotte added a comment -

              Based on a conversation that Mark and I had, I've heavily modified the test case (icefaces2/samples/test/dyninc) to provide 3-way page navigation as well as output a bit more information. The basic finding is the same as noted in the discussion in that both stock JSF and ICEfaces with compat components appear to behave in the same way. Adding the 3rd page did show the behaviour that Mark hypothesized with the navigation defaulting back to the default page when the navigation bean is request scoped. To summarize:

              Request-scoped bean: The initial include correctly renders the default - page 1. Switching to dynamically include page 2 or 3 works but subsequent navigation always returns to page 1.

              View-scoped bean: The initial include correctly renders the default - page 1. Switching to dynamically include page 2 or 3 works but subsequent navigation does not work - whatever page your are on you stay on.

              Session-scoped bean: Dynamic includes appear to work all over.

              Window-scoped bean: Dynamic includes appear to work if the page is enabled for ICEfaces. In stock JSF it behaves as a Request-scoped bean.

              Show
              Deryk Sinotte added a comment - Based on a conversation that Mark and I had, I've heavily modified the test case (icefaces2/samples/test/dyninc) to provide 3-way page navigation as well as output a bit more information. The basic finding is the same as noted in the discussion in that both stock JSF and ICEfaces with compat components appear to behave in the same way. Adding the 3rd page did show the behaviour that Mark hypothesized with the navigation defaulting back to the default page when the navigation bean is request scoped. To summarize: Request-scoped bean: The initial include correctly renders the default - page 1. Switching to dynamically include page 2 or 3 works but subsequent navigation always returns to page 1. View-scoped bean: The initial include correctly renders the default - page 1. Switching to dynamically include page 2 or 3 works but subsequent navigation does not work - whatever page your are on you stay on. Session-scoped bean: Dynamic includes appear to work all over. Window-scoped bean: Dynamic includes appear to work if the page is enabled for ICEfaces. In stock JSF it behaves as a Request-scoped bean.
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #22116 Fri Aug 06 10:47:25 MDT 2010 deryk.sinotte ICE-5970: remove old package
              Files Changed
              Commit graph DEL /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/java/com
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #22117 Fri Aug 06 11:05:09 MDT 2010 deryk.sinotte ICE-5970: adding NavBean
              Files Changed
              Commit graph ADD /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/java/org
              Commit graph ADD /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/java/org/icefaces/test
              Commit graph ADD /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/java/org/icefaces/test/dyninc/NavBean.java
              Commit graph ADD /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/java/org/icefaces
              Commit graph ADD /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/java/org/icefaces/test/dyninc
              Hide
              Permalink
              Deryk Sinotte added a comment -

              Adding the forum reference.

              At this point, the test case is working for us. I'll build a .war and try it on Glassfish 3 with Mojarra 2.0.3 (as I ran on Tomcat previously). If it runs as expected, then I'll post the .war file to the forum for the poster to try in their environment.

              Show
              Deryk Sinotte added a comment - Adding the forum reference. At this point, the test case is working for us. I'll build a .war and try it on Glassfish 3 with Mojarra 2.0.3 (as I ran on Tomcat previously). If it runs as expected, then I'll post the .war file to the forum for the poster to try in their environment.
              Deryk Sinotte made changes -
              ICEsoft Forum Reference http://www.icefaces.org/JForum/posts/list/30/16949.page#64021
              Salesforce Case []
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #22126 Fri Aug 06 13:44:54 MDT 2010 deryk.sinotte ICE-5970: add partialSubmit
              Files Changed
              Commit graph MODIFY /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/webapp/stock/start.xhtml
              Commit graph MODIFY /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/webapp/stock/page01.xhtml
              Commit graph MODIFY /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/webapp/ice/start.xhtml
              Commit graph MODIFY /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/webapp/stock/page03.xhtml
              Commit graph MODIFY /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/webapp/ice/page03.xhtml
              Commit graph MODIFY /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/webapp/ice/page01.xhtml
              Commit graph MODIFY /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/webapp/stock/page02.xhtml
              Commit graph MODIFY /icefaces2/trunk/icefaces/samples/test/dyninc/src/main/webapp/ice/page02.xhtml
              Hide
              Permalink
              Deryk Sinotte added a comment -

              Our test case appears to work and I can't duplicate the issue described in the forum so marking this resolved (Can't Reproduce).

              Show
              Deryk Sinotte added a comment - Our test case appears to work and I can't duplicate the issue described in the forum so marking this resolved (Can't Reproduce).
              Deryk Sinotte made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Fix Version/s 2.0-Beta2 [ 10242 ]
              Resolution Cannot Reproduce [ 5 ]
              Ken Fyten made changes -
              Fix Version/s 2.0.0 [ 10230 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Deryk Sinotte
                  Reporter:
                  Mark Collette
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: