From f0c5e7eca3842411d4e3ee6efb3fd6ac0ad85f4a Mon Sep 17 00:00:00 2001
From: Brion Vibber <brion@pobox.com>
Date: Sat, 26 Jun 2010 15:07:32 -0400
Subject: Fix for bug #2382: releasing claim on failed queue item works again
 with DB-based queues. Warning: DB-based queue doesn't currently implement
 discarding of items after a retry limit. Failed items will be retried until
 they succeed.

---
 lib/dbqueuemanager.php | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

(limited to 'lib/dbqueuemanager.php')

diff --git a/lib/dbqueuemanager.php b/lib/dbqueuemanager.php
index 3032e4ec7..3dda9fd1a 100644
--- a/lib/dbqueuemanager.php
+++ b/lib/dbqueuemanager.php
@@ -135,9 +135,7 @@ class DBQueueManager extends QueueManager
         if (empty($qi->claimed)) {
             $this->_log(LOG_WARNING, "[$queue:item $qi->id] Ignoring failure for unclaimed queue item");
         } else {
-            $orig = clone($qi);
-            $qi->claimed = null;
-            $qi->update($orig);
+            $qi->releaseClaim();
         }
 
         $this->stats('error', $queue);
-- 
cgit v1.2.3-54-g00ecf