ICEfaces
  1. ICEfaces
  2. ICE-10160

JavaScript error when loading all demos in Showcase Mobi demo

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.0.BETA
    • Fix Version/s: 4.0
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      ICEfaces Mobi components
    • Assignee Priority:
      P1

      Description

      Load the showcase, click on the Mobi Components tab, and then select the Cloud Push demo. You should see an error logged in the browser console similar to:

      Uncaught Error: jsf.ajax.request: source not set jsf.js.jsf?ln=javax.faces&v=4_0_0_140805:1
      request jsf.js.jsf?ln=javax.faces&v=4_0_0_140805:1
      im.viewManager.showView VM1071:6784
      onclick showcase-mobile.jsf:25

        Activity

        Hide
        Deryk Sinotte added a comment -

        The additional information was logged by QA:

        (Chrome) - Browser console error upon launch of application:
        Uncaught Error: jsf.ajax.request: source not set ......
        request jsf.js.jsf?ln=javax.faces&v=4_0_0_140804:1
        im.viewManager.showView VM799:6784
        onclick showcase-mobile.jsf:25

        (FF) Pressing the Enable Cloud Push button causes a navigation error in FF:
        "Firefox doesn't know how to open this address, because one of the following protocols (icemobile) isn't associated with any program or is not allowed in this context."

        (IE11) - Browser console error upon launch of application:
        SCRIPT5022: jsf.ajax.request: source not set
        File: jsf.js.jsf, Line: 1, Column: 32071.
        Pressing the Enable Cloud Push button causes a navigation error

        I was able to see the problem with other demo besides Cloud Push as well. According to the work done in ICE-9838, Art was also seeing a JavaScript error when he first added the Cloud Push component demo. According to his notes in that case:

        "It only happens once or twice, and it doesn't seem to affect functionality. It only happens when resource coalescing is on. If you remove the very last h:outputText on the page, the error doesn't occur."

        Show
        Deryk Sinotte added a comment - The additional information was logged by QA: (Chrome) - Browser console error upon launch of application: Uncaught Error: jsf.ajax.request: source not set ...... request jsf.js.jsf?ln=javax.faces&v=4_0_0_140804:1 im.viewManager.showView VM799:6784 onclick showcase-mobile.jsf:25 (FF) Pressing the Enable Cloud Push button causes a navigation error in FF: "Firefox doesn't know how to open this address, because one of the following protocols (icemobile) isn't associated with any program or is not allowed in this context." (IE11) - Browser console error upon launch of application: SCRIPT5022: jsf.ajax.request: source not set File: jsf.js.jsf, Line: 1, Column: 32071. Pressing the Enable Cloud Push button causes a navigation error I was able to see the problem with other demo besides Cloud Push as well. According to the work done in ICE-9838 , Art was also seeing a JavaScript error when he first added the Cloud Push component demo. According to his notes in that case: "It only happens once or twice, and it doesn't seem to affect functionality. It only happens when resource coalescing is on. If you remove the very last h:outputText on the page, the error doesn't occur."
        Hide
        Liana Munroe added a comment -

        This js error is seen in all showcase-mobile demos since around the time of revision 42057. Additional issue that is occurring since that revision is the top left corner navigation is no longer rendered on the demos when viewed on a mobile device.

        Show
        Liana Munroe added a comment - This js error is seen in all showcase-mobile demos since around the time of revision 42057. Additional issue that is occurring since that revision is the top left corner navigation is no longer rendered on the demos when viewed on a mobile device.
        Hide
        Mircea Toma added a comment - - edited

        Modified ice.mobi.viewManager.showView function to pass in the submitting form (as the javax.faces.source parameter) when calling jsf.ajax.request function. This avoids having the warning Mojarra bridge issues.

        Show
        Mircea Toma added a comment - - edited Modified ice.mobi.viewManager.showView function to pass in the submitting form (as the javax.faces.source parameter) when calling jsf.ajax.request function. This avoids having the warning Mojarra bridge issues.
        Hide
        Liana Munroe added a comment -

        Verified Icefaces 4 trunk r42085. Tomcat 7, all browsers. All issues resolved except for the navigation errors in FF and IE when using the Enable Cloud Push button.

        Show
        Liana Munroe added a comment - Verified Icefaces 4 trunk r42085. Tomcat 7, all browsers. All issues resolved except for the navigation errors in FF and IE when using the Enable Cloud Push button.
        Hide
        Deryk Sinotte added a comment -

        Please take a look at the navigation errors noted by QA.

        Show
        Deryk Sinotte added a comment - Please take a look at the navigation errors noted by QA.
        Hide
        Mircea Toma added a comment -

        When Enable Cloud Push button is clicked the browser navigates to a URL that uses icemobile: protocol. This protocol is normally implemented by our ICEmobile containers (for different devices). The desktop browser does not know anything about this protocol, hence the error.

        This problem should maybe solved in the demo app by hiding the button when running in a desktop browser.

        Show
        Mircea Toma added a comment - When Enable Cloud Push button is clicked the browser navigates to a URL that uses icemobile: protocol. This protocol is normally implemented by our ICEmobile containers (for different devices). The desktop browser does not know anything about this protocol, hence the error. This problem should maybe solved in the demo app by hiding the button when running in a desktop browser.
        Hide
        Ken Fyten added a comment -

        The cloudPush issue has been spun off into ICE-10211.

        Marking this issue Resolved.

        Show
        Ken Fyten added a comment - The cloudPush issue has been spun off into ICE-10211 . Marking this issue Resolved.

          People

          • Assignee:
            Ted Goddard
            Reporter:
            Deryk Sinotte
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: