Pluto
  1. Pluto
  2. PLUTO-451

JSP21ExpressionEvaluatorProxy is causing an exception in Tomcat 6.0 if compiled using JDK 5.0

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.1.5, 2.0.0
    • Fix Version/s: 1.1.5, 2.0.0
    • Labels:
      None
    • Environment:
      JDK 5.0, Tomcat 6.0

      Description

      JDK 5.0 changed the signature of Method.invoke() to using varargs for the second argument. The ExpressionEvaluatorProxy for JSP 2.1 passes null, which presumably gets coerced into new Object[]

      { null }

      by the compiler. This causes an exception because the runtime arguments for JspApplicationContexts.getExpressionFactory() do not match its signature. Passing an empty array Object[] {} to Method.invoke() resolves this ambiguity.

      1. taglib-problem.diff
        0.7 kB
        Benjamin Gould

        Activity

        Benjamin Gould created issue -
        Benjamin Gould made changes -
        Field Original Value New Value
        Attachment taglib-problem.diff [ 12371531 ]
        Hide
        Craig Doremus added a comment -

        Applied patch to 1.1-286-trunk-merge branch in SVN rev 604442. Thank you Ben!

        Show
        Craig Doremus added a comment - Applied patch to 1.1-286-trunk-merge branch in SVN rev 604442. Thank you Ben!
        Hide
        Craig Doremus added a comment -

        Applied patch to trunk in SVN rev 604444.

        Show
        Craig Doremus added a comment - Applied patch to trunk in SVN rev 604444.
        Hide
        Craig Doremus added a comment -

        JSP21ExpressionEvaluatorProxy is not part of 1.1-286-COMPATIBILITY branch.

        Show
        Craig Doremus added a comment - JSP21ExpressionEvaluatorProxy is not part of 1.1-286-COMPATIBILITY branch.
        Craig Doremus made changes -
        Affects Version/s 1.1-286-COMPATIBILITY [ 12312052 ]
        Fix Version/s 1.1-286-COMPATIBILITY [ 12312052 ]
        Hide
        Craig Doremus added a comment -

        Patch applied to both branches. Thank you Ben!

        Show
        Craig Doremus added a comment - Patch applied to both branches. Thank you Ben!
        Craig Doremus made changes -
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Closed [ 6 ]
        Mark Thomas made changes -
        Workflow jira [ 12419168 ] Default workflow, editable Closed status [ 12564834 ]
        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12564834 ] jira [ 12586388 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Closed Closed
        2d 22h 23m 1 Craig Doremus 15/Dec/07 16:01

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development