[patch] Patch proposal

DefaultInstanceManager.java.patch (text/plain), 1.09 KB, created by Violeta Georgieva on 2012-05-02 20:27:25 UTC

(hide)

Description: Patch proposal

Filename:

MIME Type:

Creator: Violeta Georgieva

Created: 2012-05-02 20:27:25 UTC

Size: 1.09 KB

patch

obsolete

>Index: java/org/apache/catalina/core/DefaultInstanceManager.java >=================================================================== >--- java/org/apache/catalina/core/DefaultInstanceManager.java (revision 1333175) >+++ java/org/apache/catalina/core/DefaultInstanceManager.java (working copy) >@@ -342,7 +342,9 @@ > String methodName = method.getName(); > if (context != null) { > // Resource injection only if JNDI is enabled >- if (injections != null && methodName.startsWith("set") && methodName.length() > 3) { >+ if (injections != null && methodName.startsWith("set") && methodName.length() > 3 >+ && method.getParameterTypes().length == 1 >+ && method.getReturnType().getName().equals("void")) { > String fieldName = Character.toLowerCase(methodName.charAt(3)) + methodName.substring(4); > if (injections.containsKey(fieldName)) { > annotations.add(new AnnotationCacheEntry( View Attachment As Diff View Attachment As Raw

This is ASF Bugzilla: the Apache Software Foundation bug system. In case of problems with the functioning of ASF Bugzilla, please contact bugzilla-admin@apache.org. Please Note: this e-mail address is only for reporting problems with ASF Bugzilla. Mail about any other subject will be silently ignored.

• • Home
  • | New
  • | Browse
  • | Search
  • |
    [?]
  • | Reports
  • | Help
  • | New Account
  • | Log In
    Remember [x]
  • | Forgot Password
    Login: [x]