Author: rfeng
Date: Thu Jul  9 18:14:18 2009
New Revision: 792622

URL: http://svn.apache.org/viewvc?rev=792622&view=rev
Log:
Use the list of policies instead of policySet to drive the logging 

Modified:
    
tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingImplementationPolicyProvider.java
    
tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingPolicyInterceptor.java
    
tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingPolicyProviderFactory.java
    
tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingReferencePolicyProvider.java
    
tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingServicePolicyProvider.java

Modified: 
tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingImplementationPolicyProvider.java
URL: 
http://svn.apache.org/viewvc/tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingImplementationPolicyProvider.java?rev=792622&r1=792621&r2=792622&view=diff
==============================================================================
--- 
tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingImplementationPolicyProvider.java
 (original)
+++ 
tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingImplementationPolicyProvider.java
 Thu Jul  9 18:14:18 2009
@@ -19,10 +19,11 @@
 
 package org.apache.tuscany.sca.policy.logging.jdk;
 
+import java.util.List;
+
 import org.apache.tuscany.sca.interfacedef.Operation;
 import org.apache.tuscany.sca.invocation.Phase;
 import org.apache.tuscany.sca.invocation.PhasedInterceptor;
-import org.apache.tuscany.sca.policy.PolicySet;
 import org.apache.tuscany.sca.provider.BasePolicyProvider;
 import org.apache.tuscany.sca.runtime.RuntimeComponent;
 
@@ -39,9 +40,9 @@
      * @see 
org.apache.tuscany.sca.provider.PolicyProvider#createInterceptor(org.apache.tuscany.sca.interfacedef.Operation)
      */
     public PhasedInterceptor createInterceptor(Operation operation) {
-        PolicySet ps = findPolicySet();
-        return ps == null ? null : new 
JDKLoggingPolicyInterceptor(getContext(), operation, ps,
-                                                                   
Phase.IMPLEMENTATION_POLICY);
+        List<JDKLoggingPolicy> policies = findPolicies();
+        return policies.isEmpty() ? null : new 
JDKLoggingPolicyInterceptor(getContext(), operation, policies,
+                                                                           
Phase.IMPLEMENTATION_POLICY);
     }
 
 }

Modified: 
tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingPolicyInterceptor.java
URL: 
http://svn.apache.org/viewvc/tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingPolicyInterceptor.java?rev=792622&r1=792621&r2=792622&view=diff
==============================================================================
--- 
tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingPolicyInterceptor.java
 (original)
+++ 
tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingPolicyInterceptor.java
 Thu Jul  9 18:14:18 2009
@@ -18,6 +18,7 @@
  */
 package org.apache.tuscany.sca.policy.logging.jdk;
 
+import java.util.List;
 import java.util.logging.ConsoleHandler;
 import java.util.logging.Handler;
 import java.util.logging.Level;
@@ -29,7 +30,6 @@
 import org.apache.tuscany.sca.invocation.Invoker;
 import org.apache.tuscany.sca.invocation.Message;
 import org.apache.tuscany.sca.invocation.PhasedInterceptor;
-import org.apache.tuscany.sca.policy.PolicySet;
 
 /**
  * Policy handler to handle PolicySet related to Logging with the QName
@@ -44,39 +44,44 @@
 
     private Invoker next;
     private Operation operation;
-    private PolicySet policySet = null;
+    private List<JDKLoggingPolicy> policies;
     private String context;
     private String phase;
 
-    public JDKLoggingPolicyInterceptor(String context, Operation operation, 
PolicySet policySet, String phase) {
+    public JDKLoggingPolicyInterceptor(String context,
+                                       Operation operation,
+                                       List<JDKLoggingPolicy> policies,
+                                       String phase) {
         super();
         this.operation = operation;
-        this.policySet = policySet;
+        this.policies = policies;
         this.context = context;
         this.phase = phase;
         init();
     }
 
+    public void start() {
+        init();
+    }
+
     private void init() {
-        if (policySet != null) {
-            JDKLoggingPolicy policy = 
(JDKLoggingPolicy)policySet.getPolicies().get(0);
-            logger = Logger.getLogger(policy.getLoggerName());
-            logger.setLevel(policy.getLogLevel());
-            logger.setUseParentHandlers(policy.isUseParentHandlers());
-
-            boolean found = false;
-            for (Handler handler : logger.getHandlers()) {
-                if (handler instanceof ConsoleHandler) {
-                    found = true;
-                    break;
-                }
+        JDKLoggingPolicy policy = policies.get(0);
+        logger = Logger.getLogger(policy.getLoggerName());
+        logger.setLevel(policy.getLogLevel());
+        logger.setUseParentHandlers(policy.isUseParentHandlers());
+
+        boolean found = false;
+        for (Handler handler : logger.getHandlers()) {
+            if (handler instanceof ConsoleHandler) {
+                found = true;
+                break;
             }
+        }
 
-            if (!found) {
-                ConsoleHandler consoleHandler = new ConsoleHandler();
-                consoleHandler.setLevel(Level.ALL);
-                logger.addHandler(consoleHandler);
-            }
+        if (!found) {
+            ConsoleHandler consoleHandler = new ConsoleHandler();
+            consoleHandler.setLevel(Level.ALL);
+            logger.addHandler(consoleHandler);
         }
     }
 
@@ -132,5 +137,5 @@
     public String getPhase() {
         return phase;
     }
-    
+
 }

Modified: 
tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingPolicyProviderFactory.java
URL: 
http://svn.apache.org/viewvc/tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingPolicyProviderFactory.java?rev=792622&r1=792621&r2=792622&view=diff
==============================================================================
--- 
tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingPolicyProviderFactory.java
 (original)
+++ 
tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingPolicyProviderFactory.java
 Thu Jul  9 18:14:18 2009
@@ -61,9 +61,8 @@
     /**
      * @see org.apache.tuscany.sca.provider.ProviderFactory#getModelType()
      */
-    public Class getModelType() {
-        // TODO Auto-generated method stub
-        return null;
+    public Class<JDKLoggingPolicy> getModelType() {
+        return JDKLoggingPolicy.class;
     }
 
 }

Modified: 
tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingReferencePolicyProvider.java
URL: 
http://svn.apache.org/viewvc/tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingReferencePolicyProvider.java?rev=792622&r1=792621&r2=792622&view=diff
==============================================================================
--- 
tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingReferencePolicyProvider.java
 (original)
+++ 
tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingReferencePolicyProvider.java
 Thu Jul  9 18:14:18 2009
@@ -19,22 +19,27 @@
 
 package org.apache.tuscany.sca.policy.logging.jdk;
 
-import org.apache.tuscany.sca.assembly.Binding;
+import java.util.List;
+
 import org.apache.tuscany.sca.assembly.EndpointReference;
+import org.apache.tuscany.sca.interfacedef.Operation;
+import org.apache.tuscany.sca.invocation.Phase;
+import org.apache.tuscany.sca.invocation.PhasedInterceptor;
 import org.apache.tuscany.sca.provider.BasePolicyProvider;
-import org.apache.tuscany.sca.runtime.RuntimeComponent;
-import org.apache.tuscany.sca.runtime.RuntimeComponentReference;
 
 /**
  * @version $Rev$ $Date$
  */
 public class JDKLoggingReferencePolicyProvider extends 
BasePolicyProvider<JDKLoggingPolicy> {
-    private RuntimeComponent component;
-    private RuntimeComponentReference reference;
-    private Binding binding;
 
     public JDKLoggingReferencePolicyProvider(EndpointReference 
endpointReference) {
         super(JDKLoggingPolicy.class, endpointReference);
     }
 
+    public PhasedInterceptor createInterceptor(Operation operation) {
+        List<JDKLoggingPolicy> policies = findPolicies();
+        return policies.isEmpty() ? null : new 
JDKLoggingPolicyInterceptor(getContext(), operation, policies,
+                                                                           
Phase.REFERENCE_POLICY);
+    }
+
 }

Modified: 
tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingServicePolicyProvider.java
URL: 
http://svn.apache.org/viewvc/tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingServicePolicyProvider.java?rev=792622&r1=792621&r2=792622&view=diff
==============================================================================
--- 
tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingServicePolicyProvider.java
 (original)
+++ 
tuscany/java/sca/modules/policy-logging/src/main/java/org/apache/tuscany/sca/policy/logging/jdk/JDKLoggingServicePolicyProvider.java
 Thu Jul  9 18:14:18 2009
@@ -19,11 +19,12 @@
 
 package org.apache.tuscany.sca.policy.logging.jdk;
 
+import java.util.List;
+
 import org.apache.tuscany.sca.assembly.Endpoint;
 import org.apache.tuscany.sca.interfacedef.Operation;
 import org.apache.tuscany.sca.invocation.Phase;
 import org.apache.tuscany.sca.invocation.PhasedInterceptor;
-import org.apache.tuscany.sca.policy.PolicySet;
 import org.apache.tuscany.sca.provider.BasePolicyProvider;
 
 /**
@@ -36,8 +37,8 @@
     }
 
     public PhasedInterceptor createInterceptor(Operation operation) {
-        PolicySet ps = findPolicySet();
-        return ps == null ? null : new 
JDKLoggingPolicyInterceptor(getContext(), operation, ps, Phase.SERVICE_POLICY);
+        List<JDKLoggingPolicy> policies = findPolicies();
+        return policies.isEmpty() ? null : new 
JDKLoggingPolicyInterceptor(getContext(), operation, policies, 
Phase.SERVICE_POLICY);
     }
 
 }


Reply via email to