You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tuscany.apache.org by rf...@apache.org on 2010/07/16 19:49:02 UTC
svn commit: r964878 -
/tuscany/sca-java-2.x/trunk/modules/builder/src/main/java/org/apache/tuscany/sca/builder/impl/CompositePolicyBuilderImpl.java
Author: rfeng
Date: Fri Jul 16 17:49:02 2010
New Revision: 964878
URL: http://svn.apache.org/viewvc?rev=964878&view=rev
Log:
Remove the compile dependency on assembly-xml to fix build error
Modified:
tuscany/sca-java-2.x/trunk/modules/builder/src/main/java/org/apache/tuscany/sca/builder/impl/CompositePolicyBuilderImpl.java
Modified: tuscany/sca-java-2.x/trunk/modules/builder/src/main/java/org/apache/tuscany/sca/builder/impl/CompositePolicyBuilderImpl.java
URL: http://svn.apache.org/viewvc/tuscany/sca-java-2.x/trunk/modules/builder/src/main/java/org/apache/tuscany/sca/builder/impl/CompositePolicyBuilderImpl.java?rev=964878&r1=964877&r2=964878&view=diff
==============================================================================
--- tuscany/sca-java-2.x/trunk/modules/builder/src/main/java/org/apache/tuscany/sca/builder/impl/CompositePolicyBuilderImpl.java (original)
+++ tuscany/sca-java-2.x/trunk/modules/builder/src/main/java/org/apache/tuscany/sca/builder/impl/CompositePolicyBuilderImpl.java Fri Jul 16 17:49:02 2010
@@ -25,6 +25,7 @@ import java.util.Set;
import javax.xml.namespace.QName;
+import org.apache.tuscany.sca.assembly.Base;
import org.apache.tuscany.sca.assembly.Component;
import org.apache.tuscany.sca.assembly.ComponentReference;
import org.apache.tuscany.sca.assembly.ComponentService;
@@ -36,7 +37,6 @@ import org.apache.tuscany.sca.assembly.b
import org.apache.tuscany.sca.assembly.builder.CompositeBuilder;
import org.apache.tuscany.sca.assembly.builder.CompositeBuilderException;
import org.apache.tuscany.sca.assembly.builder.PolicyBuilder;
-import org.apache.tuscany.sca.assembly.xml.Constants;
import org.apache.tuscany.sca.core.ExtensionPointRegistry;
import org.apache.tuscany.sca.monitor.Monitor;
import org.apache.tuscany.sca.policy.Intent;
@@ -52,7 +52,7 @@ import org.apache.tuscany.sca.policy.uti
* @version $Rev$ $Date$
*/
public class CompositePolicyBuilderImpl extends ComponentPolicyBuilderImpl implements CompositeBuilder {
-
+ private final static QName NOLISTENER_INTENT = new QName(Base.SCA11_NS, "noListener");
private CompositeBuilder policyAppliesToBuilder = null;
public CompositePolicyBuilderImpl(ExtensionPointRegistry registry) {
@@ -219,34 +219,30 @@ public class CompositePolicyBuilderImpl
}
private void checkForNoListenerIntent(Endpoint ep, BuilderContext context) {
- PolicyHelper helper = new PolicyHelper();
- if ( helper.getIntent(ep, Constants.NOLISTENER_INTENT) != null ) {
- error(context.getMonitor(),
- "NoListenerIntentSpecifiedOnService",
- this,
- ep.toString());
- }
-
- }
-
- private void removeDirectPolicySetsIfExternalExists(PolicySubject subject,
- BuilderContext context) {
- boolean foundExternalPolicySet = false;
- for (PolicySet ps : subject.getPolicySets() ) {
- if ( ps.getAttachTo() != null )
- foundExternalPolicySet = true;
- }
-
- if ( foundExternalPolicySet ) {
- List<PolicySet> copy = new ArrayList<PolicySet>(subject.getPolicySets());
- for ( PolicySet ps : copy ) {
- if ( ps.getAttachTo() == null ) {
- subject.getPolicySets().remove(ps);
- }
- }
- }
-
- }
+ PolicyHelper helper = new PolicyHelper();
+ if (helper.getIntent(ep, NOLISTENER_INTENT) != null) {
+ error(context.getMonitor(), "NoListenerIntentSpecifiedOnService", this, ep.toString());
+ }
+
+ }
+
+ private void removeDirectPolicySetsIfExternalExists(PolicySubject subject, BuilderContext context) {
+ boolean foundExternalPolicySet = false;
+ for (PolicySet ps : subject.getPolicySets()) {
+ if (ps.getAttachTo() != null)
+ foundExternalPolicySet = true;
+ }
+
+ if (foundExternalPolicySet) {
+ List<PolicySet> copy = new ArrayList<PolicySet>(subject.getPolicySets());
+ for (PolicySet ps : copy) {
+ if (ps.getAttachTo() == null) {
+ subject.getPolicySets().remove(ps);
+ }
+ }
+ }
+
+ }
/**
* This is mainly about removing policies that don't "applyTo" the element where