You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@activemq.apache.org by ch...@apache.org on 2012/10/30 16:56:05 UTC

svn commit: r1403751 - /activemq/trunk/activemq-core/src/main/java/org/apache/activemq/filter/PropertyExpression.java

Author: chirino
Date: Tue Oct 30 15:56:05 2012
New Revision: 1403751

URL: http://svn.apache.org/viewvc?rev=1403751&view=rev
Log:
Fixes AMQ-4144: When used in a message selector, the JMSDeliveryMode header field should be treated as having the values 'PERSISTENT' and 'NON_PERSISTENT'.

Modified:
    activemq/trunk/activemq-core/src/main/java/org/apache/activemq/filter/PropertyExpression.java

Modified: activemq/trunk/activemq-core/src/main/java/org/apache/activemq/filter/PropertyExpression.java
URL: http://svn.apache.org/viewvc/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/filter/PropertyExpression.java?rev=1403751&r1=1403750&r2=1403751&view=diff
==============================================================================
--- activemq/trunk/activemq-core/src/main/java/org/apache/activemq/filter/PropertyExpression.java (original)
+++ activemq/trunk/activemq-core/src/main/java/org/apache/activemq/filter/PropertyExpression.java Tue Oct 30 15:56:05 2012
@@ -74,7 +74,7 @@ public class PropertyExpression implemen
         JMS_PROPERTY_EXPRESSIONS.put("JMSDeliveryMode", new SubExpression() {
 
             public Object evaluate(Message message) {
-                return Integer.valueOf(message.isPersistent() ? DeliveryMode.PERSISTENT : DeliveryMode.NON_PERSISTENT);
+                return message.isPersistent() ? "PERSISTENT" : "NON_PERSISTENT";
             }
         });
         JMS_PROPERTY_EXPRESSIONS.put("JMSPriority", new SubExpression() {