Author: ieb
Date: Mon Feb 25 05:45:31 2013
New Revision: 1449604
URL: http://svn.apache.org/r1449604
Log:
SLING-2748 Adjusted compartable checks to match output of the Comparable API.
Modified:
sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/DefaultJobManager.java
Modified:
sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/DefaultJobManager.java
URL:
http://svn.apache.org/viewvc/sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/DefaultJobManager.java?rev=1449604&r1=1449603&r2=1449604&view=diff
==============================================================================
---
sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/DefaultJobManager.java
(original)
+++
sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/DefaultJobManager.java
Mon Feb 25 05:45:31 2013
@@ -609,13 +609,13 @@ public class DefaultJobManager
if ( value instanceof Comparable ) {
@SuppressWarnings({ "unchecked", "rawtypes" })
final int result =
((Comparable)value).compareTo(job.event.getProperty(propName));
- if ( op == Operation.LESS && result != -1 ) {
+ if ( op == Operation.LESS && result >= 0 ) {
return false;
- } else if ( op == Operation.LESS_OR_EQUALS && result
== 1 ) {
+ } else if ( op == Operation.LESS_OR_EQUALS && result >
0 ) {
return false;
- } else if ( op == Operation.GREATER_OR_EQUALS &&
result == -1 ) {
+ } else if ( op == Operation.GREATER_OR_EQUALS &&
result < 0 ) {
return false;
- } else if ( op == Operation.GREATER && result != 1 ) {
+ } else if ( op == Operation.GREATER && result <= 0 ) {
return false;
}
} else {
