el-spec
  1. el-spec
  2. EL_SPEC-9

ELProcessor.defineFunction methods do not check for null args...

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Labels:
      None
    • Environment:

      N/A

      Description

      API documentation states for both of the defineFunction methods that a NullPointerException should be thrown if any arg is null. Niether on of the methods looks to be testing for null args.

      Example below.

      public void defineFunction(String prefix, String function, Method method) {

      if (prefix == null || function == null || method == null)

      { <-- need to add something like this. throw new NullPointerException(); }

      if (function.equals(""))

      { function = method.getName(); }

      elManager.mapFunction(prefix, function, method);
      }

        Activity

        Hide
        kchung added a comment -

        Fixed.

        Note the spec has also been modified to also throw a NoSuchMethodException if the method is not static. See the javadocs for detailed.

        Show
        kchung added a comment - Fixed. Note the spec has also been modified to also throw a NoSuchMethodException if the method is not static. See the javadocs for detailed.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: