You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2011/10/13 23:14:17 UTC

svn commit: r1183105 [2/2] - /tomcat/trunk/java/org/apache/catalina/deploy/WebXml.java

Modified: tomcat/trunk/java/org/apache/catalina/deploy/WebXml.java
URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/deploy/WebXml.java?rev=1183105&r1=1183104&r2=1183105&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/deploy/WebXml.java (original)
+++ tomcat/trunk/java/org/apache/catalina/deploy/WebXml.java Thu Oct 13 21:14:16 2011
@@ -1,2217 +1,2217 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements.  See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License.  You may obtain a copy of the License at
- * 
- *      http://www.apache.org/licenses/LICENSE-2.0
- * 
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-
-package org.apache.catalina.deploy;
-
-import java.net.URL;
-import java.util.ArrayList;
-import java.util.EnumSet;
-import java.util.HashMap;
-import java.util.HashSet;
-import java.util.Iterator;
-import java.util.LinkedHashMap;
-import java.util.LinkedHashSet;
-import java.util.LinkedList;
-import java.util.List;
-import java.util.Map;
-import java.util.Map.Entry;
-import java.util.Set;
-
-import javax.servlet.MultipartConfigElement;
-import javax.servlet.SessionCookieConfig;
-import javax.servlet.SessionTrackingMode;
-import javax.servlet.descriptor.JspPropertyGroupDescriptor;
-import javax.servlet.descriptor.TaglibDescriptor;
-
-import org.apache.catalina.Context;
-import org.apache.catalina.Wrapper;
-import org.apache.catalina.core.ApplicationJspPropertyGroupDescriptor;
-import org.apache.catalina.core.ApplicationTaglibDescriptor;
-import org.apache.tomcat.util.res.StringManager;
-
-/**
- * Representation of common elements of web.xml and web-fragment.xml. Provides
- * a repository for parsed data before the elements are merged.
- * Validation is spread between multiple classes:
- * The digester checks for structural correctness (eg single login-config)
- * This class checks for invalid duplicates (eg filter/servlet names)
- * StandardContext will check validity of values (eg URL formats etc)
- */
-public class WebXml {
-    
-    protected static final String ORDER_OTHERS =
-        "org.apache.catalina.order.others";
-    
-    private static final StringManager sm =
-        StringManager.getManager(Constants.Package);
-
-    private static final org.apache.juli.logging.Log log=
-        org.apache.juli.logging.LogFactory.getLog(WebXml.class);
-
-    // Global defaults are overridable but Servlets and Servlet mappings need to
-    // be unique. Duplicates normally trigger an error. This flag indicates if
-    // newly added Servlet elements are marked as overridable.
-    private boolean overridable = false;
-    public boolean isOverridable() {
-        return overridable;
-    }
-    public void setOverridable(boolean overridable) {
-        this.overridable = overridable;
-    }
-
-    // web.xml only elements
-    // Absolute Ordering
-    private Set<String> absoluteOrdering = null;
-    public void addAbsoluteOrdering(String fragmentName) {
-        if (absoluteOrdering == null) {
-            absoluteOrdering = new LinkedHashSet<String>();
-        }
-        absoluteOrdering.add(fragmentName);
-    }
-    public void addAbsoluteOrderingOthers() {
-        if (absoluteOrdering == null) {
-            absoluteOrdering = new LinkedHashSet<String>();
-        }
-        absoluteOrdering.add(ORDER_OTHERS);
-    }
-    public Set<String> getAbsoluteOrdering() {
-        return absoluteOrdering;
-    }
-
-    // web-fragment.xml only elements
-    // Relative ordering
-    private Set<String> after = new LinkedHashSet<String>();
-    public void addAfterOrdering(String fragmentName) {
-        after.add(fragmentName);
-    }
-    public void addAfterOrderingOthers() {
-        if (before.contains(ORDER_OTHERS)) {
-            throw new IllegalArgumentException(sm.getString(
-                    "webXml.multipleOther"));
-        }
-        after.add(ORDER_OTHERS);
-    }
-    public Set<String> getAfterOrdering() { return after; }
-    
-    private Set<String> before = new LinkedHashSet<String>();
-    public void addBeforeOrdering(String fragmentName) {
-        before.add(fragmentName);
-    }
-    public void addBeforeOrderingOthers() {
-        if (after.contains(ORDER_OTHERS)) {
-            throw new IllegalArgumentException(sm.getString(
-                    "webXml.multipleOther"));
-        }
-        before.add(ORDER_OTHERS);
-    }
-    public Set<String> getBeforeOrdering() { return before; }
-
-    // Common elements and attributes
-    
-    // Required attribute of web-app element
-    public String getVersion() {
-        StringBuilder sb = new StringBuilder(3);
-        sb.append(majorVersion);
-        sb.append('.');
-        sb.append(minorVersion);
-        return sb.toString();
-    }
-    /**
-     * Set the version for this web.xml file
-     * @param version   Values of <code>null</code> will be ignored
-     */
-    public void setVersion(String version) {
-        if (version == null) return;
-        
-        // Update major and minor version
-        // Expected format is n.n - allow for any number of digits just in case
-        String major = null;
-        String minor = null;
-        int split = version.indexOf('.');
-        if (split < 0) {
-            // Major only
-            major = version;
-        } else {
-            major = version.substring(0, split);
-            minor = version.substring(split + 1);
-        }
-        if (major == null || major.length() == 0) {
-            majorVersion = 0;
-        } else {
-            try {
-                majorVersion = Integer.parseInt(major);
-            } catch (NumberFormatException nfe) {
-                log.warn(sm.getString("webXml.version.nfe", major, version),
-                        nfe);
-                majorVersion = 0;
-            }
-        }
-        
-        if (minor == null || minor.length() == 0) {
-            minorVersion = 0;
-        } else {
-            try {
-                minorVersion = Integer.parseInt(minor);
-            } catch (NumberFormatException nfe) {
-                log.warn(sm.getString("webXml.version.nfe", minor, version),
-                        nfe);
-                minorVersion = 0;
-            }
-        }
-    }
-
-
-    // Optional publicId attribute
-    private String publicId = null;
-    public String getPublicId() { return publicId; }
-    public void setPublicId(String publicId) {
-        // Update major and minor version
-        if (publicId == null) {
-            // skip
-        } else if (org.apache.catalina.startup.Constants.WebSchemaPublicId_30.
-                equalsIgnoreCase(publicId) ||
-                org.apache.catalina.startup.Constants.WebFragmentSchemaPublicId_30.
-                equalsIgnoreCase(publicId)) {
-            majorVersion = 3;
-            minorVersion = 0;
-            this.publicId = publicId;
-        } else if (org.apache.catalina.startup.Constants.WebSchemaPublicId_25.
-                equalsIgnoreCase(publicId)) {
-            majorVersion = 2;
-            minorVersion = 5;
-            this.publicId = publicId;
-        } else if (org.apache.catalina.startup.Constants.WebSchemaPublicId_24.
-                equalsIgnoreCase(publicId)) {
-            majorVersion = 2;
-            minorVersion = 4;
-            this.publicId = publicId;
-        } else if (org.apache.catalina.startup.Constants.WebDtdPublicId_23.
-                equalsIgnoreCase(publicId)) {
-            majorVersion = 2;
-            minorVersion = 3;
-            this.publicId = publicId;
-        } else if (org.apache.catalina.startup.Constants.WebDtdPublicId_22.
-                equalsIgnoreCase(publicId)) {
-            majorVersion = 2;
-            minorVersion = 2;
-            this.publicId = publicId;
-        } else if ("datatypes".equals(publicId)) {
-            // Will occur when validation is enabled and dependencies are
-            // traced back. Ignore it.
-        } else {
-            // Unrecognised publicId
-            log.warn(sm.getString("webxml.unrecognisedPublicId", publicId));
-        }
-    }
-    
-    // Optional metadata-complete attribute
-    private boolean metadataComplete = false;
-    public boolean isMetadataComplete() { return metadataComplete; }
-    public void setMetadataComplete(boolean metadataComplete) {
-        this.metadataComplete = metadataComplete; }
-    
-    // Optional name element
-    private String name = null;
-    public String getName() { return name; }
-    public void setName(String name) {
-        if (ORDER_OTHERS.equalsIgnoreCase(name)) {
-            // This is unusual. This name will be ignored. Log the fact.
-            log.warn(sm.getString("webXml.reservedName", name));
-        } else {
-            this.name = name;
-        }
-    }
-
-    // Derived major and minor version attributes
-    // Default to 3.0 until we know otherwise
-    private int majorVersion = 3;
-    private int minorVersion = 0;
-    public int getMajorVersion() { return majorVersion; }
-    public int getMinorVersion() { return minorVersion; }
-    
-    // web-app elements
-    // TODO: Ignored elements:
-    // - description
-    // - icon
-
-    // display-name - TODO should support multiple with language
-    private String displayName = null;
-    public String getDisplayName() { return displayName; }
-    public void setDisplayName(String displayName) {
-        this.displayName = displayName;
-    }
-    
-    // distributable
-    private boolean distributable = false;
-    public boolean isDistributable() { return distributable; }
-    public void setDistributable(boolean distributable) {
-        this.distributable = distributable;
-    }
-    
-    // context-param
-    // TODO: description (multiple with language) is ignored
-    private Map<String,String> contextParams = new HashMap<String,String>();
-    public void addContextParam(String param, String value) {
-        contextParams.put(param, value);
-    }
-    public Map<String,String> getContextParams() { return contextParams; }
-    
-    // filter
-    // TODO: Should support multiple description elements with language
-    // TODO: Should support multiple display-name elements with language
-    // TODO: Should support multiple icon elements
-    // TODO: Description for init-param is ignored
-    private Map<String,FilterDef> filters =
-        new LinkedHashMap<String,FilterDef>();
-    public void addFilter(FilterDef filter) {
-        if (filters.containsKey(filter.getFilterName())) {
-            // Filter names must be unique within a web(-fragment).xml
-            throw new IllegalArgumentException(
-                    sm.getString("webXml.duplicateFilter",
-                            filter.getFilterName()));
-        }
-        filters.put(filter.getFilterName(), filter);
-    }
-    public Map<String,FilterDef> getFilters() { return filters; }
-    
-    // filter-mapping
-    private Set<FilterMap> filterMaps = new LinkedHashSet<FilterMap>();
-    private Set<String> filterMappingNames = new HashSet<String>();
-    public void addFilterMapping(FilterMap filterMap) {
-        filterMaps.add(filterMap);
-        filterMappingNames.add(filterMap.getFilterName());
-    }
-    public Set<FilterMap> getFilterMappings() { return filterMaps; }
-    
-    // listener
-    // TODO: description (multiple with language) is ignored
-    // TODO: display-name (multiple with language) is ignored
-    // TODO: icon (multiple) is ignored
-    private Set<String> listeners = new LinkedHashSet<String>();
-    public void addListener(String className) {
-        listeners.add(className);
-    }
-    public Set<String> getListeners() { return listeners; }
-    
-    // servlet
-    // TODO: description (multiple with language) is ignored
-    // TODO: display-name (multiple with language) is ignored
-    // TODO: icon (multiple) is ignored
-    // TODO: init-param/description (multiple with language) is ignored
-    // TODO: security-role-ref/description (multiple with language) is ignored
-    private Map<String,ServletDef> servlets = new HashMap<String,ServletDef>();
-    public void addServlet(ServletDef servletDef) {
-        servlets.put(servletDef.getServletName(), servletDef);
-        if (overridable) {
-            servletDef.setOverridable(overridable);
-        }
-    }
-    public Map<String,ServletDef> getServlets() { return servlets; }
-    
-    // servlet-mapping
-    private Map<String,String> servletMappings = new HashMap<String,String>();
-    private Set<String> servletMappingNames = new HashSet<String>();
-    public void addServletMapping(String urlPattern, String servletName) {
-        servletMappings.put(urlPattern, servletName);
-        servletMappingNames.add(servletName);
-    }
-    public Map<String,String> getServletMappings() { return servletMappings; }
-    
-    // session-config
-    // Digester will check there is only one of these
-    private SessionConfig sessionConfig = new SessionConfig();
-    public void setSessionConfig(SessionConfig sessionConfig) {
-        this.sessionConfig = sessionConfig;
-    }
-    public SessionConfig getSessionConfig() { return sessionConfig; }
-    
-    // mime-mapping
-    private Map<String,String> mimeMappings = new HashMap<String,String>();
-    public void addMimeMapping(String extension, String mimeType) {
-        mimeMappings.put(extension, mimeType);
-    }
-    public Map<String,String> getMimeMappings() { return mimeMappings; }
-    
-    // welcome-file-list merge control
-    private boolean replaceWelcomeFiles = false;
-    private boolean alwaysAddWelcomeFiles = true;
-    /**
-     * When merging/parsing web.xml files into this web.xml should the current
-     * set be completely replaced?
-     */
-    public void setReplaceWelcomeFiles(boolean replaceWelcomeFiles) {
-        this.replaceWelcomeFiles = replaceWelcomeFiles;
-    }
-    /**
-     * When merging from this web.xml, should the welcome files be added to the
-     * target web.xml even if it already contains welcome file definitions.
-     */
-    public void setAlwaysAddWelcomeFiles(boolean alwaysAddWelcomeFiles) {
-        this.alwaysAddWelcomeFiles = alwaysAddWelcomeFiles;
-    }
-
-    // welcome-file-list
-    private Set<String> welcomeFiles = new LinkedHashSet<String>();
-    public void addWelcomeFile(String welcomeFile) {
-        if (replaceWelcomeFiles) {
-            welcomeFiles.clear();
-            replaceWelcomeFiles = false;
-        }
-        welcomeFiles.add(welcomeFile);
-    }
-    public Set<String> getWelcomeFiles() { return welcomeFiles; }
-    
-    // error-page
-    private Map<String,ErrorPage> errorPages = new HashMap<String,ErrorPage>();
-    public void addErrorPage(ErrorPage errorPage) {
-        errorPages.put(errorPage.getName(), errorPage);
-    }
-    public Map<String,ErrorPage> getErrorPages() { return errorPages; }
-    
-    // Digester will check there is only one jsp-config
-    // jsp-config/taglib or taglib (2.3 and earlier)
-    private Map<String,String> taglibs = new HashMap<String,String>();
-    public void addTaglib(String uri, String location) {
-        if (taglibs.containsKey(uri)) {
-            // Taglib URIs must be unique within a web(-fragment).xml
-            throw new IllegalArgumentException(
-                    sm.getString("webXml.duplicateTaglibUri", uri));
-        }
-        taglibs.put(uri, location);
-    }
-    public Map<String,String> getTaglibs() { return taglibs; }
-    
-    // jsp-config/jsp-property-group
-    private Set<JspPropertyGroup> jspPropertyGroups =
-        new HashSet<JspPropertyGroup>();
-    public void addJspPropertyGroup(JspPropertyGroup propertyGroup) {
-        jspPropertyGroups.add(propertyGroup);
-    }
-    public Set<JspPropertyGroup> getJspPropertyGroups() {
-        return jspPropertyGroups;
-    }
-
-    // security-constraint
-    // TODO: Should support multiple display-name elements with language
-    // TODO: Should support multiple description elements with language
-    private Set<SecurityConstraint> securityConstraints =
-        new HashSet<SecurityConstraint>();
-    public void addSecurityConstraint(SecurityConstraint securityConstraint) {
-        securityConstraints.add(securityConstraint);
-    }
-    public Set<SecurityConstraint> getSecurityConstraints() {
-        return securityConstraints;
-    }
-    
-    // login-config
-    // Digester will check there is only one of these
-    private LoginConfig loginConfig = null;
-    public void setLoginConfig(LoginConfig loginConfig) {
-        this.loginConfig = loginConfig;
-    }
-    public LoginConfig getLoginConfig() { return loginConfig; }
-    
-    // security-role
-    // TODO: description (multiple with language) is ignored
-    private Set<String> securityRoles = new HashSet<String>();
-    public void addSecurityRole(String securityRole) {
-        securityRoles.add(securityRole);
-    }
-    public Set<String> getSecurityRoles() { return securityRoles; }
-    
-    // env-entry
-    // TODO: Should support multiple description elements with language
-    private Map<String,ContextEnvironment> envEntries =
-        new HashMap<String,ContextEnvironment>();
-    public void addEnvEntry(ContextEnvironment envEntry) {
-        if (envEntries.containsKey(envEntry.getName())) {
-            // env-entry names must be unique within a web(-fragment).xml
-            throw new IllegalArgumentException(
-                    sm.getString("webXml.duplicateEnvEntry",
-                            envEntry.getName()));
-        }
-        envEntries.put(envEntry.getName(),envEntry);
-    }
-    public Map<String,ContextEnvironment> getEnvEntries() { return envEntries; }
-    
-    // ejb-ref
-    // TODO: Should support multiple description elements with language
-    private Map<String,ContextEjb> ejbRefs = new HashMap<String,ContextEjb>();
-    public void addEjbRef(ContextEjb ejbRef) {
-        ejbRefs.put(ejbRef.getName(),ejbRef);
-    }
-    public Map<String,ContextEjb> getEjbRefs() { return ejbRefs; }
-    
-    // ejb-local-ref
-    // TODO: Should support multiple description elements with language
-    private Map<String,ContextLocalEjb> ejbLocalRefs =
-        new HashMap<String,ContextLocalEjb>();
-    public void addEjbLocalRef(ContextLocalEjb ejbLocalRef) {
-        ejbLocalRefs.put(ejbLocalRef.getName(),ejbLocalRef);
-    }
-    public Map<String,ContextLocalEjb> getEjbLocalRefs() {
-        return ejbLocalRefs;
-    }
-    
-    // service-ref
-    // TODO: Should support multiple description elements with language
-    // TODO: Should support multiple display-names elements with language
-    // TODO: Should support multiple icon elements ???
-    private Map<String,ContextService> serviceRefs =
-        new HashMap<String,ContextService>();
-    public void addServiceRef(ContextService serviceRef) {
-        serviceRefs.put(serviceRef.getName(), serviceRef);
-    }
-    public Map<String,ContextService> getServiceRefs() { return serviceRefs; }
-    
-    // resource-ref
-    // TODO: Should support multiple description elements with language
-    private Map<String,ContextResource> resourceRefs =
-        new HashMap<String,ContextResource>();
-    public void addResourceRef(ContextResource resourceRef) {
-        if (resourceRefs.containsKey(resourceRef.getName())) {
-            // resource-ref names must be unique within a web(-fragment).xml
-            throw new IllegalArgumentException(
-                    sm.getString("webXml.duplicateResourceRef",
-                            resourceRef.getName()));
-        }
-        resourceRefs.put(resourceRef.getName(), resourceRef);
-    }
-    public Map<String,ContextResource> getResourceRefs() {
-        return resourceRefs;
-    }
-    
-    // resource-env-ref
-    // TODO: Should support multiple description elements with language
-    private Map<String,ContextResourceEnvRef> resourceEnvRefs =
-        new HashMap<String,ContextResourceEnvRef>();
-    public void addResourceEnvRef(ContextResourceEnvRef resourceEnvRef) {
-        if (resourceEnvRefs.containsKey(resourceEnvRef.getName())) {
-            // resource-env-ref names must be unique within a web(-fragment).xml
-            throw new IllegalArgumentException(
-                    sm.getString("webXml.duplicateResourceEnvRef",
-                            resourceEnvRef.getName()));
-        }
-        resourceEnvRefs.put(resourceEnvRef.getName(), resourceEnvRef);
-    }
-    public Map<String,ContextResourceEnvRef> getResourceEnvRefs() {
-        return resourceEnvRefs;
-    }
-    
-    // message-destination-ref
-    // TODO: Should support multiple description elements with language
-    private Map<String,MessageDestinationRef> messageDestinationRefs =
-        new HashMap<String,MessageDestinationRef>();
-    public void addMessageDestinationRef(
-            MessageDestinationRef messageDestinationRef) {
-        if (messageDestinationRefs.containsKey(
-                messageDestinationRef.getName())) {
-            // message-destination-ref names must be unique within a
-            // web(-fragment).xml
-            throw new IllegalArgumentException(sm.getString(
-                    "webXml.duplicateMessageDestinationRef",
-                    messageDestinationRef.getName()));
-        }
-        messageDestinationRefs.put(messageDestinationRef.getName(),
-                messageDestinationRef);
-    }
-    public Map<String,MessageDestinationRef> getMessageDestinationRefs() {
-        return messageDestinationRefs;
-    }
-    
-    // message-destination
-    // TODO: Should support multiple description elements with language
-    // TODO: Should support multiple display-names elements with language
-    // TODO: Should support multiple icon elements ???
-    private Map<String,MessageDestination> messageDestinations =
-        new HashMap<String,MessageDestination>();
-    public void addMessageDestination(
-            MessageDestination messageDestination) {
-        if (messageDestinations.containsKey(
-                messageDestination.getName())) {
-            // message-destination names must be unique within a
-            // web(-fragment).xml
-            throw new IllegalArgumentException(
-                    sm.getString("webXml.duplicateMessageDestination",
-                            messageDestination.getName()));
-        }
-        messageDestinations.put(messageDestination.getName(),
-                messageDestination);
-    }
-    public Map<String,MessageDestination> getMessageDestinations() {
-        return messageDestinations;
-    }
-    
-    // locale-encoging-mapping-list
-    private Map<String,String> localeEncodingMappings =
-        new HashMap<String,String>();
-    public void addLocaleEncodingMapping(String locale, String encoding) {
-        localeEncodingMappings.put(locale, encoding);
-    }
-    public Map<String,String> getLocalEncodingMappings() {
-        return localeEncodingMappings;
-    }
-    
-
-    // Attributes not defined in web.xml or web-fragment.xml
-    
-    // URL of JAR / exploded JAR for this web-fragment
-    private URL uRL = null;
-    public void setURL(URL url) { this.uRL = url; }
-    public URL getURL() { return uRL; }
-
-
-    @Override
-    public String toString() {
-        StringBuilder buf = new StringBuilder(32);
-        buf.append("Name: ");
-        buf.append(getName());
-        buf.append(", URL: ");
-        buf.append(getURL());
-        return buf.toString();
-    }
-    
-    private static final String INDENT2 = "  ";
-    private static final String INDENT4 = "    ";
-    private static final String INDENT6 = "      ";
-    
-    /**
-     * Generate a web.xml in String form that matches the representation stored
-     * in this object.
-     * 
-     * @return The complete contents of web.xml as a String
-     */
-    public String toXml() {
-        StringBuilder sb = new StringBuilder(2048);
-        
-        // TODO - Various, icon, description etc elements are skipped - mainly
-        //        because they are ignored when web.xml is parsed - see above
-
-        // Declaration
-        sb.append("<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n");
-        
-        // Root element
-        sb.append("<web-app xmlns=\"http://java.sun.com/xml/ns/javaee\"\n");
-        sb.append("         xmlns:xsi=");
-        sb.append("\"http://www.w3.org/2001/XMLSchema-instance\"\n");
-        sb.append("         xsi:schemaLocation=");
-        sb.append("\"http://java.sun.com/xml/ns/javaee/web-app_3_0.xsd\"\n");
-        sb.append("         version=\"");
-        sb.append(getVersion());
-        sb.append("\"\n");
-        sb.append("         metadata-complete=\"true\">\n\n");
-
-        appendElement(sb, INDENT2, "display-name", displayName);
-        
-        if (isDistributable()) {
-            sb.append("  <distributable/>\n\n");
-        }
-        
-        for (Map.Entry<String, String> entry : contextParams.entrySet()) {
-            sb.append("  <context-param>\n");
-            appendElement(sb, INDENT4, "param-name", entry.getKey());
-            appendElement(sb, INDENT4, "param-valuee", entry.getValue());
-            sb.append("  </context-param>\n");
-        }
-        sb.append('\n');
-        
-        for (Map.Entry<String, FilterDef> entry : filters.entrySet()) {
-            FilterDef filterDef = entry.getValue();
-            sb.append("  <filter>\n");
-            appendElement(sb, INDENT4, "description",
-                    filterDef.getDescription());
-            appendElement(sb, INDENT4, "display-name",
-                    filterDef.getDisplayName());
-            appendElement(sb, INDENT4, "filter-name",
-                    filterDef.getFilterName());
-            appendElement(sb, INDENT4, "filter-class",
-                    filterDef.getFilterClass());
-            appendElement(sb, INDENT4, "async-supported",
-                    filterDef.getAsyncSupported());
-            for (Map.Entry<String, String> param :
-                    filterDef.getParameterMap().entrySet()) {
-                sb.append("    <init-param>\n");
-                appendElement(sb, INDENT6, "param-name", param.getKey());
-                appendElement(sb, INDENT6, "param-value", param.getValue());
-                sb.append("    </init-param>\n");
-            }
-            sb.append("  </filter>\n");
-        }
-        sb.append('\n');
-
-        for (FilterMap filterMap : filterMaps) {
-            sb.append("  <filter-mapping>\n");
-            appendElement(sb, INDENT4, "filter-name",
-                    filterMap.getFilterName());
-            if (filterMap.getMatchAllServletNames()) {
-                sb.append("    <servlet-name>*</servlet-name>\n");
-            } else {
-                for (String servletName : filterMap.getServletNames()) {
-                    appendElement(sb, INDENT4, "servlet-name", servletName);
-                }
-            }
-            if (filterMap.getMatchAllUrlPatterns()) {
-                sb.append("    <url-pattern>*</url-pattern>\n");
-            } else {
-                for (String urlPattern : filterMap.getURLPatterns()) {
-                    appendElement(sb, INDENT4, "url-pattern", urlPattern);
-                }
-            }
-            for (String dispatcher : filterMap.getDispatcherNames()) {
-                appendElement(sb, INDENT4, "dispatcher", dispatcher);
-            }
-            sb.append("  </filter-mapping>\n");
-        }
-        sb.append('\n');
-
-        for (String listener : listeners) {
-            sb.append("  <listener>\n");
-            appendElement(sb, INDENT4, "listener-class", listener);
-            sb.append("  </listener>\n");
-        }
-        sb.append('\n');
-
-        for (Map.Entry<String, ServletDef> entry : servlets.entrySet()) {
-            ServletDef servletDef = entry.getValue();
-            sb.append("  <servlet>\n");
-            appendElement(sb, INDENT4, "description",
-                    servletDef.getDescription());
-            appendElement(sb, INDENT4, "display-name",
-                    servletDef.getDisplayName());
-            appendElement(sb, INDENT4, "servlet-name", entry.getKey());
-            appendElement(sb, INDENT4, "servlet-class",
-                    servletDef.getServletClass());
-            appendElement(sb, INDENT4, "jsp-file", servletDef.getJspFile());
-            for (Map.Entry<String, String> param :
-                    servletDef.getParameterMap().entrySet()) {
-                sb.append("    <init-param>\n");
-                appendElement(sb, INDENT6, "param-name", param.getKey());
-                appendElement(sb, INDENT6, "param-value", param.getValue());
-                sb.append("    </init-param>\n");
-            }
-            appendElement(sb, INDENT4, "load-on-startup",
-                    servletDef.getLoadOnStartup());
-            appendElement(sb, INDENT4, "enabled", servletDef.getEnabled());
-            appendElement(sb, INDENT4, "async-supported",
-                    servletDef.getAsyncSupported());
-            if (servletDef.getRunAs() != null) {
-                sb.append("    <run-as>\n");
-                appendElement(sb, INDENT6, "role-name", servletDef.getRunAs());
-                sb.append("    </run-as>\n");
-            }
-            for (SecurityRoleRef roleRef : servletDef.getSecurityRoleRefs()) {
-                sb.append("    <security-role-ref>\n");
-                appendElement(sb, INDENT6, "role-name", roleRef.getName());
-                appendElement(sb, INDENT6, "role-link", roleRef.getLink());
-                sb.append("    </security-role-ref>\n");
-            }
-            MultipartDef multipartDef = servletDef.getMultipartDef();
-            if (multipartDef != null) {
-                sb.append("    <multipart-config>\n");
-                appendElement(sb, INDENT6, "location",
-                        multipartDef.getLocation());
-                appendElement(sb, INDENT6, "max-file-size",
-                        multipartDef.getMaxFileSize());
-                appendElement(sb, INDENT6, "max-request-size",
-                        multipartDef.getMaxRequestSize());
-                appendElement(sb, INDENT6, "file-size-threshold",
-                        multipartDef.getFileSizeThreshold());
-                sb.append("    </multipart-config>\n");
-            }
-            sb.append("  </servlet>\n");
-        }
-        sb.append('\n');
-
-        for (Map.Entry<String, String> entry : servletMappings.entrySet()) {
-            sb.append("  <servlet-mapping>\n");
-            appendElement(sb, INDENT4, "servlet-name", entry.getValue());
-            appendElement(sb, INDENT4, "url-pattern", entry.getKey());
-            sb.append("  </servlet-mapping>\n");
-        }
-        sb.append('\n');
-        
-        if (sessionConfig != null) {
-            sb.append("  <session-config>\n");
-            appendElement(sb, INDENT4, "session-timeout",
-                    sessionConfig.getSessionTimeout());
-            sb.append("    <cookie-config>\n");
-            appendElement(sb, INDENT6, "name", sessionConfig.getCookieName());
-            appendElement(sb, INDENT6, "domain",
-                    sessionConfig.getCookieDomain());
-            appendElement(sb, INDENT6, "path", sessionConfig.getCookiePath());
-            appendElement(sb, INDENT6, "comment",
-                    sessionConfig.getCookieComment());
-            appendElement(sb, INDENT6, "http-only",
-                    sessionConfig.getCookieHttpOnly());
-            appendElement(sb, INDENT6, "secure",
-                    sessionConfig.getCookieSecure());
-            appendElement(sb, INDENT6, "max-age",
-                    sessionConfig.getCookieMaxAge());
-            sb.append("    </cookie-config>\n");
-            for (SessionTrackingMode stm :
-                    sessionConfig.getSessionTrackingModes()) {
-                appendElement(sb, INDENT4, "tracking-mode", stm.name());
-            }
-            sb.append("  </session-config>\n\n");
-        }
-        
-        for (Map.Entry<String, String> entry : mimeMappings.entrySet()) {
-            sb.append("  <mime-mapping>\n");
-            appendElement(sb, INDENT4, "extension", entry.getKey());
-            appendElement(sb, INDENT4, "mime-type", entry.getValue());
-            sb.append("  </mime-mapping>\n");
-        }
-        sb.append('\n');
-        
-        if (welcomeFiles.size() > 0) {
-            sb.append("  <welcome-file-list>\n");
-            for (String welcomeFile : welcomeFiles) {
-                appendElement(sb, INDENT4, "welcome-file", welcomeFile);
-            }
-            sb.append("  </welcome-file-list>\n\n");
-        }
-        
-        for (ErrorPage errorPage : errorPages.values()) {
-            sb.append("  <error-page>\n");
-            if (errorPage.getExceptionType() == null) {
-                appendElement(sb, INDENT4, "error-code",
-                        Integer.toString(errorPage.getErrorCode()));
-            } else {
-                appendElement(sb, INDENT4, "exception-type",
-                        errorPage.getExceptionType());
-            }
-            appendElement(sb, INDENT4, "location", errorPage.getLocation());
-            sb.append("  </error-page>\n");
-        }
-        sb.append('\n');
-
-        if (taglibs.size() > 0 || jspPropertyGroups.size() > 0) {
-            sb.append("  <jsp-config>\n");
-            for (Map.Entry<String, String> entry : taglibs.entrySet()) {
-                sb.append("    <taglib>\n");
-                appendElement(sb, INDENT6, "taglib-uri", entry.getKey());
-                appendElement(sb, INDENT6, "taglib-location", entry.getValue());
-                sb.append("    </taglib>\n");
-            }
-            for (JspPropertyGroup jpg : jspPropertyGroups) {
-                sb.append("    <jsp-property-group>\n");
-                appendElement(sb, INDENT6, "url-pattern", jpg.getUrlPattern());
-                appendElement(sb, INDENT6, "el-ignored", jpg.getElIgnored());
-                appendElement(sb, INDENT6, "scripting-invalid",
-                        jpg.getScriptingInvalid());
-                appendElement(sb, INDENT6, "page-encoding",
-                        jpg.getPageEncoding());
-                for (String prelude : jpg.getIncludePreludes()) {
-                    appendElement(sb, INDENT6, "include-prelude", prelude);
-                }
-                for (String coda : jpg.getIncludeCodas()) {
-                    appendElement(sb, INDENT6, "include-coda", coda);
-                }
-                appendElement(sb, INDENT6, "is-xml", jpg.getIsXml());
-                appendElement(sb, INDENT6, "deferred-syntax-allowed-as-literal",
-                        jpg.getDeferredSyntax());
-                appendElement(sb, INDENT6, "trim-directive-whitespaces",
-                        jpg.getTrimWhitespace());
-                appendElement(sb, INDENT6, "default-content-type",
-                        jpg.getDefaultContentType());
-                appendElement(sb, INDENT6, "buffer", jpg.getBuffer());
-                appendElement(sb, INDENT6, "error-on-undeclared-namespace",
-                        jpg.getErrorOnUndeclaredNamespace());
-                sb.append("    </jsp-property-group>\n");
-            }
-            sb.append("  </jsp-config>\n\n");
-        }
-        
-        for (SecurityConstraint constraint : securityConstraints) {
-            sb.append("  <security-constraint>\n");
-            appendElement(sb, INDENT4, "display-name",
-                    constraint.getDisplayName());
-            for (SecurityCollection collection : constraint.findCollections()) {
-                sb.append("    <web-resource-collection>\n");
-                appendElement(sb, INDENT6, "web-resource-name",
-                        collection.getName());
-                appendElement(sb, INDENT6, "description",
-                        collection.getDescription());
-                for (String urlPattern : collection.findPatterns()) {
-                    appendElement(sb, INDENT6, "url-pattern", urlPattern);
-                }
-                for (String method : collection.findMethods()) {
-                    appendElement(sb, INDENT6, "http-method", method);
-                }
-                for (String method : collection.findOmittedMethods()) {
-                    appendElement(sb, INDENT6, "http-method-omission", method);
-                }
-                sb.append("    </web-resource-collection>\n");
-            }
-            if (constraint.findAuthRoles().length > 0) {
-                sb.append("    <auth-constraint>\n");
-                for (String role : constraint.findAuthRoles()) {
-                    appendElement(sb, INDENT6, "role-name", role);
-                }
-                sb.append("    </auth-constraint>\n");
-            }
-            if (constraint.getUserConstraint() != null) {
-                sb.append("    <user-data-constraint>\n");
-                appendElement(sb, INDENT6, "transport-guarantee",
-                        constraint.getUserConstraint());
-                sb.append("    </user-data-constraint>\n");
-            }
-            sb.append("  </security-constraint>\n");
-        }
-        sb.append('\n');
-
-        if (loginConfig != null) {
-            sb.append("  <login-config>\n");
-            appendElement(sb, INDENT4, "auth-method",
-                    loginConfig.getAuthMethod());
-            appendElement(sb,INDENT4, "realm-name",
-                    loginConfig.getRealmName());
-            if (loginConfig.getErrorPage() != null ||
-                        loginConfig.getLoginPage() != null) {
-                sb.append("    <form-login-config>\n");
-                appendElement(sb, INDENT6, "form-login-page",
-                        loginConfig.getLoginPage());
-                appendElement(sb, INDENT6, "form-error-page",
-                        loginConfig.getErrorPage());
-                sb.append("    </form-login-config>\n");
-            }
-            sb.append("  </login-config>\n\n");
-        }
-        
-        for (String roleName : securityRoles) {
-            sb.append("  <security-role>\n");
-            appendElement(sb, INDENT4, "role-name", roleName);
-            sb.append("  </security-role>\n");
-        }
-        
-        for (ContextEnvironment envEntry : envEntries.values()) {
-            sb.append("  <env-entry>\n");
-            appendElement(sb, INDENT4, "description",
-                    envEntry.getDescription());
-            appendElement(sb, INDENT4, "env-entry-name", envEntry.getName());
-            appendElement(sb, INDENT4, "env-entry-type", envEntry.getType());
-            appendElement(sb, INDENT4, "env-entry-value", envEntry.getValue());
-            // TODO mapped-name
-            for (InjectionTarget target : envEntry.getInjectionTargets()) {
-                sb.append("    <injection-target>\n");
-                appendElement(sb, INDENT6, "injection-target-class",
-                        target.getTargetClass());
-                appendElement(sb, INDENT6, "injection-target-name",
-                        target.getTargetName());
-                sb.append("    </injection-target>\n");
-            }
-            // TODO lookup-name
-            sb.append("  </env-entry>\n");
-        }
-        sb.append('\n');
-
-        for (ContextEjb ejbRef : ejbRefs.values()) {
-            sb.append("  <ejb-ref>\n");
-            appendElement(sb, INDENT4, "description", ejbRef.getDescription());
-            appendElement(sb, INDENT4, "ejb-ref-name", ejbRef.getName());
-            appendElement(sb, INDENT4, "ejb-ref-type", ejbRef.getType());
-            appendElement(sb, INDENT4, "home", ejbRef.getHome());
-            appendElement(sb, INDENT4, "remote", ejbRef.getRemote());
-            appendElement(sb, INDENT4, "ejb-link", ejbRef.getLink());
-            // TODO mapped-name
-            for (InjectionTarget target : ejbRef.getInjectionTargets()) {
-                sb.append("    <injection-target>\n");
-                appendElement(sb, INDENT6, "injection-target-class",
-                        target.getTargetClass());
-                appendElement(sb, INDENT6, "injection-target-name",
-                        target.getTargetName());
-                sb.append("    </injection-target>\n");
-            }
-            // TODO lookup-name
-            sb.append("  </ejb-ref>\n");
-        }
-        sb.append('\n');
-
-        for (ContextLocalEjb ejbLocalRef : ejbLocalRefs.values()) {
-            sb.append("  <ejb-local-ref>\n");
-            appendElement(sb, INDENT4, "description",
-                    ejbLocalRef.getDescription());
-            appendElement(sb, INDENT4, "ejb-ref-name", ejbLocalRef.getName());
-            appendElement(sb, INDENT4, "ejb-ref-type", ejbLocalRef.getType());
-            appendElement(sb, INDENT4, "local-home", ejbLocalRef.getHome());
-            appendElement(sb, INDENT4, "local", ejbLocalRef.getLocal());
-            appendElement(sb, INDENT4, "ejb-link", ejbLocalRef.getLink());
-            // TODO mapped-name
-            for (InjectionTarget target : ejbLocalRef.getInjectionTargets()) {
-                sb.append("    <injection-target>\n");
-                appendElement(sb, INDENT6, "injection-target-class",
-                        target.getTargetClass());
-                appendElement(sb, INDENT6, "injection-target-name",
-                        target.getTargetName());
-                sb.append("    </injection-target>\n");
-            }
-            // TODO lookup-name
-            sb.append("  </ejb-local-ref>\n");
-        }
-        sb.append('\n');
-        
-        for (ContextService serviceRef : serviceRefs.values()) {
-            sb.append("  <service-ref>\n");
-            appendElement(sb, INDENT4, "description",
-                    serviceRef.getDescription());
-            appendElement(sb, INDENT4, "display-name",
-                    serviceRef.getDisplayname());
-            appendElement(sb, INDENT4, "service-ref-name",
-                    serviceRef.getName());
-            appendElement(sb, INDENT4, "service-interface",
-                    serviceRef.getInterface());
-            appendElement(sb, INDENT4, "service-ref-type",
-                    serviceRef.getType());
-            appendElement(sb, INDENT4, "wsdl-file", serviceRef.getWsdlfile());
-            appendElement(sb, INDENT4, "jaxrpc-mapping-file",
-                    serviceRef.getJaxrpcmappingfile());
-            String qname = serviceRef.getServiceqnameNamespaceURI();
-            if (qname != null) {
-                qname = qname + ":";
-            }
-            qname = qname + serviceRef.getServiceqnameLocalpart();
-            appendElement(sb, INDENT4, "service-qname", qname);
-            Iterator<String> endpointIter = serviceRef.getServiceendpoints();
-            while (endpointIter.hasNext()) {
-                String endpoint = endpointIter.next();
-                sb.append("    <port-component-ref>\n");
-                appendElement(sb, INDENT6, "service-endpoint-interface",
-                        endpoint);
-                appendElement(sb, INDENT6, "port-component-link",
-                        serviceRef.getProperty(endpoint));
-                sb.append("    </port-component-ref>\n");
-            }
-            Iterator<String> handlerIter = serviceRef.getHandlers();
-            while (handlerIter.hasNext()) {
-                String handler = handlerIter.next();
-                sb.append("    <handler>\n");
-                ContextHandler ch = serviceRef.getHandler(handler);
-                appendElement(sb, INDENT6, "handler-name", ch.getName());
-                appendElement(sb, INDENT6, "handler-class",
-                        ch.getHandlerclass());
-                sb.append("    </handler>\n");
-            }
-            // TODO handler-chains
-            // TODO mapped-name
-            for (InjectionTarget target : serviceRef.getInjectionTargets()) {
-                sb.append("    <injection-target>\n");
-                appendElement(sb, INDENT6, "injection-target-class",
-                        target.getTargetClass());
-                appendElement(sb, INDENT6, "injection-target-name",
-                        target.getTargetName());
-                sb.append("    </injection-target>\n");
-            }
-            // TODO lookup-name
-            sb.append("  </service-ref>\n");
-        }
-        sb.append('\n');
-        
-        for (ContextResource resourceRef : resourceRefs.values()) {
-            sb.append("  <resource-ref>\n");
-            appendElement(sb, INDENT4, "description",
-                    resourceRef.getDescription());
-            appendElement(sb, INDENT4, "res-ref-name", resourceRef.getName());
-            appendElement(sb, INDENT4, "res-type", resourceRef.getType());
-            appendElement(sb, INDENT4, "res-auth", resourceRef.getAuth());
-            appendElement(sb, INDENT4, "res-sharing-scope",
-                    resourceRef.getScope());
-            // TODO mapped-name
-            for (InjectionTarget target : resourceRef.getInjectionTargets()) {
-                sb.append("    <injection-target>\n");
-                appendElement(sb, INDENT6, "injection-target-class",
-                        target.getTargetClass());
-                appendElement(sb, INDENT6, "injection-target-name",
-                        target.getTargetName());
-                sb.append("    </injection-target>\n");
-            }
-            // TODO lookup-name
-            sb.append("  </resource-ref>\n");
-        }
-        sb.append('\n');
-
-        for (ContextResourceEnvRef resourceEnvRef : resourceEnvRefs.values()) {
-            sb.append("  <resource-env-ref>\n");
-            appendElement(sb, INDENT4, "description",
-                    resourceEnvRef.getDescription());
-            appendElement(sb, INDENT4, "resource-env-ref-name",
-                    resourceEnvRef.getName());
-            appendElement(sb, INDENT4, "resource-env-ref-type",
-                    resourceEnvRef.getType());
-            // TODO mapped-name
-            for (InjectionTarget target :
-                    resourceEnvRef.getInjectionTargets()) {
-                sb.append("    <injection-target>\n");
-                appendElement(sb, INDENT6, "injection-target-class",
-                        target.getTargetClass());
-                appendElement(sb, INDENT6, "injection-target-name",
-                        target.getTargetName());
-                sb.append("    </injection-target>\n");
-            }
-            // TODO lookup-name
-            sb.append("  </resource-env-ref>\n");
-        }
-        sb.append('\n');
-
-        for (MessageDestinationRef mdr : messageDestinationRefs.values()) {
-            sb.append("  <message-destination-ref>\n");
-            appendElement(sb, INDENT4, "description", mdr.getDescription());
-            appendElement(sb, INDENT4, "message-destination-ref-name",
-                    mdr.getName());
-            appendElement(sb, INDENT4, "message-destination-type",
-                    mdr.getType());
-            appendElement(sb, INDENT4, "message-destination-usage",
-                    mdr.getUsage());
-            appendElement(sb, INDENT4, "message-destination-link",
-                    mdr.getLink());
-            // TODO mapped-name
-            for (InjectionTarget target : mdr.getInjectionTargets()) {
-                sb.append("    <injection-target>\n");
-                appendElement(sb, INDENT6, "injection-target-class",
-                        target.getTargetClass());
-                appendElement(sb, INDENT6, "injection-target-name",
-                        target.getTargetName());
-                sb.append("    </injection-target>\n");
-            }
-            // TODO lookup-name
-            sb.append("  </message-destination-ref>\n");
-        }
-        sb.append('\n');
-
-        for (MessageDestination md : messageDestinations.values()) {
-            sb.append("  <message-destination>\n");
-            appendElement(sb, INDENT4, "description", md.getDescription());
-            appendElement(sb, INDENT4, "display-name", md.getDisplayName());
-            appendElement(sb, INDENT4, "message-destination-name",
-                    md.getName());
-            // TODO mapped-name
-            sb.append("  </message-destination>\n");
-        }
-        sb.append('\n');
-
-        if (localeEncodingMappings.size() > 0) {
-            sb.append("  <locale-encoding-mapping-list>\n");
-            for (Map.Entry<String, String> entry :
-                    localeEncodingMappings.entrySet()) {
-                sb.append("    <locale-encoding-mapping>\n");
-                appendElement(sb, INDENT6, "locale", entry.getKey());
-                appendElement(sb, INDENT6, "encoding", entry.getValue());
-                sb.append("    </locale-encoding-mapping>\n");
-            }
-            sb.append("  </locale-encoding-mapping-list>\n");
-        }
-        sb.append("</web-app>");
-        return sb.toString();
-    }
-
-    private static void appendElement(StringBuilder sb, String indent,
-            String elementName, String value) {
-        if (value == null || value.length() == 0) return;
-        sb.append(indent);
-        sb.append('<');
-        sb.append(elementName);
-        sb.append('>');
-        sb.append(escapeXml(value));
-        sb.append("</");
-        sb.append(elementName);
-        sb.append(">\n");
-    }
-
-    private static void appendElement(StringBuilder sb, String indent,
-            String elementName, Object value) {
-        if (value == null) return;
-        appendElement(sb, indent, elementName, value.toString());
-    }
-
-
-    /**
-     * Escape the 5 entities defined by XML.
-     */
-    private static String escapeXml(String s) {
-        if (s == null)
-            return null;
-        StringBuilder sb = new StringBuilder();
-        for (int i = 0; i < s.length(); i++) {
-            char c = s.charAt(i);
-            if (c == '<') {
-                sb.append("&lt;");
-            } else if (c == '>') {
-                sb.append("&gt;");
-            } else if (c == '\'') {
-                sb.append("&apos;");
-            } else if (c == '&') {
-                sb.append("&amp;");
-            } else if (c == '"') {
-                sb.append("&quot;");
-            } else {
-                sb.append(c);
-            }
-        }
-        return sb.toString();
-    }
-
-
-    /**
-     * Configure a {@link Context} using the stored web.xml representation.
-     *  
-     * @param context   The context to be configured
-     */
-    public void configureContext(Context context) {
-        // As far as possible, process in alphabetical order so it is easy to
-        // check everything is present
-        // Some validation depends on correct public ID
-        context.setPublicId(publicId);
-
-        // Everything else in order
-        context.setEffectiveMajorVersion(getMajorVersion());
-        context.setEffectiveMinorVersion(getMinorVersion());
-        
-        for (Entry<String, String> entry : contextParams.entrySet()) {
-            context.addParameter(entry.getKey(), entry.getValue());
-        }
-        context.setDisplayName(displayName);
-        context.setDistributable(distributable);
-        for (ContextLocalEjb ejbLocalRef : ejbLocalRefs.values()) {
-            context.getNamingResources().addLocalEjb(ejbLocalRef);
-        }
-        for (ContextEjb ejbRef : ejbRefs.values()) {
-            context.getNamingResources().addEjb(ejbRef);
-        }
-        for (ContextEnvironment environment : envEntries.values()) {
-            context.getNamingResources().addEnvironment(environment);
-        }
-        for (ErrorPage errorPage : errorPages.values()) {
-            context.addErrorPage(errorPage);
-        }
-        for (FilterDef filter : filters.values()) {
-            if (filter.getAsyncSupported() == null) {
-                filter.setAsyncSupported("false");
-            }
-            context.addFilterDef(filter);
-        }
-        for (FilterMap filterMap : filterMaps) {
-            context.addFilterMap(filterMap);
-        }
-        for (JspPropertyGroup jspPropertyGroup : jspPropertyGroups) {
-            JspPropertyGroupDescriptor descriptor =
-                new ApplicationJspPropertyGroupDescriptor(jspPropertyGroup);
-            context.getJspConfigDescriptor().getJspPropertyGroups().add(
-                    descriptor);
-        }
-        for (String listener : listeners) {
-            context.addApplicationListener(listener);
-        }
-        for (Entry<String, String> entry : localeEncodingMappings.entrySet()) {
-            context.addLocaleEncodingMappingParameter(entry.getKey(),
-                    entry.getValue());
-        }
-        // Prevents IAE
-        if (loginConfig != null) {
-            context.setLoginConfig(loginConfig);
-        }
-        for (MessageDestinationRef mdr : messageDestinationRefs.values()) {
-            context.getNamingResources().addMessageDestinationRef(mdr);
-        }
-
-        // messageDestinations were ignored in Tomcat 6, so ignore here
-        
-        context.setIgnoreAnnotations(metadataComplete);
-        for (Entry<String, String> entry : mimeMappings.entrySet()) {
-            context.addMimeMapping(entry.getKey(), entry.getValue());
-        }
-        // Name is just used for ordering
-        for (ContextResourceEnvRef resource : resourceEnvRefs.values()) {
-            context.getNamingResources().addResourceEnvRef(resource);
-        }
-        for (ContextResource resource : resourceRefs.values()) {
-            context.getNamingResources().addResource(resource);
-        }
-        for (SecurityConstraint constraint : securityConstraints) {
-            context.addConstraint(constraint);
-        }
-        for (String role : securityRoles) {
-            context.addSecurityRole(role);
-        }
-        for (ContextService service : serviceRefs.values()) {
-            context.getNamingResources().addService(service);
-        }
-        for (ServletDef servlet : servlets.values()) {
-            Wrapper wrapper = context.createWrapper();
-            // Description is ignored
-            // Display name is ignored
-            // Icons are ignored
-            
-            // jsp-file gets passed to the JSP Servlet as an init-param
-
-            if (servlet.getLoadOnStartup() != null) {
-                wrapper.setLoadOnStartup(servlet.getLoadOnStartup().intValue());
-            }
-            if (servlet.getEnabled() != null) {
-                wrapper.setEnabled(servlet.getEnabled().booleanValue());
-            }
-            wrapper.setName(servlet.getServletName());
-            Map<String,String> params = servlet.getParameterMap(); 
-            for (Entry<String, String> entry : params.entrySet()) {
-                wrapper.addInitParameter(entry.getKey(), entry.getValue());
-            }
-            wrapper.setRunAs(servlet.getRunAs());
-            Set<SecurityRoleRef> roleRefs = servlet.getSecurityRoleRefs();
-            for (SecurityRoleRef roleRef : roleRefs) {
-                wrapper.addSecurityReference(
-                        roleRef.getName(), roleRef.getLink());
-            }
-            wrapper.setServletClass(servlet.getServletClass());
-            MultipartDef multipartdef = servlet.getMultipartDef();
-            if (multipartdef != null) {
-                if (multipartdef.getMaxFileSize() != null &&
-                        multipartdef.getMaxRequestSize()!= null &&
-                        multipartdef.getFileSizeThreshold() != null) {
-                    wrapper.setMultipartConfigElement(new MultipartConfigElement(
-                            multipartdef.getLocation(),
-                            Long.parseLong(multipartdef.getMaxFileSize()),
-                            Long.parseLong(multipartdef.getMaxRequestSize()),
-                            Integer.parseInt(
-                                    multipartdef.getFileSizeThreshold())));
-                } else {
-                    wrapper.setMultipartConfigElement(new MultipartConfigElement(
-                            multipartdef.getLocation()));
-                }
-            }
-            if (servlet.getAsyncSupported() != null) {
-                wrapper.setAsyncSupported(
-                        servlet.getAsyncSupported().booleanValue());
-            }
-            wrapper.setOverridable(servlet.isOverridable());
-            context.addChild(wrapper);
-        }
-        for (Entry<String, String> entry : servletMappings.entrySet()) {
-            context.addServletMapping(entry.getKey(), entry.getValue());
-        }
-        if (sessionConfig != null) {
-            if (sessionConfig.getSessionTimeout() != null) {
-                context.setSessionTimeout(
-                        sessionConfig.getSessionTimeout().intValue());
-            }
-            SessionCookieConfig scc =
-                context.getServletContext().getSessionCookieConfig();
-            scc.setName(sessionConfig.getCookieName());
-            scc.setDomain(sessionConfig.getCookieDomain());
-            scc.setPath(sessionConfig.getCookiePath());
-            scc.setComment(sessionConfig.getCookieComment());
-            if (sessionConfig.getCookieHttpOnly() != null) {
-                scc.setHttpOnly(sessionConfig.getCookieHttpOnly().booleanValue());
-            }
-            if (sessionConfig.getCookieSecure() != null) {
-                scc.setSecure(sessionConfig.getCookieSecure().booleanValue());
-            }
-            if (sessionConfig.getCookieMaxAge() != null) {
-                scc.setMaxAge(sessionConfig.getCookieMaxAge().intValue());
-            }
-            if (sessionConfig.getSessionTrackingModes().size() > 0) {
-                context.getServletContext().setSessionTrackingModes(
-                        sessionConfig.getSessionTrackingModes());
-            }
-        }
-        for (Entry<String, String> entry : taglibs.entrySet()) {
-            TaglibDescriptor descriptor = new ApplicationTaglibDescriptor(
-                    entry.getValue(), entry.getKey());
-            context.getJspConfigDescriptor().getTaglibs().add(descriptor);
-        }
-        
-        // Context doesn't use version directly
-        
-        for (String welcomeFile : welcomeFiles) {
-            /*
-             * The following will result in a welcome file of "" so don't add
-             * that to the context 
-             * <welcome-file-list>
-             *   <welcome-file/>
-             * </welcome-file-list>
-             */
-            if (welcomeFile != null && welcomeFile.length() > 0) {
-                context.addWelcomeFile(welcomeFile);
-            }
-        }
-
-        // Do this last as it depends on servlets
-        for (JspPropertyGroup jspPropertyGroup : jspPropertyGroups) {
-            String jspServletName = context.findServletMapping("*.jsp");
-            if (jspServletName == null) {
-                jspServletName = "jsp";
-            }
-            if (context.findChild(jspServletName) != null) {
-                context.addServletMapping(jspPropertyGroup.getUrlPattern(),
-                        jspServletName, true);
-            } else {
-                if(log.isDebugEnabled())
-                    log.debug("Skiping " + jspPropertyGroup.getUrlPattern() +
-                            " , no servlet " + jspServletName);
-            }
-        }
-    }
-    
-    /**
-     * Merge the supplied web fragments into this main web.xml.
-     * 
-     * @param fragments     The fragments to merge in
-     * @return <code>true</code> if merge is successful, else
-     *         <code>false</code>
-     */
-    public boolean merge(Set<WebXml> fragments) {
-        // As far as possible, process in alphabetical order so it is easy to
-        // check everything is present
-        
-        // Merge rules vary from element to element. See SRV.8.2.3
-
-        WebXml temp = new WebXml();
-        Map<String,Boolean> mergeInjectionFlags =
-            new HashMap<String, Boolean>();
-
-        for (WebXml fragment : fragments) {
-            if (!mergeMap(fragment.getContextParams(), contextParams,
-                    temp.getContextParams(), fragment, "Context Parameter")) {
-                return false;
-            }
-        }
-        contextParams.putAll(temp.getContextParams());
-
-        if (displayName == null) {
-            for (WebXml fragment : fragments) {
-                String value = fragment.getDisplayName(); 
-                if (value != null) {
-                    if (temp.getDisplayName() == null) {
-                        temp.setDisplayName(value);
-                    } else {
-                        log.error(sm.getString(
-                                "webXml.mergeConflictDisplayName",
-                                fragment.getName(),
-                                fragment.getURL()));
-                        return false;
-                    }
-                }
-            }
-            displayName = temp.getDisplayName();
-        }
-
-        if (distributable) {
-            for (WebXml fragment : fragments) {
-                if (!fragment.isDistributable()) {
-                    distributable = false;
-                    break;
-                }
-            }
-        }
-
-        for (WebXml fragment : fragments) {
-            if (!mergeResourceMap(fragment.getEjbLocalRefs(), ejbLocalRefs,
-                    temp.getEjbLocalRefs(), mergeInjectionFlags, fragment)) {
-                return false;
-            }
-        }
-        ejbLocalRefs.putAll(temp.getEjbLocalRefs());
-        mergeInjectionFlags.clear();
-
-        for (WebXml fragment : fragments) {
-            if (!mergeResourceMap(fragment.getEjbRefs(), ejbRefs,
-                    temp.getEjbRefs(), mergeInjectionFlags, fragment)) {
-                return false;
-            }
-        }
-        ejbRefs.putAll(temp.getEjbRefs());
-        mergeInjectionFlags.clear();
-
-        for (WebXml fragment : fragments) {
-            if (!mergeResourceMap(fragment.getEnvEntries(), envEntries,
-                    temp.getEnvEntries(), mergeInjectionFlags, fragment)) {
-                return false;
-            }
-        }
-        envEntries.putAll(temp.getEnvEntries());
-        mergeInjectionFlags.clear();
-
-        for (WebXml fragment : fragments) {
-            if (!mergeMap(fragment.getErrorPages(), errorPages,
-                    temp.getErrorPages(), fragment, "Error Page")) {
-                return false;
-            }
-        }
-        errorPages.putAll(temp.getErrorPages());
-
-        // As per 'clarification' from the Servlet EG, filter definitions in the
-        // main web.xml override those in fragments and those in fragments
-        // override those in annotations
-        List<FilterMap> filterMapsToAdd = new ArrayList<FilterMap>();
-        for (WebXml fragment : fragments) {
-            for (FilterMap filterMap : fragment.getFilterMappings()) {
-                if (!filterMappingNames.contains(filterMap.getFilterName())) {
-                    filterMapsToAdd.add(filterMap);
-                }
-            }
-        }
-        for (FilterMap filterMap : filterMapsToAdd) {
-            // Additive
-            addFilterMapping(filterMap);
-        }
-
-        for (WebXml fragment : fragments) {
-            for (Map.Entry<String,FilterDef> entry :
-                    fragment.getFilters().entrySet()) {
-                if (filters.containsKey(entry.getKey())) {
-                    mergeFilter(entry.getValue(),
-                            filters.get(entry.getKey()), false);
-                } else {
-                    if (temp.getFilters().containsKey(entry.getKey())) {
-                        if (!(mergeFilter(entry.getValue(),
-                                temp.getFilters().get(entry.getKey()), true))) {
-                            log.error(sm.getString(
-                                    "webXml.mergeConflictFilter",
-                                    entry.getKey(),
-                                    fragment.getName(),
-                                    fragment.getURL()));
-    
-                            return false;
-                        }
-                    } else {
-                        temp.getFilters().put(entry.getKey(), entry.getValue());
-                    }
-                }
-            }
-        }
-        filters.putAll(temp.getFilters());
-
-        for (WebXml fragment : fragments) {
-            for (JspPropertyGroup jspPropertyGroup :
-                    fragment.getJspPropertyGroups()) {
-                // Always additive
-                addJspPropertyGroup(jspPropertyGroup);
-            }
-        }
-
-        for (WebXml fragment : fragments) {
-            for (String listener : fragment.getListeners()) {
-                // Always additive
-                addListener(listener);
-            }
-        }
-
-        for (WebXml fragment : fragments) {
-            if (!mergeMap(fragment.getLocalEncodingMappings(),
-                    localeEncodingMappings, temp.getLocalEncodingMappings(),
-                    fragment, "Locale Encoding Mapping")) {
-                return false;
-            }
-        }
-        localeEncodingMappings.putAll(temp.getLocalEncodingMappings());
-
-        if (getLoginConfig() == null) {
-            LoginConfig tempLoginConfig = null;
-            for (WebXml fragment : fragments) {
-                LoginConfig fragmentLoginConfig = fragment.loginConfig;
-                if (fragmentLoginConfig != null) {
-                    if (tempLoginConfig == null ||
-                            fragmentLoginConfig.equals(tempLoginConfig)) {
-                        tempLoginConfig = fragmentLoginConfig;
-                    } else {
-                        log.error(sm.getString(
-                                "webXml.mergeConflictLoginConfig",
-                                fragment.getName(),
-                                fragment.getURL()));
-                    }
-                }
-            }
-            loginConfig = tempLoginConfig;
-        }
-
-        for (WebXml fragment : fragments) {
-            if (!mergeResourceMap(fragment.getMessageDestinationRefs(), messageDestinationRefs,
-                    temp.getMessageDestinationRefs(), mergeInjectionFlags, fragment)) {
-                return false;
-            }
-        }
-        messageDestinationRefs.putAll(temp.getMessageDestinationRefs());
-        mergeInjectionFlags.clear();
-
-        for (WebXml fragment : fragments) {
-            if (!mergeResourceMap(fragment.getMessageDestinations(), messageDestinations,
-                    temp.getMessageDestinations(), mergeInjectionFlags, fragment)) {
-                return false;
-            }
-        }
-        messageDestinations.putAll(temp.getMessageDestinations());
-        mergeInjectionFlags.clear();
-
-        for (WebXml fragment : fragments) {
-            if (!mergeMap(fragment.getMimeMappings(), mimeMappings,
-                    temp.getMimeMappings(), fragment, "Mime Mapping")) {
-                return false;
-            }
-        }
-        mimeMappings.putAll(temp.getMimeMappings());
-
-        for (WebXml fragment : fragments) {
-            if (!mergeResourceMap(fragment.getResourceEnvRefs(), resourceEnvRefs,
-                    temp.getResourceEnvRefs(), mergeInjectionFlags, fragment)) {
-                return false;
-            }
-        }
-        resourceEnvRefs.putAll(temp.getResourceEnvRefs());
-        mergeInjectionFlags.clear();
-
-        for (WebXml fragment : fragments) {
-            if (!mergeResourceMap(fragment.getResourceRefs(), resourceRefs,
-                    temp.getResourceRefs(), mergeInjectionFlags, fragment)) {
-                return false;
-            }
-        }
-        resourceRefs.putAll(temp.getResourceRefs());
-        mergeInjectionFlags.clear();
-
-        for (WebXml fragment : fragments) {
-            for (SecurityConstraint constraint : fragment.getSecurityConstraints()) {
-                // Always additive
-                addSecurityConstraint(constraint);
-            }
-        }
-
-        for (WebXml fragment : fragments) {
-            for (String role : fragment.getSecurityRoles()) {
-                // Always additive
-                addSecurityRole(role);
-            }
-        }
-
-        for (WebXml fragment : fragments) {
-            if (!mergeResourceMap(fragment.getServiceRefs(), serviceRefs,
-                    temp.getServiceRefs(), mergeInjectionFlags, fragment)) {
-                return false;
-            }
-        }
-        serviceRefs.putAll(temp.getServiceRefs());
-        mergeInjectionFlags.clear();
-
-        // As per 'clarification' from the Servlet EG, servlet definitions and
-        // mappings in the main web.xml override those in fragments and those in
-        // fragments override those in annotations
-        // Skip servlet definitions and mappings from fragments that are
-        // defined in web.xml
-        List<Map.Entry<String,String>> servletMappingsToAdd =
-            new ArrayList<Map.Entry<String,String>>();
-        for (WebXml fragment : fragments) {
-            for (Map.Entry<String,String> servletMap :
-                    fragment.getServletMappings().entrySet()) {
-                if (!servletMappingNames.contains(servletMap.getValue()) &&
-                        !servletMappings.containsKey(servletMap.getKey())) {
-                    servletMappingsToAdd.add(servletMap);
-                }
-            }
-        }
-        
-        // Add fragment mappings
-        for (Map.Entry<String,String> mapping : servletMappingsToAdd) {
-            addServletMapping(mapping.getKey(), mapping.getValue());
-        }
-
-        for (WebXml fragment : fragments) {
-            for (Map.Entry<String,ServletDef> entry :
-                    fragment.getServlets().entrySet()) {
-                if (servlets.containsKey(entry.getKey())) {
-                    mergeServlet(entry.getValue(),
-                            servlets.get(entry.getKey()), false);
-                } else {
-                    if (temp.getServlets().containsKey(entry.getKey())) {
-                        if (!(mergeServlet(entry.getValue(),
-                                temp.getServlets().get(entry.getKey()), true))) {
-                            log.error(sm.getString(
-                                    "webXml.mergeConflictServlet",
-                                    entry.getKey(),
-                                    fragment.getName(),
-                                    fragment.getURL()));
-    
-                            return false;
-                        }
-                    } else {
-                        temp.getServlets().put(entry.getKey(), entry.getValue());
-                    }
-                }
-            }
-        }
-        servlets.putAll(temp.getServlets());
-        
-        if (sessionConfig.getSessionTimeout() == null) {
-            for (WebXml fragment : fragments) {
-                Integer value = fragment.getSessionConfig().getSessionTimeout();
-                if (value != null) {
-                    if (temp.getSessionConfig().getSessionTimeout() == null) {
-                        temp.getSessionConfig().setSessionTimeout(value.toString());
-                    } else if (value.equals(
-                            temp.getSessionConfig().getSessionTimeout())) {
-                        // Fragments use same value - no conflict
-                    } else {
-                        log.error(sm.getString(
-                                "webXml.mergeConflictSessionTimeout",
-                                fragment.getName(),
-                                fragment.getURL()));
-                        return false;
-                    }
-                }
-            }
-            if (temp.getSessionConfig().getSessionTimeout() != null) {
-                sessionConfig.setSessionTimeout(
-                        temp.getSessionConfig().getSessionTimeout().toString());
-            }
-        }
-        
-        if (sessionConfig.getCookieName() == null) {
-            for (WebXml fragment : fragments) {
-                String value = fragment.getSessionConfig().getCookieName();
-                if (value != null) {
-                    if (temp.getSessionConfig().getCookieName() == null) {
-                        temp.getSessionConfig().setCookieName(value);
-                    } else if (value.equals(
-                            temp.getSessionConfig().getCookieName())) {
-                        // Fragments use same value - no conflict
-                    } else {
-                        log.error(sm.getString(
-                                "webXml.mergeConflictSessionCookieName",
-                                fragment.getName(),
-                                fragment.getURL()));
-                        return false;
-                    }
-                }
-            }
-            sessionConfig.setCookieName(
-                    temp.getSessionConfig().getCookieName());
-        }
-        if (sessionConfig.getCookieDomain() == null) {
-            for (WebXml fragment : fragments) {
-                String value = fragment.getSessionConfig().getCookieDomain();
-                if (value != null) {
-                    if (temp.getSessionConfig().getCookieDomain() == null) {
-                        temp.getSessionConfig().setCookieDomain(value);
-                    } else if (value.equals(
-                            temp.getSessionConfig().getCookieDomain())) {
-                        // Fragments use same value - no conflict
-                    } else {
-                        log.error(sm.getString(
-                                "webXml.mergeConflictSessionCookieDomain",
-                                fragment.getName(),
-                                fragment.getURL()));
-                        return false;
-                    }
-                }
-            }
-            sessionConfig.setCookieDomain(
-                    temp.getSessionConfig().getCookieDomain());
-        }
-        if (sessionConfig.getCookiePath() == null) {
-            for (WebXml fragment : fragments) {
-                String value = fragment.getSessionConfig().getCookiePath();
-                if (value != null) {
-                    if (temp.getSessionConfig().getCookiePath() == null) {
-                        temp.getSessionConfig().setCookiePath(value);
-                    } else if (value.equals(
-                            temp.getSessionConfig().getCookiePath())) {
-                        // Fragments use same value - no conflict
-                    } else {
-                        log.error(sm.getString(
-                                "webXml.mergeConflictSessionCookiePath",
-                                fragment.getName(),
-                                fragment.getURL()));
-                        return false;
-                    }
-                }
-            }
-            sessionConfig.setCookiePath(
-                    temp.getSessionConfig().getCookiePath());
-        }
-        if (sessionConfig.getCookieComment() == null) {
-            for (WebXml fragment : fragments) {
-                String value = fragment.getSessionConfig().getCookieComment();
-                if (value != null) {
-                    if (temp.getSessionConfig().getCookieComment() == null) {
-                        temp.getSessionConfig().setCookieComment(value);
-                    } else if (value.equals(
-                            temp.getSessionConfig().getCookieComment())) {
-                        // Fragments use same value - no conflict
-                    } else {
-                        log.error(sm.getString(
-                                "webXml.mergeConflictSessionCookieComment",
-                                fragment.getName(),
-                                fragment.getURL()));
-                        return false;
-                    }
-                }
-            }
-            sessionConfig.setCookieComment(
-                    temp.getSessionConfig().getCookieComment());
-        }
-        if (sessionConfig.getCookieHttpOnly() == null) {
-            for (WebXml fragment : fragments) {
-                Boolean value = fragment.getSessionConfig().getCookieHttpOnly();
-                if (value != null) {
-                    if (temp.getSessionConfig().getCookieHttpOnly() == null) {
-                        temp.getSessionConfig().setCookieHttpOnly(value.toString());
-                    } else if (value.equals(
-                            temp.getSessionConfig().getCookieHttpOnly())) {
-                        // Fragments use same value - no conflict
-                    } else {
-                        log.error(sm.getString(
-                                "webXml.mergeConflictSessionCookieHttpOnly",
-                                fragment.getName(),
-                                fragment.getURL()));
-                        return false;
-                    }
-                }
-            }
-            if (temp.getSessionConfig().getCookieHttpOnly() != null) {
-                sessionConfig.setCookieHttpOnly(
-                        temp.getSessionConfig().getCookieHttpOnly().toString());
-            }
-        }
-        if (sessionConfig.getCookieSecure() == null) {
-            for (WebXml fragment : fragments) {
-                Boolean value = fragment.getSessionConfig().getCookieSecure();
-                if (value != null) {
-                    if (temp.getSessionConfig().getCookieSecure() == null) {
-                        temp.getSessionConfig().setCookieSecure(value.toString());
-                    } else if (value.equals(
-                            temp.getSessionConfig().getCookieSecure())) {
-                        // Fragments use same value - no conflict
-                    } else {
-                        log.error(sm.getString(
-                                "webXml.mergeConflictSessionCookieSecure",
-                                fragment.getName(),
-                                fragment.getURL()));
-                        return false;
-                    }
-                }
-            }
-            if (temp.getSessionConfig().getCookieSecure() != null) {
-                sessionConfig.setCookieSecure(
-                        temp.getSessionConfig().getCookieSecure().toString());
-            }
-        }
-        if (sessionConfig.getCookieMaxAge() == null) {
-            for (WebXml fragment : fragments) {
-                Integer value = fragment.getSessionConfig().getCookieMaxAge();
-                if (value != null) {
-                    if (temp.getSessionConfig().getCookieMaxAge() == null) {
-                        temp.getSessionConfig().setCookieMaxAge(value.toString());
-                    } else if (value.equals(
-                            temp.getSessionConfig().getCookieMaxAge())) {
-                        // Fragments use same value - no conflict
-                    } else {
-                        log.error(sm.getString(
-                                "webXml.mergeConflictSessionCookieMaxAge",
-                                fragment.getName(),
-                                fragment.getURL()));
-                        return false;
-                    }
-                }
-            }
-            if (temp.getSessionConfig().getCookieMaxAge() != null) {
-                sessionConfig.setCookieMaxAge(
-                        temp.getSessionConfig().getCookieMaxAge().toString());
-            }
-        }
-
-        if (sessionConfig.getSessionTrackingModes().size() == 0) {
-            for (WebXml fragment : fragments) {
-                EnumSet<SessionTrackingMode> value =
-                    fragment.getSessionConfig().getSessionTrackingModes();
-                if (value.size() > 0) {
-                    if (temp.getSessionConfig().getSessionTrackingModes().size() == 0) {
-                        temp.getSessionConfig().getSessionTrackingModes().addAll(value);
-                    } else if (value.equals(
-                            temp.getSessionConfig().getSessionTrackingModes())) {
-                        // Fragments use same value - no conflict
-                    } else {
-                        log.error(sm.getString(
-                                "webXml.mergeConflictSessionTrackingMode",
-                                fragment.getName(),
-                                fragment.getURL()));
-                        return false;
-                    }
-                }
-            }
-            sessionConfig.getSessionTrackingModes().addAll(
-                    temp.getSessionConfig().getSessionTrackingModes());
-        }
-        
-        for (WebXml fragment : fragments) {
-            if (!mergeMap(fragment.getTaglibs(), taglibs,
-                    temp.getTaglibs(), fragment, "Taglibs")) {
-                return false;
-            }
-        }
-        taglibs.putAll(temp.getTaglibs());
-
-        for (WebXml fragment : fragments) {
-            if (fragment.alwaysAddWelcomeFiles || welcomeFiles.size() == 0) {
-                for (String welcomeFile : fragment.getWelcomeFiles()) {
-                    addWelcomeFile(welcomeFile);
-                }
-            }
-        }
-
-        return true;
-    }
-    
-    private static <T extends ResourceBase> boolean mergeResourceMap(
-            Map<String, T> fragmentResources, Map<String, T> mainResources,
-            Map<String, T> tempResources,
-            Map<String,Boolean> mergeInjectionFlags, WebXml fragment) {
-        for (T resource : fragmentResources.values()) {
-            String resourceName = resource.getName();
-            boolean mergeInjectionFlag = false;
-            if (mainResources.containsKey(resourceName)) {
-                if (mergeInjectionFlags.containsKey(resourceName)) {
-                    mergeInjectionFlag =
-                        mergeInjectionFlags.get(resourceName).booleanValue(); 
-                } else {
-                    if (mainResources.get(
-                            resourceName).getInjectionTargets().size() == 0) {
-                        mergeInjectionFlag = true;
-                    }
-                    mergeInjectionFlags.put(resourceName,
-                            Boolean.valueOf(mergeInjectionFlag));
-                }
-                if (mergeInjectionFlag) {
-                    mainResources.get(resourceName).getInjectionTargets().addAll(
-                            resource.getInjectionTargets());
-                }
-            } else {
-                // Not defined in main web.xml
-                if (tempResources.containsKey(resourceName)) {
-                    log.error(sm.getString(
-                            "webXml.mergeConflictResource",
-                            resourceName,
-                            fragment.getName(),
-                            fragment.getURL()));
-                    return false;
-                } 
-                tempResources.put(resourceName, resource);
-            }
-        }
-        return true;
-    }
-    
-    private static <T> boolean mergeMap(Map<String,T> fragmentMap,
-            Map<String,T> mainMap, Map<String,T> tempMap, WebXml fragment,
-            String mapName) {
-        for (Entry<String, T> entry : fragmentMap.entrySet()) {
-            final String key = entry.getKey();
-            if (!mainMap.containsKey(key)) {
-                // Not defined in main web.xml
-                T value = entry.getValue();
-                if (tempMap.containsKey(key)) {
-                    if (value != null && !value.equals(
-                            tempMap.get(key))) {
-                        log.error(sm.getString(
-                                "webXml.mergeConflictString",
-                                mapName,
-                                key,
-                                fragment.getName(),
-                                fragment.getURL()));
-                        return false;
-                    }
-                } else {
-                    tempMap.put(key, value);
-                }
-            }
-        }
-        return true;
-    }
-    
-    private static boolean mergeFilter(FilterDef src, FilterDef dest,
-            boolean failOnConflict) {
-        if (dest.getAsyncSupported() == null) {
-            dest.setAsyncSupported(src.getAsyncSupported());
-        } else if (src.getAsyncSupported() != null) {
-            if (failOnConflict &&
-                    !src.getAsyncSupported().equals(dest.getAsyncSupported())) {
-                return false;
-            }
-        }
-
-        if (dest.getFilterClass()  == null) {
-            dest.setFilterClass(src.getFilterClass());
-        } else if (src.getFilterClass() != null) {
-            if (failOnConflict &&
-                    !src.getFilterClass().equals(dest.getFilterClass())) {
-                return false;
-            }
-        }
-        
-        for (Map.Entry<String,String> srcEntry :
-                src.getParameterMap().entrySet()) {
-            if (dest.getParameterMap().containsKey(srcEntry.getKey())) {
-                if (failOnConflict && !dest.getParameterMap().get(
-                        srcEntry.getKey()).equals(srcEntry.getValue())) {
-                    return false;
-                }
-            } else {
-                dest.addInitParameter(srcEntry.getKey(), srcEntry.getValue());
-            }
-        }
-        return true;
-    }
-    
-    private static boolean mergeServlet(ServletDef src, ServletDef dest,
-            boolean failOnConflict) {
-        // These tests should be unnecessary...
-        if (dest.getServletClass() != null && dest.getJspFile() != null) {
-            return false;
-        }
-        if (src.getServletClass() != null && src.getJspFile() != null) {
-            return false;
-        }
-        
-        
-        if (dest.getServletClass() == null && dest.getJspFile() == null) {
-            dest.setServletClass(src.getServletClass());
-            dest.setJspFile(src.getJspFile());
-        } else if (failOnConflict) {
-            if (src.getServletClass() != null &&
-                    (dest.getJspFile() != null ||
-                            !src.getServletClass().equals(dest.getServletClass()))) {
-                return false;
-            }

[... 2442 lines stripped ...]


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org