javaserverfaces
  1. javaserverfaces
  2. JAVASERVERFACES-2550

ExternalContext.lo() should throw NPE when args are null

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.1.14, 2.2.0-m06
    • Component/s: context
    • Labels:
      None
    • Environment:

      N/A

      Description

      Mojarra API states the following for both of these methods:

      ExternalContext.log(java.lang.String msg)
      Throws:
      java.lang.NullPointerException - if message is null

      ExternalContext.log(java.lang.String msg, java.lang.Throwable throwable)
      Throws:
      java.lang.NullPointerException - if message or exception is null

      Looking at the ServletContext's perspective methods they mention nothing of throwing or handling nulls.

        Activity

        Hide
        rogerk added a comment -

        Changes.

        Show
        rogerk added a comment - Changes.
        Hide
        rogerk added a comment -

        Committed to trunk:
        Sending jsf-ri/src/main/java/com/sun/faces/context/ExternalContextImpl.java
        Sending test/agnostic/context/regular/src/main/java/com/sun/faces/test/agnostic/context/regular/ExternalContextBean.java
        Adding test/agnostic/context/regular/src/main/webapp/issue2550.xhtml
        Sending test/agnostic/context/regular/src/test/java/com/sun/faces/test/agnostic/context/regular/ExternalContextIT.java
        Transmitting file data ....
        Committed revision 10859.

        Show
        rogerk added a comment - Committed to trunk: Sending jsf-ri/src/main/java/com/sun/faces/context/ExternalContextImpl.java Sending test/agnostic/context/regular/src/main/java/com/sun/faces/test/agnostic/context/regular/ExternalContextBean.java Adding test/agnostic/context/regular/src/main/webapp/issue2550.xhtml Sending test/agnostic/context/regular/src/test/java/com/sun/faces/test/agnostic/context/regular/ExternalContextIT.java Transmitting file data .... Committed revision 10859.
        Hide
        rogerk added a comment -

        Committed to MOJARRA_2_1X_ROLLING branch:
        Sending jsf-ri/src/main/java/com/sun/faces/context/ExternalContextImpl.java
        Sending test/agnostic/context/regular/src/main/java/com/sun/faces/test/agnostic/context/regular/ExternalContextBean.java
        Adding test/agnostic/context/regular/src/main/webapp/issue2550.xhtml
        Sending test/agnostic/context/regular/src/test/java/com/sun/faces/test/agnostic/context/regular/ExternalContextIT.java
        Transmitting file data ....
        Committed revision 10860.

        Show
        rogerk added a comment - Committed to MOJARRA_2_1X_ROLLING branch: Sending jsf-ri/src/main/java/com/sun/faces/context/ExternalContextImpl.java Sending test/agnostic/context/regular/src/main/java/com/sun/faces/test/agnostic/context/regular/ExternalContextBean.java Adding test/agnostic/context/regular/src/main/webapp/issue2550.xhtml Sending test/agnostic/context/regular/src/test/java/com/sun/faces/test/agnostic/context/regular/ExternalContextIT.java Transmitting file data .... Committed revision 10860.
        Hide
        rogerk added a comment -

        Committed.

        Show
        rogerk added a comment - Committed.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: