This is an automated email from the ASF dual-hosted git repository. borinquenkid pushed a commit to branch test/document-datamapping-core-finders in repository https://gitbox.apache.org/repos/asf/grails-core.git
commit e77bcc0d8f608d7a028527cbd072234da0a04eb3 Author: Walter Duque de Estrada <[email protected]> AuthorDate: Sat Aug 15 14:54:42 2026 -0500 Resolve the GROOVY-9460 TODO on populateArgumentsForCriteria Checked: GROOVY-9460 (Class<D> rejected against a Class<?> parameter under static compilation) was fixed in Groovy 2.5.16/3.0.6/4.0.0-alpha-1. This project is on Groovy 5.0.x, well past the fix, so the Class<? extends Object> workaround is obsolete - simplify to Class<?>. Verified against every real call site, including the @CompileStatic grails.gorm.DetachedCriteria#list, which passes a Class<T> field (exactly the pattern the bug affected): grails-datamapping-core, grails-datamapping-rx, and grails-data-hibernate5-core all compile clean, and the core test suite is green. Co-Authored-By: Claude Sonnet 5 <[email protected]> --- .../main/groovy/org/grails/datastore/gorm/finders/DynamicFinder.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/grails-datamapping-core/src/main/groovy/org/grails/datastore/gorm/finders/DynamicFinder.java b/grails-datamapping-core/src/main/groovy/org/grails/datastore/gorm/finders/DynamicFinder.java index 63d59320b4..c5fd28e2e8 100644 --- a/grails-datamapping-core/src/main/groovy/org/grails/datastore/gorm/finders/DynamicFinder.java +++ b/grails-datamapping-core/src/main/groovy/org/grails/datastore/gorm/finders/DynamicFinder.java @@ -428,8 +428,7 @@ public class DynamicFinder implements FinderGrammar { * @param query The query * @param argMap The query arguments */ - //TODO: Change {code}Class<? extends Object>{code} to {class<?>} once GROOVY-9460 is fixed. - public static void populateArgumentsForCriteria(@SuppressWarnings("unused") Class<? extends Object> targetClass, Query query, Map argMap) { + public static void populateArgumentsForCriteria(@SuppressWarnings("unused") Class<?> targetClass, Query query, Map argMap) { if (argMap == null) { return; }
