Refine ignored @PropertySource log warning

If the enclosing environment does not implement ConfigurableEnvironment,
then @PropertySource annotations are ignored because there is no way to
add them to the Environment. Now checking first to see if there are any
@PropertySource annotations present before issuing the warning.

Issue: SPR-8314

git-svn-id: https://src.springframework.org/svn/spring-framework/trunk@4295 50f2f4bb-b051-0410-bef5-90022cba6387

org.springframework.context/src/main/java/org/springframework/context/annotation/ConfigurationClassPostProcessor.java

@@ -247,15 +247,17 @@ public class ConfigurationClassPostProcessor implements BeanDefinitionRegistryPo
 		parser.validate();
 
 		// Handle any @PropertySource annotations
-		if (!(this.environment instanceof ConfigurableEnvironment)) {
-			logger.warn("Ignoring @PropertySource annotations. " +
-					"Reason: Environment must implement ConfigurableEnvironment");
-		}
-		else {
-			MutablePropertySources envPropertySources = ((ConfigurableEnvironment)this.environment).getPropertySources();
-			Stack<PropertySource<?>> parsedPropertySources = parser.getPropertySources();
-			while (!parsedPropertySources.isEmpty()) {
-				envPropertySources.addLast(parsedPropertySources.pop());
+		Stack<PropertySource<?>> parsedPropertySources = parser.getPropertySources();
+		if (!parsedPropertySources.isEmpty()) {
+			if (!(this.environment instanceof ConfigurableEnvironment)) {
+				logger.warn("Ignoring @PropertySource annotations. " +
+						"Reason: Environment must implement ConfigurableEnvironment");
+			}
+			else {
+				MutablePropertySources envPropertySources = ((ConfigurableEnvironment)this.environment).getPropertySources();
+				while (!parsedPropertySources.isEmpty()) {
+					envPropertySources.addLast(parsedPropertySources.pop());
+				}
 			}
 		}