summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--sca-java-2.x/trunk/modules/builder/src/main/java/org/apache/tuscany/sca/builder/impl/ModelBuilderImpl.java3
1 files changed, 3 insertions, 0 deletions
diff --git a/sca-java-2.x/trunk/modules/builder/src/main/java/org/apache/tuscany/sca/builder/impl/ModelBuilderImpl.java b/sca-java-2.x/trunk/modules/builder/src/main/java/org/apache/tuscany/sca/builder/impl/ModelBuilderImpl.java
index 62a835d214..89fdc09ae1 100644
--- a/sca-java-2.x/trunk/modules/builder/src/main/java/org/apache/tuscany/sca/builder/impl/ModelBuilderImpl.java
+++ b/sca-java-2.x/trunk/modules/builder/src/main/java/org/apache/tuscany/sca/builder/impl/ModelBuilderImpl.java
@@ -103,6 +103,9 @@ public class ModelBuilderImpl implements CompositeBuilder {
// Apply policy external attachment. Happens before the composite type
// is created so that suitable promotion and structural processing is
// applied to the attached policies
+ // TODO - I don't think this is correct. The policy spec (4.12.1)implies that
+ // policy attachment happens independently of implementation or
+ // structural inheritance
composite = policyAttachmentBuilder.build(composite, context);
// Process the implementation hierarchy by calculating the component type