Author: michiel
Date: 2010-05-03 14:46:24 +0200 (Mon, 03 May 2010)
New Revision: 42087

Modified:
   
mmbase/trunk/applications/crontab/src/main/java/org/mmbase/applications/crontab/CronEntry.java
   
mmbase/trunk/applications/crontab/src/main/java/org/mmbase/applications/crontab/builders/NodeCronEntry.java
Log:
some people change the node in the nodeentry itself (e.g. the configuration). 
This should however not influence the job

Modified: 
mmbase/trunk/applications/crontab/src/main/java/org/mmbase/applications/crontab/CronEntry.java
===================================================================
--- 
mmbase/trunk/applications/crontab/src/main/java/org/mmbase/applications/crontab/CronEntry.java
      2010-05-03 12:44:59 UTC (rev 42086)
+++ 
mmbase/trunk/applications/crontab/src/main/java/org/mmbase/applications/crontab/CronEntry.java
      2010-05-03 12:46:24 UTC (rev 42087)
@@ -399,7 +399,6 @@
         result = HashCodeUtil.hashCode(result, name);
         result = HashCodeUtil.hashCode(result, className);
         result = HashCodeUtil.hashCode(result, cronTime);
-        result = HashCodeUtil.hashCode(result, configuration);
         return result;
     }
 

Modified: 
mmbase/trunk/applications/crontab/src/main/java/org/mmbase/applications/crontab/builders/NodeCronEntry.java
===================================================================
--- 
mmbase/trunk/applications/crontab/src/main/java/org/mmbase/applications/crontab/builders/NodeCronEntry.java
 2010-05-03 12:44:59 UTC (rev 42086)
+++ 
mmbase/trunk/applications/crontab/src/main/java/org/mmbase/applications/crontab/builders/NodeCronEntry.java
 2010-05-03 12:46:24 UTC (rev 42087)
@@ -152,4 +152,18 @@
         return "NODE: " + super.toString();
     }
 
+
+
+    /**
+     * Two NodeCronEntry's as considered equal if they have the same id
+     */
+    @Override
+    public boolean equals(Object o) {
+        if (!(o instanceof NodeCronEntry)) {
+            return false;
+        }
+        CronEntry other = (CronEntry)o;
+        return getId().equals(other.getId());
+    }
+
 }

_______________________________________________
Cvs mailing list
Cvs@lists.mmbase.org
http://lists.mmbase.org/mailman/listinfo/cvs

Reply via email to