macrodef element names are case insensentive due to use of DynamicConfigurator

PR: 26225
Reported by: John Sichi


git-svn-id: https://svn.apache.org/repos/asf/ant/core/trunk@275929 13f79535-47bb-0310-9956-ffa450edef68

docs/manual/CoreTasks/macrodef.html

@@ -98,6 +98,9 @@
       The contents of the nested elements of the task instance
       are placed in the templated task at the tag name.
     </p>
+    <p>
+      The case of the element name is ignored.
+    </p>
     <h3>Parameters</h3>
     <table border="1" cellpadding="2" cellspacing="0">
       <tr>

src/etc/testcases/taskdefs/macrodef.xml

@@ -86,4 +86,19 @@
     <ignore myattribute="a"/>
     <ignore Myattribute="b"/>
   </target>
+
+  <target name="ignore-element-case">
+    <macrodef name="ignore">
+      <element name="MyElement"/>
+      <sequential>
+        <myElement/>
+        <MyElEmEnT/>
+      </sequential>
+    </macrodef>
+    <ignore>
+      <MYELEMENT>
+        <echo>nested element</echo>
+      </MYELEMENT>
+    </ignore>
+  </target>
 </project>

src/main/org/apache/tools/ant/taskdefs/MacroDef.java

@@ -395,7 +395,7 @@ public class MacroDef extends AntlibDefinition  {
                 throw new BuildException(
                     "Illegal name [" + name + "] for attribute");
             }
-            this.name = name;
+            this.name = name.toLowerCase(Locale.US);
         }
 
         /**

src/main/org/apache/tools/ant/taskdefs/MacroInstance.java

@@ -277,7 +277,7 @@ public class MacroInstance extends Task implements DynamicConfigurator {
         while (e.hasMoreElements()) {
             RuntimeConfigurable r = (RuntimeConfigurable) e.nextElement();
             UnknownElement unknownElement = (UnknownElement) r.getProxy();
-            String tag = unknownElement.getTaskType();
+            String tag = unknownElement.getTaskType().toLowerCase(Locale.US);
             MacroDef.TemplateElement templateElement =
                 (MacroDef.TemplateElement) getNsElements().get(tag);
             if (templateElement == null) {

src/testcases/org/apache/tools/ant/taskdefs/MacroDefTest.java

@@ -109,5 +109,11 @@ public class MacroDefTest extends BuildFileTest {
             "ignorecase",
             "a is ab is b");
     }
+
+    public void testIgnoreElementCase() {
+        expectLog(
+            "ignore-element-case",
+            "nested elementnested element");
+    }
 }