svn commit: r1151643 - /tuscany/sca-java-2.x/trunk/modules/binding-jms-runtime/src/main/java/org/apache/tuscany/sca/binding/jms/provider/AbstractMessageProcessor.java

[scottkurz]

Author: scottkurz
Date: Wed Jul 27 22:00:59 2011
New Revision: 1151643

URL: http://svn.apache.org/viewvc?rev=1151643&view=rev
Log:
Fix NPE which I think was just causing noise in the logs in the case of a 1-way 
operation, as we were hitting NPE in the response path but never needed to send 
a response anyway.

Modified:
    
tuscany/sca-java-2.x/trunk/modules/binding-jms-runtime/src/main/java/org/apache/tuscany/sca/binding/jms/provider/AbstractMessageProcessor.java

Modified: 
tuscany/sca-java-2.x/trunk/modules/binding-jms-runtime/src/main/java/org/apache/tuscany/sca/binding/jms/provider/AbstractMessageProcessor.java
URL: 
http://svn.apache.org/viewvc/tuscany/sca-java-2.x/trunk/modules/binding-jms-runtime/src/main/java/org/apache/tuscany/sca/binding/jms/provider/AbstractMessageProcessor.java?rev=1151643&r1=1151642&r2=1151643&view=diff
==============================================================================
--- 
tuscany/sca-java-2.x/trunk/modules/binding-jms-runtime/src/main/java/org/apache/tuscany/sca/binding/jms/provider/AbstractMessageProcessor.java
 (original)
+++ 
tuscany/sca-java-2.x/trunk/modules/binding-jms-runtime/src/main/java/org/apache/tuscany/sca/binding/jms/provider/AbstractMessageProcessor.java
 Wed Jul 27 22:00:59 2011
@@ -70,9 +70,9 @@ public abstract class AbstractMessagePro
      */
     public void setOperationName(String operationName, Message message) {
         try {
-
-            message.setStringProperty(operationPropertyName, operationName);
-
+            if (message != null) {
+                message.setStringProperty(operationPropertyName, 
operationName);
+            }
         } catch (JMSException e) {
             throw new JMSBindingException("Exception setting the operation 
name on message", e);
         }