jamesfredley commented on PR #15557: URL: https://github.com/apache/grails-core/pull/15557#issuecomment-4731473209
### GROOVY-11985 workaround removed (burn-down) [GROOVY-11985](https://issues.apache.org/jira/browse/GROOVY-11985) - *Static method override on trait implementer ignored when called via `this` in trait body* - landed on `GROOVY_5_0_X` ([apache/groovy@aaa7cdf](https://github.com/apache/groovy/commit/aaa7cdf351f07d602be1bf173ca06e021efc3f5f)) and is now **Resolved / Fixed**, fix-version **5.0.7**. Verified present in the consumed `5.0.7-SNAPSHOT` (build `5.0.7-20260617.134244-12`). The reflective `resolveDefaultNullable(...)` helpers are gone: - `Validateable` is back to the idiomatic in-trait `this.defaultNullable()` calls, with `getConstraintsMap()` again `@CompileDynamic`. - `BeanPropertyAccessorFactory` is back to `metaClass.invokeStaticMethod(type, 'defaultNullable')`. Both files are once again **byte-identical to `8.0.x`** - their Groovy-5 diff is now zero. **Verified green on the snapshot** (`--refresh-dependencies`): - `:grails-validation:test` - including `ValidateableTraitSpec` *...overridden defaultNullable which are not explicitly constrained are not accessed during validation*. - `:grails-fields:test` - including `CommandPropertyAccessorSpec` *respect defaultNullable() when evaluating constraints of a Validateable*. That leaves a single open item in the workarounds table at the top - [GROOVY-12063](https://issues.apache.org/jira/browse/GROOVY-12063) (`ConstrainedProperty.DEFAULT_MESSAGES`), still **Open** with no fix-version. Commit 5fc96aee06. -- 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]
