diff --git a/core/src/main/java/org/apache/struts2/components/UIBean.java b/core/src/main/java/org/apache/struts2/components/UIBean.java
index 4f806f4fc..dbd98ff3e 100644
--- a/core/src/main/java/org/apache/struts2/components/UIBean.java
+++ b/core/src/main/java/org/apache/struts2/components/UIBean.java
@@ -859,7 +859,7 @@ public abstract class UIBean extends Component {
 	protected String escape(String name) {
         // escape any possible values that can make the ID painful to work with in JavaScript
         if (name != null) {
-            return name.replaceAll("[^a-zA-Z0-9_]", "_");
+            return name.replaceAll("[\\/\\.\\[\\]]", "_");
         } else {
             return "";
         }
diff --git a/core/src/main/resources/template/xhtml/form-close-validate.ftl b/core/src/main/resources/template/xhtml/form-close-validate.ftl
index 533a4f78f..b9521058c 100644
--- a/core/src/main/resources/template/xhtml/form-close-validate.ftl
+++ b/core/src/main/resources/template/xhtml/form-close-validate.ftl
@@ -35,7 +35,7 @@ END SNIPPET: supported-validators
 -->
 <#if ((parameters.validate?default(false) == true) && (parameters.performValidation?default(false) == true))>
 <script type="text/javascript">
-    function validateForm_${parameters.id}() {
+    function validateForm_${parameters.id?replace('[^a-zA-Z0-9_]', '_', 'r')}() {
         form = document.getElementById("${parameters.id}");
         clearErrorMessages(form);
         clearErrorLabels(form);
diff --git a/core/src/main/resources/template/xhtml/form-validate.ftl b/core/src/main/resources/template/xhtml/form-validate.ftl
index 5d9fc7bce..6597f4eb6 100644
--- a/core/src/main/resources/template/xhtml/form-validate.ftl
+++ b/core/src/main/resources/template/xhtml/form-validate.ftl
@@ -24,8 +24,8 @@
 	<script type="text/javascript" src="${base}/struts/xhtml/validation.js"></script>
 	<script type="text/javascript" src="${base}/struts/utils.js"></script>
 	<#if parameters.onsubmit??>
-		${tag.addParameter('onsubmit', "${parameters.onsubmit}; return validateForm_${parameters.id}();")}
+		${tag.addParameter('onsubmit', "${parameters.onsubmit}; return validateForm_${parameters.id?replace('[^a-zA-Z0-9_]', '_', 'r')}();")}
 	<#else>
-		${tag.addParameter('onsubmit', "return validateForm_${parameters.id}();")}
+		${tag.addParameter('onsubmit', "return validateForm_${parameters.id?replace('[^a-zA-Z0-9_]', '_', 'r')}();")}
 	</#if>
 </#if>
diff --git a/core/src/test/java/org/apache/struts2/components/UIBeanTest.java b/core/src/test/java/org/apache/struts2/components/UIBeanTest.java
index accf885d1..0f00690f5 100644
--- a/core/src/test/java/org/apache/struts2/components/UIBeanTest.java
+++ b/core/src/test/java/org/apache/struts2/components/UIBeanTest.java
@@ -98,8 +98,8 @@ public class UIBeanTest extends StrutsTestCase {
         assertEquals(bean.escape("hello[world"), "hello_world");
         assertEquals(bean.escape("hello.world"), "hello_world");
         assertEquals(bean.escape("hello]world"), "hello_world");
-        assertEquals(bean.escape("hello!world"), "hello_world");
-        assertEquals(bean.escape("hello!@#$%^&*()world"), "hello__________world");
+        assertEquals(bean.escape("hello!world"), "hello!world");
+        assertEquals(bean.escape("hello!@#$%^&*()world"), "hello!@#$%^&*()world");
     }
 
     public void testEscapeId() throws Exception {
diff --git a/core/src/test/resources/org/apache/struts2/views/jsp/ui/Radio-6.txt b/core/src/test/resources/org/apache/struts2/views/jsp/ui/Radio-6.txt
index c8cc2e640..d58184796 100644
--- a/core/src/test/resources/org/apache/struts2/views/jsp/ui/Radio-6.txt
+++ b/core/src/test/resources/org/apache/struts2/views/jsp/ui/Radio-6.txt
@@ -1,11 +1,11 @@
 <tr>
     <td class="tdLabel"></td>
     <td>
-        <input type="radio" name="myMap['name']" id="myMap__name__"value=""/>
-        <label for="myMap__name__">N/A</label>
-        <input type="radio" name="myMap['name']" id="myMap__name__Opt." value="Opt."/>
-        <label for="myMap__name__Opt.">Opt.</label>
-        <input type="radio" name="myMap['name']" id="myMap__name__Std." checked="checked" value="Std."/>
-        <label for="myMap__name__Std.">Std.</label>
+        <input type="radio" name="myMap['name']" id="myMap_'name'_"value=""/>
+        <label for="myMap_'name'_">N/A</label>
+        <input type="radio" name="myMap['name']" id="myMap_'name'_Opt." value="Opt."/>
+        <label for="myMap_'name'_Opt.">Opt.</label>
+        <input type="radio" name="myMap['name']" id="myMap_'name'_Std." checked="checked" value="Std."/>
+        <label for="myMap_'name'_Std.">Std.</label>
     </td>
 </tr>