ICEpush
  1. ICEpush
  2. PUSH-200

icepush-ee includes mobi.js on non-ICEfaces pages leading to JavaScript errors

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-3.0.0.GA_P01
    • Fix Version/s: EE-3.2.0.BETA, EE-3.2.0.GA
    • Component/s: Push Library
    • Labels:
      None
    • Environment:
      ICEpush
    • Assignee Priority:
      P1

      Description


      icepush-ee.jar contains a SystemEventListener MobiSetup that adds mobi.js to all JSF pages. However, if the ice namespace is not defined, a JavaScript exception occurs.

      icepush-ee.jar should be tested with basic on both the ICEfaces and the non-ICEfaces page.

        Activity

        Migration created issue -
        Hide
        Ted Goddard added a comment - - edited

        MobiSetup should attempt to determine if ICEpush is loaded on the page since it cannot function without ICEpush. If this is not possible (perhaps due to timing of the SystemEventListener) mobi.js should at least always load with no errors.

        Show
        Ted Goddard added a comment - - edited MobiSetup should attempt to determine if ICEpush is loaded on the page since it cannot function without ICEpush. If this is not possible (perhaps due to timing of the SystemEventListener) mobi.js should at least always load with no errors.
        Migration made changes -
        Field Original Value New Value
        Reporter Migration [ remote ] Ted Goddard [ ted.goddard ]
        Migration made changes -
        Assignee Mircea Toma [ mircea.toma ]
        Ken Fyten made changes -
        Fix Version/s EE-3.2.0.GA [ 10323 ]
        Ken Fyten made changes -
        Assignee Priority P1 [ 10010 ]
        Hide
        Mircea Toma added a comment -

        Test for the existence of ICEfaces namespace before testing for the existence of the ICEpush namespace since both are required by icepush/core-ee code.

        Show
        Mircea Toma added a comment - Test for the existence of ICEfaces namespace before testing for the existence of the ICEpush namespace since both are required by icepush/core-ee code.
        Mircea Toma made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Fix Version/s EE-3.2.0.BETA [ 10574 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Ted Goddard
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: