This is an automated email from the ASF dual-hosted git repository.

borinquenkid pushed a commit to branch test/document-datamapping-core-services
in repository https://gitbox.apache.org/repos/asf/grails-core.git

commit fe5da6abe6534d4ce55b2dfebf5012420e90e1e5
Author: Walter Duque de Estrada <[email protected]>
AuthorDate: Sat Aug 15 19:58:17 2026 -0500

    Suppress unused-parameter warnings on getFindMethodToInvoke
    
    Same shape as FindOneImplementer.findMethodToInvoke: classNode/methodNode
    are unused in the base implementation, but the method is a protected
    extension point and FindOneStringQueryImplementer has a real subclass
    (FindOneInterfaceProjectionStringQueryImplementer), so the parameters
    stay and the warning is suppressed instead.
    
    Co-Authored-By: Claude Sonnet 5 <[email protected]>
---
 .../gorm/services/implementers/FindOneStringQueryImplementer.groovy   | 4 ++++
 1 file changed, 4 insertions(+)

diff --git 
a/grails-datamapping-core/src/main/groovy/org/grails/datastore/gorm/services/implementers/FindOneStringQueryImplementer.groovy
 
b/grails-datamapping-core/src/main/groovy/org/grails/datastore/gorm/services/implementers/FindOneStringQueryImplementer.groovy
index 1d0887735e..c6fbfd8b98 100644
--- 
a/grails-datamapping-core/src/main/groovy/org/grails/datastore/gorm/services/implementers/FindOneStringQueryImplementer.groovy
+++ 
b/grails-datamapping-core/src/main/groovy/org/grails/datastore/gorm/services/implementers/FindOneStringQueryImplementer.groovy
@@ -66,6 +66,10 @@ class FindOneStringQueryImplementer extends 
AbstractStringQueryImplementer imple
         )
     }
 
+    // classNode/methodNode are unused here, but kept so subclasses (e.g.
+    // FindOneInterfaceProjectionStringQueryImplementer) can override this as a
+    // polymorphic extension point and vary the query method by domain 
class/method
+    @SuppressWarnings('unused')
     protected String getFindMethodToInvoke(ClassNode classNode, MethodNode 
methodNode, ClassNode returnType) {
         if (AstUtils.isDomainClass(returnType)) {
             return 'find'

Reply via email to