Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Works as designed
    • Affects Version/s: None
    • Fix Version/s: 2.2.5
    • Component/s: expression language
    • Labels:
      None
    • Status Whiteboard:
      Hide

      size_medium importance_medium

      Show
      size_medium importance_medium

      Description

      Hi;

      I have added ELContextListener to the Application object. When
      FacesContext#getELContext is called, EL context listeners are not called.

      I have looked at the source code of FacesContextImpl, and

      public ELContext getELContext() {
      assertNotReleased();
      if (elContext == null) {
      elContext = new ELContextImpl(getApplication().getELResolver());
      elContext.putContext(FacesContext.class, this);
      UIViewRoot root = this.getViewRoot();
      if (null != root)

      { elContext.setLocale(root.getLocale()); }

      }
      return elContext;
      }

      This code does not call any EL context listeners that are registered with
      Application.
      Description
      Hi; I have added ELContextListener to the Application object. When FacesContext#getELContext is called, EL context listeners are not called. I have looked at the source code of FacesContextImpl, and
      public ELContext getELContext() {
      assertNotReleased();
      if (elContext == null) {
      elContext = new ELContextImpl(getApplication().getELResolver());
      elContext.putContext(FacesContext.class, this);
      UIViewRoot root = this.getViewRoot();
      if (null != root)

      { elContext.setLocale(root.getLocale()); }


      }
      return elContext;
      }
      This code does not call any EL context listeners that are registered with Application.
      Show »

        Activity

        Hide
        Ed Burns added a comment -

        Moved from JAVASERVERFACES_SPEC_PUBLIC JIRA.

        Show
        Ed Burns added a comment - Moved from JAVASERVERFACES_SPEC_PUBLIC JIRA.
        Hide
        Manfred Riem added a comment - - edited

        Manual inspection of the 2.2 branch shows the following for the getELContext method,
        as it appears the ELContextListeners are fired. Closing this as "Works as Designed".

        /**

        • @see javax.faces.context.FacesContext#getELContext()
          */
          @Override
          public ELContext getELContext() {
          assertNotReleased();
          if (elContext == null) {
          Application app = getApplication();
          elContext = new ELContextImpl(app.getELResolver());
          elContext.putContext(FacesContext.class, this);
          UIViewRoot root = this.getViewRoot();
          if (null != root) { elContext.setLocale(root.getLocale()); }

          ELContextListener[] listeners = app.getELContextListeners();
          if (listeners.length > 0)

          Unknown macro: { ELContextEvent event = new ELContextEvent(elContext); for (ELContextListener listener}

          }
          return elContext;

        }

        Show
        Manfred Riem added a comment - - edited Manual inspection of the 2.2 branch shows the following for the getELContext method, as it appears the ELContextListeners are fired. Closing this as "Works as Designed". /** @see javax.faces.context.FacesContext#getELContext() */ @Override public ELContext getELContext() { assertNotReleased(); if (elContext == null) { Application app = getApplication(); elContext = new ELContextImpl(app.getELResolver()); elContext.putContext(FacesContext.class, this); UIViewRoot root = this.getViewRoot(); if (null != root) { elContext.setLocale(root.getLocale()); } ELContextListener[] listeners = app.getELContextListeners(); if (listeners.length > 0) Unknown macro: { ELContextEvent event = new ELContextEvent(elContext); for (ELContextListener listener} } return elContext; }

          People

          • Assignee:
            Manfred Riem
            Reporter:
            rogerk
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: