You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by kw...@apache.org on 2016/04/20 14:08:24 UTC

svn commit: r1740129 - in /qpid/java/trunk: broker-plugins/management-http/src/test/java/org/apache/qpid/server/management/plugin/servlet/query/ConfiguredObjectQueryTest.java common/src/main/java/org/apache/qpid/filter/UnaryExpression.java

Author: kwall
Date: Wed Apr 20 12:08:24 2016
New Revision: 1740129

URL: http://svn.apache.org/viewvc?rev=1740129&view=rev
Log:
QPID-7206: [Java Broker] Query API - ensure that IN expressions with numbers consider all items within the Set

Modified:
    qpid/java/trunk/broker-plugins/management-http/src/test/java/org/apache/qpid/server/management/plugin/servlet/query/ConfiguredObjectQueryTest.java
    qpid/java/trunk/common/src/main/java/org/apache/qpid/filter/UnaryExpression.java

Modified: qpid/java/trunk/broker-plugins/management-http/src/test/java/org/apache/qpid/server/management/plugin/servlet/query/ConfiguredObjectQueryTest.java
URL: http://svn.apache.org/viewvc/qpid/java/trunk/broker-plugins/management-http/src/test/java/org/apache/qpid/server/management/plugin/servlet/query/ConfiguredObjectQueryTest.java?rev=1740129&r1=1740128&r2=1740129&view=diff
==============================================================================
--- qpid/java/trunk/broker-plugins/management-http/src/test/java/org/apache/qpid/server/management/plugin/servlet/query/ConfiguredObjectQueryTest.java (original)
+++ qpid/java/trunk/broker-plugins/management-http/src/test/java/org/apache/qpid/server/management/plugin/servlet/query/ConfiguredObjectQueryTest.java Wed Apr 20 12:08:24 2016
@@ -406,6 +406,7 @@ public class ConfiguredObjectQueryTest e
         assertEquals("Unexpected number of results - two attributes of type enum", 1, results.size());
 
     }
+
     public void testQuery_EnumEquality_InExpresssions() throws Exception
     {
         final UUID objectUuid = UUID.randomUUID();

Modified: qpid/java/trunk/common/src/main/java/org/apache/qpid/filter/UnaryExpression.java
URL: http://svn.apache.org/viewvc/qpid/java/trunk/common/src/main/java/org/apache/qpid/filter/UnaryExpression.java?rev=1740129&r1=1740128&r2=1740129&view=diff
==============================================================================
--- qpid/java/trunk/common/src/main/java/org/apache/qpid/filter/UnaryExpression.java (original)
+++ qpid/java/trunk/common/src/main/java/org/apache/qpid/filter/UnaryExpression.java Wed Apr 20 12:08:24 2016
@@ -269,7 +269,10 @@ public abstract class UnaryExpression<T>
                 {
                     Number num1 = (Number) currentRvalue;
                     Number num2 = (Number) listItemValue;
-                    return num1.doubleValue() == num2.doubleValue() && num1.longValue() == num2.longValue();
+                    if (num1.doubleValue() == num2.doubleValue() && num1.longValue() == num2.longValue())
+                    {
+                        return true;
+                    }
                 }
             }
             return false;



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@qpid.apache.org
For additional commands, e-mail: commits-help@qpid.apache.org