fix VIDP configuration in combination with other OAs

1 files changed, 22 insertions, 17 deletions

diff --git a/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/validation/oa/OASTORKConfigValidation.java b/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/validation/oa/OASTORKConfigValidation.java
index 9e877605e..554aa9ff0 100644
--- a/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/validation/oa/OASTORKConfigValidation.java
+++ b/id/ConfigWebTool/src/main/java/at/gv/egovernment/moa/id/configuration/validation/oa/OASTORKConfigValidation.java
@@ -49,26 +49,31 @@ public class OASTORKConfigValidation {
 					new Object[] {qaa} ));
 		}
 		
-		for(AttributeProviderPlugin current : oageneral.getAttributeProviderPlugins()) {
-			if (MiscUtil.isEmpty(current.getUrl()) || !ValidationHelper.validateURL(current.getUrl())) {
-				log.info("AttributeProviderPlugin URL has no valid form.");
-				errors.add(LanguageHelper.getErrorString("validation.stork.ap.url.valid"));
-			}
-			if (MiscUtil.isEmpty(current.getName())) {
-				log.info("AttributeProviderPlugin Name is empty.");
-				errors.add(LanguageHelper.getErrorString("validation.stork.ap.name.empty"));
+		if (oageneral.isVidpEnabled()) {		
+			for(AttributeProviderPlugin current : oageneral.getAttributeProviderPlugins()) {
+				if (MiscUtil.isEmpty(current.getUrl()) || !ValidationHelper.validateURL(current.getUrl())) {
+					log.info("AttributeProviderPlugin URL has no valid form.");
+					errors.add(LanguageHelper.getErrorString("validation.stork.ap.url.valid"));
+				}
+				if (MiscUtil.isEmpty(current.getName())) {
+					log.info("AttributeProviderPlugin Name is empty.");
+					errors.add(LanguageHelper.getErrorString("validation.stork.ap.name.empty"));
+					
+				} else {
+					if (!oageneral.getAvailableAttributeProviderPlugins().contains(current.getName())) {
+						log.info("AttributeProviderPlugin Name is not supported.");
+						errors.add(LanguageHelper.getErrorString("validation.stork.ap.name.valid"));
+					}				
+				}
 				
-			} else {
-				if (!oageneral.getAvailableAttributeProviderPlugins().contains(current.getName())) {
-					log.info("AttributeProviderPlugin Name is not supported.");
-					errors.add(LanguageHelper.getErrorString("validation.stork.ap.name.valid"));
-				}				
+				if (MiscUtil.isEmpty(current.getAttributes()) || !current.getAttributes().matches("[a-zA-Z]+(, ?[a-zA-Z]+)*")) {
+					log.info("AttributeProviderPlugin attributes are empty or do not match csv format.");
+					errors.add(LanguageHelper.getErrorString("validation.stork.ap.attributes.valid"));
+				}
 			}
 			
-			if (MiscUtil.isEmpty(current.getAttributes()) || !current.getAttributes().matches("[a-zA-Z]+(, ?[a-zA-Z]+)*")) {
-				log.info("AttributeProviderPlugin attributes are empty or do not match csv format.");
-				errors.add(LanguageHelper.getErrorString("validation.stork.ap.attributes.valid"));
-			}
+		} else {
+			oageneral.setAttributeProviderPlugins(null);
 		}
 
 		return errors;