jamesfredley commented on code in PR #16491:
URL: https://github.com/apache/grails-core/pull/16491#discussion_r4175044052
##########
grails-web-databinding/src/main/groovy/grails/web/databinding/DataBindingUtils.java:
##########
@@ -150,6 +176,10 @@ static List getBindablePropertyNames(final Object object) {
}
static List getUnbindablePropertyNames(final Object object) {
+ if (BindingIncludeLists.inheritsConstraintsMap(object.getClass())) {
+ // The inherited accessor holds the superclass's constraints, so
the class's own are evaluated once.
+ return getUnbindablePropertyNames(object.getClass());
Review Comment:
Resolved in 9fd34ca839. The fallback now matches only an inherited static
getConstraintsMap that carries Validateable's @TraitBridge. An inherited
instance getter no longer matches, so the instance map is used again. The new
spec covers two instances with different bindable: false rules through both
bindData and DataBindingUtils.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]