|
|
|
[
Permlink
| « Hide
]
Gunnar Sletta - 19/May/08 02:06 AM
Original task: http://jira.codehaus.org/browse/GROOVY-2326
Java Base Class:
public class Framework { public final void setProperty(String name, Object value) { } public final Object getProperty(String name) { return null; } } Groovy subclass Attached patch (for 1_5_X branch) changes Verifier to not add getProperty, setProperty, etc. if they are found in a super class. This fixes the issue but doesn't try to address whether we should do something smarter, e.g. have our own fallback getProperty if the framework one fails?
Strange but this broke 47 tests in the Grails build. I changed Verifier for the time being to just look within a class for getProperty and setProperty methods (not in super classes) before adding in the default implentations of those. This doesn't seem right to me but I haven't investigated enough to determine what the cause is. See here for furher info about the additional broken tests:
http://bamboo.ci.codehaus.org/browse/GRAILS-GROOVY15GRAILS-122 |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||