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 ce77eacb21185fcc1bccc7c423366541cb34c372
Author: Walter Duque de Estrada <[email protected]>
AuthorDate: Sat Aug 15 15:05:39 2026 -0500

    Collapse the redundant LAZY-check branch in getFetchMode
    
    Both the "select"/LAZY-named-mode if-branch and the final fallback
    returned FetchType.LAZY, so the if condition was a no-op guard around
    an outcome identical to just falling through - drop it.
    
    Co-Authored-By: Claude Sonnet 5 <[email protected]>
---
 .../main/groovy/org/grails/datastore/gorm/finders/DynamicFinder.java   | 3 ---
 1 file changed, 3 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 1cd8dfac6d..37a873855d 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
@@ -537,9 +537,6 @@ public class DynamicFinder implements FinderGrammar {
         if (name.equalsIgnoreCase(FetchType.EAGER.toString()) || 
name.equalsIgnoreCase("join")) {
             return FetchType.EAGER;
         }
-        if (name.equalsIgnoreCase(FetchType.LAZY.toString()) || 
name.equalsIgnoreCase("select")) {
-            return FetchType.LAZY;
-        }
         return FetchType.LAZY;
     }
 

Reply via email to