From: <jbo...@li...> - 2006-02-12 03:33:35
|
Author: mar...@jb... Date: 2006-02-11 22:33:32 -0500 (Sat, 11 Feb 2006) New Revision: 2374 Modified: trunk/labs/jbossrules/drools-core/src/main/java/org/drools/util/asm/ClassFieldInspector.java Log: -Update visitor to check no argument method calls Modified: trunk/labs/jbossrules/drools-core/src/main/java/org/drools/util/asm/ClassFieldInspector.java =================================================================== --- trunk/labs/jbossrules/drools-core/src/main/java/org/drools/util/asm/ClassFieldInspector.java 2006-02-12 02:09:40 UTC (rev 2373) +++ trunk/labs/jbossrules/drools-core/src/main/java/org/drools/util/asm/ClassFieldInspector.java 2006-02-12 03:33:32 UTC (rev 2374) @@ -151,12 +151,12 @@ String[] exceptions) { //only want public methods that start with 'get' or 'is' if (access == Opcodes.ACC_PUBLIC) { - if (name.startsWith("get") || name.startsWith("is")) { + if (signature.startsWith( "()" ) && ( name.startsWith("get") || name.startsWith("is") ) ) { try { Method method = clazz.getMethod(name, null); methodList.add(method); } catch (NoSuchMethodException e) { - //TODO: must be a better way. We only want fields with no args. + //TODO: this shouldn't swallow } } } |