activemq-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From gtu...@apache.org
Subject svn commit: r987090 - in /activemq/trunk/activemq-core/src: main/java/org/apache/activemq/command/ActiveMQMessage.java test/java/org/apache/activemq/command/ActiveMQMessageTest.java
Date Thu, 19 Aug 2010 09:05:21 GMT
Author: gtully
Date: Thu Aug 19 09:05:21 2010
New Revision: 987090

URL: http://svn.apache.org/viewvc?rev=987090&view=rev
Log:
fix regression - spec compliance with patch for https://issues.apache.org/activemq/browse/AMQ-2840,
standard headers must be omitted from getPropopertyNames()

Modified:
    activemq/trunk/activemq-core/src/main/java/org/apache/activemq/command/ActiveMQMessage.java
    activemq/trunk/activemq-core/src/test/java/org/apache/activemq/command/ActiveMQMessageTest.java

Modified: activemq/trunk/activemq-core/src/main/java/org/apache/activemq/command/ActiveMQMessage.java
URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/command/ActiveMQMessage.java?rev=987090&r1=987089&r2=987090&view=diff
==============================================================================
--- activemq/trunk/activemq-core/src/main/java/org/apache/activemq/command/ActiveMQMessage.java
(original)
+++ activemq/trunk/activemq-core/src/main/java/org/apache/activemq/command/ActiveMQMessage.java
Thu Aug 19 09:05:21 2010
@@ -282,7 +282,7 @@ public class ActiveMQMessage extends Mes
     public Enumeration getPropertyNames() throws JMSException {
         try {
             Vector<String> result = new Vector<String>(this.getProperties().keySet());
-            result.addAll(new Vector<String>(JMS_PROPERTY_SETERS.keySet()));
+            // omit stand jms props are per spec
             return result.elements();
         } catch (IOException e) {
             throw JMSExceptionSupport.create(e);

Modified: activemq/trunk/activemq-core/src/test/java/org/apache/activemq/command/ActiveMQMessageTest.java
URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-core/src/test/java/org/apache/activemq/command/ActiveMQMessageTest.java?rev=987090&r1=987089&r2=987090&view=diff
==============================================================================
--- activemq/trunk/activemq-core/src/test/java/org/apache/activemq/command/ActiveMQMessageTest.java
(original)
+++ activemq/trunk/activemq-core/src/test/java/org/apache/activemq/command/ActiveMQMessageTest.java
Thu Aug 19 09:05:21 2010
@@ -354,8 +354,8 @@ public class ActiveMQMessageTest extends
         ActiveMQMessage msg = new ActiveMQMessage();
         String name1 = "floatProperty";
         msg.setFloatProperty(name1, 1.3f);
-        String name2 = "JMSXDeliveryCount";
-        msg.setIntProperty("name2", 1);
+        String name2 = "intProperty";
+        msg.setIntProperty(name2, 1);
         boolean found1 = false;
         boolean found2 = false;
         for (Enumeration iter = msg.getPropertyNames(); iter.hasMoreElements();) {



Mime
View raw message