added setValidating to XmlBeanDefinitionReader itself as well (SPR-6336)

Author: jhoeller

org.springframework.beans/src/main/java/org/springframework/beans/factory/xml/XmlBeanDefinitionReader.java

@@ -135,15 +135,31 @@ public XmlBeanDefinitionReader(BeanDefinitionRegistry registry) {
 	}
 
 
+	/**
+	 * Set whether to use XML validation. Default is <code>true</code>.
+	 * <p>This method switches namespace awareness on if validation is turned off,
+	 * in order to still process schema namespaces properly in such a scenario.
+	 * @see #setValidationMode
+	 * @see #setNamespaceAware
+	 */
+	public void setValidating(boolean validating) {
+		this.validationMode = (validating ? VALIDATION_AUTO : VALIDATION_NONE);
+		this.namespaceAware = !validating;
+	}
+
 	/**
 	 * Set the validation mode to use by name. Defaults to {@link #VALIDATION_AUTO}.
+	 * @see #setValidationMode
 	 */
 	public void setValidationModeName(String validationModeName) {
 		setValidationMode(constants.asNumber(validationModeName).intValue());
 	}
 
 	/**
 	 * Set the validation mode to use. Defaults to {@link #VALIDATION_AUTO}.
+	 * <p>Note that this only activates or deactivates validation itself.
+	 * If you are switching validation off for schema files, you might need to
+	 * activate schema namespace support explicitly: see {@link #setNamespaceAware}.
 	 */
 	public void setValidationMode(int validationMode) {
 		this.validationMode = validationMode;
@@ -159,6 +175,9 @@ public int getValidationMode() {
 	/**
 	 * Set whether or not the XML parser should be XML namespace aware.
 	 * Default is "false".
+	 * <p>This is typically not needed when schema validation is active.
+	 * However, without validation, this has to be switched to "true"
+	 * in order to properly process schema namespaces.
 	 */
 	public void setNamespaceAware(boolean namespaceAware) {
 		this.namespaceAware = namespaceAware;

org.springframework.context/src/main/java/org/springframework/context/support/AbstractXmlApplicationContext.java

@@ -97,14 +97,11 @@ protected void loadBeanDefinitions(DefaultListableBeanFactory beanFactory) throw
 	 * definitions of this context. Default implementation is empty.
 	 * <p>Can be overridden in subclasses, e.g. for turning off XML validation
 	 * or using a different XmlBeanDefinitionParser implementation.
-	 * @param beanDefinitionReader the bean definition reader used by this context
+	 * @param reader the bean definition reader used by this context
 	 * @see org.springframework.beans.factory.xml.XmlBeanDefinitionReader#setDocumentReaderClass
 	 */
-	protected void initBeanDefinitionReader(XmlBeanDefinitionReader beanDefinitionReader) {
-		if (!this.validating) {
-			beanDefinitionReader.setValidationMode(XmlBeanDefinitionReader.VALIDATION_NONE);
-			beanDefinitionReader.setNamespaceAware(true);
-		}
+	protected void initBeanDefinitionReader(XmlBeanDefinitionReader reader) {
+		reader.setValidating(this.validating);
 	}
 
 	/**

org.springframework.context/src/main/java/org/springframework/context/support/GenericXmlApplicationContext.java

@@ -72,9 +72,7 @@ public GenericXmlApplicationContext(String... resourceLocations) {
 	 * Set whether to use XML validation. Default is <code>true</code>.
 	 */
 	public void setValidating(boolean validating) {
-		this.reader.setValidationMode(validating ?
-				XmlBeanDefinitionReader.VALIDATION_AUTO : XmlBeanDefinitionReader.VALIDATION_NONE);
-		this.reader.setNamespaceAware(!validating);
+		this.reader.setValidating(validating);
 	}