ICEfaces
  1. ICEfaces
  2. ICE-6845

Modify ICEpushResourceHandler to use extension mechanism to load the MainServlet

    Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.1, EE-2.0.0.GA
    • Fix Version/s: 2.1-Beta, 3.0
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      JSF

      Description

      Modify ICEpushResourceHandler to use the extension mechanism (ExtensionRegistry) to load the MainServlet either from ICEpush or MOBI depending on what is available in the classpath.

        Activity

        Hide
        Mircea Toma added a comment -

        Used ExtensionRegistry to load the correct MainServlet. Delayed MAinServlet instantiation to allow for the extensions to register during context startup. Lookup PushContext on each PortableRenderer.push call to avoid NPEs during startup. Updated ICepush library to revision 24319.

        Show
        Mircea Toma added a comment - Used ExtensionRegistry to load the correct MainServlet. Delayed MAinServlet instantiation to allow for the extensions to register during context startup. Lookup PushContext on each PortableRenderer.push call to avoid NPEs during startup. Updated ICepush library to revision 24319.
        Hide
        Mircea Toma added a comment -

        NPE is triggered during application startup because ICEpushResourceHandler tries to access the PushContext during 'restore view' phase but PushContext is initialized only before 'render view' phase.

        The fix changes MainServlet initialization to be performed before 'restore view' phase instead of before 'render view' phase.

        Show
        Mircea Toma added a comment - NPE is triggered during application startup because ICEpushResourceHandler tries to access the PushContext during 'restore view' phase but PushContext is initialized only before 'render view' phase. The fix changes MainServlet initialization to be performed before 'restore view' phase instead of before 'render view' phase.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: