ICEfaces
  1. ICEfaces
  2. ICE-6681

Navigation in preRenderView event throws Error in extras bootstrap JavaScript exception

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 2.0.0, 2.0.0-EE-Beta1
          • Fix Version/s: 3.0
          • Component/s: Framework
          • Labels:
            None
          • Environment:
            IE and Firefox
          • Assignee Priority:
            P2
          • Workaround Exists:
            Yes
          • Workaround Description:
            Use a PhaseListener instead, this has been implemented in attached test case but had been commented out to show original issue.

            Description

            When using the following code:

            <f:metadata>
                        <f:event type="preRenderView" id="prePayDayApproveEvent" listener="#{testBean.someAction}"/>
            </f:metadata>

            A forward navigation takes place in the event listener. The navigation is causing a JavaScript error to be thrown. See attachments for details.

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Arran Mccullough added a comment -

              Attached test case war file built for Tomcat 6. Attached screen shots of the errors in Firefox.

              Show
              Arran Mccullough added a comment - Attached test case war file built for Tomcat 6. Attached screen shots of the errors in Firefox.
              Hide
              Permalink
              Mircea Toma added a comment -

              Looks like the h:head and h:body detection fails during the initial forward navigation which causes the BridgeSetup to give up on rendering the script tags that load the core bridge code, only the compat JS code being loaded.

              Show
              Mircea Toma added a comment - Looks like the h:head and h:body detection fails during the initial forward navigation which causes the BridgeSetup to give up on rendering the script tags that load the core bridge code, only the compat JS code being loaded.
              Hide
              Permalink
              Mircea Toma added a comment -

              The HeadBodyListener is not invoked during the processing of the second view (navigated to) because the PreAddComponentEvent is not fired at that time.

              Show
              Mircea Toma added a comment - The HeadBodyListener is not invoked during the processing of the second view (navigated to) because the PreAddComponentEvent is not fired at that time.
              Hide
              Permalink
              Mircea Toma added a comment - - edited

              The fix modifies the HeadBodyListener to be invoked on view root component pre-rendering just before BridgeSetup runs to allow the inspection of the component tree. Changed the test for the existence of h:head and h:body components in the component tree to iterate through the view root children. Changed BridgeSetup and ExtrasSetup to be invoked on view root component pre-render event instead pre-render view so that they're invoked after HeadBodyListener does its tests.

              Show
              Mircea Toma added a comment - - edited The fix modifies the HeadBodyListener to be invoked on view root component pre-rendering just before BridgeSetup runs to allow the inspection of the component tree. Changed the test for the existence of h:head and h:body components in the component tree to iterate through the view root children. Changed BridgeSetup and ExtrasSetup to be invoked on view root component pre-render event instead pre-render view so that they're invoked after HeadBodyListener does its tests.
              Hide
              Permalink
              Carmen Cristurean added a comment -

              Tested successfully on Icefaces2.1 trunk (revision#24992), using Tomcat6 server and FF3.6 and IE8 browsers.

              Show
              Carmen Cristurean added a comment - Tested successfully on Icefaces2.1 trunk (revision#24992), using Tomcat6 server and FF3.6 and IE8 browsers.

                People

                • Assignee:
                  Carmen Cristurean
                  Reporter:
                  Arran Mccullough
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: