Enforce data binding to instance should not affect static properties (#12666)

Author: mattmoss

grails-databinding/src/main/groovy/grails/databinding/SimpleDataBinder.groovy

@@ -264,7 +264,10 @@ class SimpleDataBinder implements DataBinder {
     }
 
     protected boolean isOkToBind(MetaProperty property, List whitelist, List blacklist) {
-        isOkToBind(property.name, whitelist, blacklist) && (property.type != null && !(ClassLoader.class.isAssignableFrom(property.type) || ProtectionDomain.class.isAssignableFrom(property.type)))
+        isOkToBind(property.name, whitelist, blacklist) &&
+                (property.type != null) &&
+                !Modifier.isStatic(property.modifiers) &&
+                !(ClassLoader.class.isAssignableFrom(property.type) || ProtectionDomain.class.isAssignableFrom(property.type))
     }
 
     protected IndexedPropertyReferenceDescriptor getIndexedPropertyReferenceDescriptor(propName) {

grails-databinding/src/test/groovy/grails/databinding/SimpleDataBinderSpec.groovy

@@ -644,6 +644,30 @@ class SimpleDataBinderSpec extends Specification {
         then:
         fromAbstractB.a.data == 'abc'
     }
+
+    void 'Test binding when property is static Object, it should not bind'() {
+        given:
+        def binder = new SimpleDataBinder()
+        def widget = new Widget()
+
+        when:
+        binder.bind widget, [objectStaticProp: 6174] as SimpleMapDataBindingSource
+
+        then:
+        widget.objectStaticProp instanceof Closure
+    }
+
+    void 'Test binding when property is static, it should not bind'() {
+        given:
+        def binder = new SimpleDataBinder()
+        def widget = new Widget()
+
+        when:
+        binder.bind widget, [stringStaticProp: 'abc'] as SimpleMapDataBindingSource
+
+        then:
+        widget.stringStaticProp == 'unchanged'
+    }
 }
 
 class Factory {
@@ -673,6 +697,8 @@ class Widget {
     List<Integer> listOfIntegers = []
     List<Long> listOfLongs = []
     Set<Factory> factories
+    static objectStaticProp = { -> 495 }
+    static String stringStaticProp = 'unchanged'
 }
 
 class Gadget extends Widget {