From: pero <pero@13f79535-47bb-0310-9956-ffa450edef68>
Date: Fri, 14 May 2010 12:07:00 +0000 (+0000)
Subject: Warn to be used wrong fragment name at web.xml!
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=eaed68d420345cc11e6557985899af121cb0a5cd;p=tomcat7.0

Warn to be used wrong fragment name at web.xml!

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

diff --git a/java/org/apache/catalina/deploy/LocalStrings.properties b/java/org/apache/catalina/deploy/LocalStrings.properties
index cb6d1ff9b..c4f0e43f5 100644
--- a/java/org/apache/catalina/deploy/LocalStrings.properties
+++ b/java/org/apache/catalina/deploy/LocalStrings.properties
@@ -41,3 +41,4 @@ webXml.mergeConflictString=The [{0}] with name [{1}] was defined inconsistently
 webXml.multipleOther=Multiple others entries in ordering
 webxml.unrecognisedPublicId=The public ID [{0}] did not match any of the known public ID's for web.xml files so the version could not be identified
 webXml.version.nfe=Unable to parse [{0}] from the version string [{1}]. This component of the version string will be ignored. 
+webXml.wrongFragmentName=Used a wrong fragment name {0} at web.xml absolute-ordering tag!
diff --git a/java/org/apache/catalina/deploy/WebXml.java b/java/org/apache/catalina/deploy/WebXml.java
index 7ee63f67c..cd85fc8a7 100644
--- a/java/org/apache/catalina/deploy/WebXml.java
+++ b/java/org/apache/catalina/deploy/WebXml.java
@@ -2101,6 +2101,8 @@ public class WebXml {
                     WebXml fragment = fragments.get(requestedName);
                     if (fragment != null) {
                         orderedFragments.add(fragment);
+                    } else {
+                        log.warn(sm.getString("webXml.wrongFragmentName",requestedName));
                     }
                 }
             }
diff --git a/test/org/apache/catalina/deploy/TestWebXmlOrdering.java b/test/org/apache/catalina/deploy/TestWebXmlOrdering.java
index a189163a4..64b59c712 100644
--- a/test/org/apache/catalina/deploy/TestWebXmlOrdering.java
+++ b/test/org/apache/catalina/deploy/TestWebXmlOrdering.java
@@ -144,6 +144,14 @@ public class TestWebXmlOrdering extends TestCase {
         assertFalse(iter.hasNext());
     }
 
+    public void testWebFragmentsAbsoluteWrongFragmentName() {
+        app.addAbsoluteOrdering("a");
+        app.addAbsoluteOrdering("z");
+        Set<WebXml> ordered = WebXml.orderWebFragments(app, fragments);
+        assertEquals(1,ordered.size());
+        assertEquals(fragments.get("a"),ordered.toArray()[0]);
+    }
+    
     public void testOrderWebFragmentsAbsoluteOthersEnd() {
         app.addAbsoluteOrdering("b");
         app.addAbsoluteOrdering("d");