antville/helma
antville/helma
1
1
Fork 0
Code Issues 5 Pull requests 7 Projects Releases 5 Activity Actions

Tentative fix for Java reflection bug where we're unable to call public methods

Browse source 
declared in non-public classes.
This commit is contained in:
hns 2002-11-27 17:22:53 +00:00
parent 63f7a2e4df
commit d9c4b3d6f3
1 changed files with 9 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
src/FESI/Interpreter/ClassInfo.java
View file

@ -399,6 +399,8 @@ public class ClassInfo {
} }
} // if class not public } // if class not public
if (Modifier.isPublic (method.getModifiers ()))
method.setAccessible (true);
// save it // save it
methodVector.addElement(method); methodVector.addElement(method);
} }
@ -486,7 +488,7 @@ public class ClassInfo {
// I am not too sure of what happens if the method is defined in an // I am not too sure of what happens if the method is defined in an
// interface... // interface...
if (!Modifier.isStatic(cls.getModifiers()) && !Modifier.isPublic(cls.getModifiers())) { if (!Modifier.isStatic(cls.getModifiers()) && !Modifier.isPublic(cls.getModifiers())) {
if (debug) System.out.println("** Bean class " + cls + if (debug) System.out.println("** Bean class " + cls +
" is not public, examining superclasses to find proper method descriptor"); " is not public, examining superclasses to find proper method descriptor");
SEARCHPUBLIC: SEARCHPUBLIC:
for (Class theClass=cls;theClass!=null;theClass=theClass.getSuperclass()) { for (Class theClass=cls;theClass!=null;theClass=theClass.getSuperclass()) {
@ -516,6 +518,8 @@ public class ClassInfo {
} }
} // for } // for
} // if class not public } // if class not public
if (Modifier.isPublic (method.getModifiers ()))
method.setAccessible (true);
// save it // save it
methodVector.addElement(method); methodVector.addElement(method);
} }
@ -523,7 +527,7 @@ public class ClassInfo {
Method [] methods = null; Method [] methods = null;
int nmbMethods = methodVector.size(); int nmbMethods = methodVector.size();
if (nmbMethods>0) { if (nmbMethods>0) {
if (debug) System.out.println("** " + nmbMethods + " methods named: '" if (debug) System.out.println("** " + nmbMethods + " methods named: '"
+ functionName + "' + found, add to bean cache"); + functionName + "' + found, add to bean cache");
methods = new Method[nmbMethods]; methods = new Method[nmbMethods];
methodVector.copyInto(methods); methodVector.copyInto(methods);
@ -532,10 +536,10 @@ public class ClassInfo {
} }
beanMethods.put(functionName, methods); beanMethods.put(functionName, methods);
} else { } else {
if (debug) System.out.println("** No bean method named: '" + if (debug) System.out.println("** No bean method named: '" +
functionName + "' + found"); functionName + "' + found");
} }
return methods; return methods;
} }
} }