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

davydotcom pushed a commit to branch feature/taglib-method-actions
in repository https://gitbox.apache.org/repos/asf/grails-core.git

commit 3854084fd3f61bb327e1cf75a2e5590e295eaa06
Author: David Estes <[email protected]>
AuthorDate: Wed Feb 25 22:27:01 2026 -0500

    Simplify FormTagLib typed overloads by removing redundant name reassignment
    
    - keep typed overloads delegating to private implementation helpers
    - remove unnecessary attrs.name writes since typed args are sourced from 
attrs
    - preserve behavior validated by focused FormTagLib and method-tag test 
suites
    
    Co-Authored-By: Oz <[email protected]>
---
 .../src/main/groovy/org/grails/plugins/web/taglib/FormTagLib.groovy   | 4 ----
 1 file changed, 4 deletions(-)

diff --git 
a/grails-gsp/plugin/src/main/groovy/org/grails/plugins/web/taglib/FormTagLib.groovy
 
b/grails-gsp/plugin/src/main/groovy/org/grails/plugins/web/taglib/FormTagLib.groovy
index 5b357f107f..7d4f84b8a8 100644
--- 
a/grails-gsp/plugin/src/main/groovy/org/grails/plugins/web/taglib/FormTagLib.groovy
+++ 
b/grails-gsp/plugin/src/main/groovy/org/grails/plugins/web/taglib/FormTagLib.groovy
@@ -112,7 +112,6 @@ class FormTagLib implements ApplicationContextAware, 
InitializingBean, TagLibrar
     }
 
     def textField(String name, Map attrs) {
-        attrs.name = name
         textFieldImpl(attrs)
     }
 
@@ -137,7 +136,6 @@ class FormTagLib implements ApplicationContextAware, 
InitializingBean, TagLibrar
     }
 
     def passwordField(String name, Map attrs) {
-        attrs.name = name
         passwordFieldImpl(attrs)
     }
 
@@ -160,7 +158,6 @@ class FormTagLib implements ApplicationContextAware, 
InitializingBean, TagLibrar
     }
 
     def hiddenField(String name, Map attrs) {
-        attrs.name = name
         hiddenFieldTagImpl(attrs)
     }
 
@@ -191,7 +188,6 @@ class FormTagLib implements ApplicationContextAware, 
InitializingBean, TagLibrar
     }
 
     def submitButton(String name, Map attrs) {
-        attrs.name = name
         submitButtonImpl(attrs)
     }
 

Reply via email to