You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by gr...@apache.org on 2010/08/03 16:18:58 UTC
svn commit: r981896 -
/qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10.java
Author: grkvlt
Date: Tue Aug 3 14:18:58 2010
New Revision: 981896
URL: http://svn.apache.org/viewvc?rev=981896&view=rev
Log:
QPID-2744: Property name check in wrong place
Modified:
qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10.java
Modified: qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10.java
URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10.java?rev=981896&r1=981895&r2=981896&view=diff
==============================================================================
--- qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10.java (original)
+++ qpid/trunk/qpid/java/client/src/main/java/org/apache/qpid/client/message/AMQMessageDelegate_0_10.java Tue Aug 3 14:18:58 2010
@@ -734,6 +734,8 @@ public class AMQMessageDelegate_0_10 ext
public void setObjectProperty(String propertyName, Object object) throws JMSException
{
+ checkPropertyName(propertyName);
+ checkWritableProperties();
if (object == null)
{
throw new MessageFormatException(AMQPInvalidClassException.INVALID_OBJECT_MSG + "null");
@@ -742,8 +744,6 @@ public class AMQMessageDelegate_0_10 ext
{
throw new MessageFormatException(AMQPInvalidClassException.INVALID_OBJECT_MSG + object.getClass());
}
- checkPropertyName(propertyName);
- checkWritableProperties();
setApplicationHeader(propertyName, object);
}
---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project: http://qpid.apache.org
Use/Interact: mailto:commits-subscribe@qpid.apache.org