Skip to content

Commit 8cd50d2

Browse files
committed
fix for GRAILS-9004 "where/findall closures can fail on entities with relationships"
1 parent 1b9e214 commit 8cd50d2

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

grails-core/src/main/groovy/org/codehaus/groovy/grails/commons/GrailsClassUtils.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -919,7 +919,10 @@ public static String getPropertyForGetter(String getterName) {
919919
}
920920

921921
private static String convertPropertyName(String prop) {
922-
if (Character.isUpperCase(prop.charAt(0)) && Character.isUpperCase(prop.charAt(1))) {
922+
if(prop.length() == 1) {
923+
return prop.toLowerCase();
924+
}
925+
else if (Character.isUpperCase(prop.charAt(0)) && Character.isUpperCase(prop.charAt(1))) {
923926
return prop;
924927
}
925928
if (Character.isDigit(prop.charAt(0))) {

grails-test-suite-uber/src/test/groovy/org/codehaus/groovy/grails/commons/GrailsClassUtilsTests.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -128,6 +128,7 @@ public void testIsGetterOrSetter() {
128128
public void testGetPropertyForGetter() {
129129
assertEquals("something", GrailsClassUtils.getPropertyForGetter("getSomething"));
130130
assertEquals("URL", GrailsClassUtils.getPropertyForGetter("getURL"));
131+
assertEquals("p", GrailsClassUtils.getPropertyForGetter("getP"));
131132
}
132133

133134
public void testGetStaticField() {

0 commit comments

Comments
 (0)