You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tuscany.apache.org by sc...@apache.org on 2011/07/28 00:01:00 UTC
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
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);
}