You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tuscany.apache.org by sl...@apache.org on 2009/11/11 16:35:43 UTC

svn commit: r834913 - in /tuscany/java/sca/modules/assembly-xml/src/main: java/org/apache/tuscany/sca/policy/xml/IntentProcessor.java resources/org/apache/tuscany/sca/policy/xml/policy-xml-validation-messages.properties

Author: slaws
Date: Wed Nov 11 15:35:43 2009
New Revision: 834913

URL: http://svn.apache.org/viewvc?rev=834913&view=rev
Log:
Add test for non-unique intent qualifier - POL-3005

Modified:
    tuscany/java/sca/modules/assembly-xml/src/main/java/org/apache/tuscany/sca/policy/xml/IntentProcessor.java
    tuscany/java/sca/modules/assembly-xml/src/main/resources/org/apache/tuscany/sca/policy/xml/policy-xml-validation-messages.properties

Modified: tuscany/java/sca/modules/assembly-xml/src/main/java/org/apache/tuscany/sca/policy/xml/IntentProcessor.java
URL: http://svn.apache.org/viewvc/tuscany/java/sca/modules/assembly-xml/src/main/java/org/apache/tuscany/sca/policy/xml/IntentProcessor.java?rev=834913&r1=834912&r2=834913&view=diff
==============================================================================
--- tuscany/java/sca/modules/assembly-xml/src/main/java/org/apache/tuscany/sca/policy/xml/IntentProcessor.java (original)
+++ tuscany/java/sca/modules/assembly-xml/src/main/java/org/apache/tuscany/sca/policy/xml/IntentProcessor.java Wed Nov 11 15:35:43 2009
@@ -159,7 +159,19 @@
                                               intent.getName().toString());
                             }
                         }
-                        intent.getQualifiedIntents().add(qualified);
+                        
+                        // check that the qualifier is unique
+                        if ( !intent.getQualifiedIntents().contains(qualified)){
+                            intent.getQualifiedIntents().add(qualified);
+                        } else {
+                            Monitor.error(context.getMonitor(), 
+                                          this, 
+                                          Messages.RESOURCE_BUNDLE, 
+                                          "QualifierIsNotUnique", 
+                                          intent.getName().toString(),
+                                          qualifierName);
+                        }
+                        
                         qualified.setQualifiableIntent(intent);
                         current = qualified;
                     }

Modified: tuscany/java/sca/modules/assembly-xml/src/main/resources/org/apache/tuscany/sca/policy/xml/policy-xml-validation-messages.properties
URL: http://svn.apache.org/viewvc/tuscany/java/sca/modules/assembly-xml/src/main/resources/org/apache/tuscany/sca/policy/xml/policy-xml-validation-messages.properties?rev=834913&r1=834912&r2=834913&view=diff
==============================================================================
--- tuscany/java/sca/modules/assembly-xml/src/main/resources/org/apache/tuscany/sca/policy/xml/policy-xml-validation-messages.properties (original)
+++ tuscany/java/sca/modules/assembly-xml/src/main/resources/org/apache/tuscany/sca/policy/xml/policy-xml-validation-messages.properties Wed Nov 11 15:35:43 2009
@@ -43,4 +43,5 @@
 QualifierNameMissing = Required attribute "name" missing for qualifier definition in PolicySet : {0}
 ConstrainedTypeNotFound = Unable to find the extension type {0} constrained by intent: {1}
 MultipleDefaultQualifiers = [POL30004] Intent {0} has more than one qualifier marked as the default qualifier
+QualifierIsNotUnique = [POL30005] The intent {0} has more than one qualifier with the name {1}