Repository: struts
Updated Branches:
  refs/heads/master 8e9f9fb89 -> 0100a6beb


WW-4801 Only one hidden field per checkboxlist. 

Project: http://git-wip-us.apache.org/repos/asf/struts/repo
Commit: http://git-wip-us.apache.org/repos/asf/struts/commit/46153e78
Tree: http://git-wip-us.apache.org/repos/asf/struts/tree/46153e78
Diff: http://git-wip-us.apache.org/repos/asf/struts/diff/46153e78

Branch: refs/heads/master
Commit: 46153e78da63c7da40d0dbd410de1e5f25b462ab
Parents: 2fb431d
Author: gregh <[email protected]>
Authored: Tue May 30 15:02:23 2017 +0100
Committer: gregh <[email protected]>
Committed: Tue May 30 15:02:23 2017 +0100

----------------------------------------------------------------------
 .../views/java/simple/CheckboxListHandler.java  | 26 ++++++++++----------
 1 file changed, 13 insertions(+), 13 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/struts/blob/46153e78/plugins/javatemplates/src/main/java/org/apache/struts2/views/java/simple/CheckboxListHandler.java
----------------------------------------------------------------------
diff --git 
a/plugins/javatemplates/src/main/java/org/apache/struts2/views/java/simple/CheckboxListHandler.java
 
b/plugins/javatemplates/src/main/java/org/apache/struts2/views/java/simple/CheckboxListHandler.java
index 3914b30..9240938 100644
--- 
a/plugins/javatemplates/src/main/java/org/apache/struts2/views/java/simple/CheckboxListHandler.java
+++ 
b/plugins/javatemplates/src/main/java/org/apache/struts2/views/java/simple/CheckboxListHandler.java
@@ -84,22 +84,22 @@ public class CheckboxListHandler extends AbstractTagHandler 
implements TagGenera
                     characters(itemValueStr);
                 super.end("label");
 
-                // Hidden input section
-                a = new Attributes();
-                a.add("type", "hidden")
-                        .add("id",
-                                "__multiselect_"
-                                        + 
StringUtils.defaultString(StringEscapeUtils.escapeHtml4(id)))
-                        .add("name",
-                                "__multiselect_"
-                                        + 
StringUtils.defaultString(StringEscapeUtils.escapeHtml4(name)))
-                        .add("value", "").addIfTrue("disabled", disabled);
-                start("input", a);
-                end("input");
-
                 stack.pop();
                 cnt++;
             }
+
+            // Hidden input section
+            Attributes a = new Attributes();
+            a.add("type", "hidden")
+                    .add("id",
+                            "__multiselect_"
+                                    + 
StringUtils.defaultString(StringEscapeUtils.escapeHtml4(id)))
+                    .add("name",
+                            "__multiselect_"
+                                    + 
StringUtils.defaultString(StringEscapeUtils.escapeHtml4(name)))
+                    .add("value", "").addIfTrue("disabled", disabled);
+            start("input", a);
+            end("input");
         }
     }
 

Reply via email to