From 39bd6892a8a8cf3cd08de85ae304bd547b6a5b5d Mon Sep 17 00:00:00 2001
From: Bharat Mediratta <bharat@menalto.com>
Date: Sun, 4 Apr 2010 14:48:19 -0700
Subject: [PATCH] Move the "Fix MPTT" task into the gallery module.

---
 modules/rescue/helpers/rescue_task.php | 85 --------------------------
 1 file changed, 85 deletions(-)

diff --git a/modules/rescue/helpers/rescue_task.php b/modules/rescue/helpers/rescue_task.php
index bb46c54d..1da25f3f 100644
--- a/modules/rescue/helpers/rescue_task.php
+++ b/modules/rescue/helpers/rescue_task.php
@@ -18,18 +18,8 @@
  * Foundation, Inc., 51 Franklin Street - Fifth Floor, Boston, MA  02110-1301, USA.
  */
 class rescue_task_Core {
-  const LEFT = 0;
-  const RIGHT = 1;
-
   static function available_tasks() {
     return array(Task_Definition::factory()
-                 ->callback("rescue_task::fix_mptt")
-                 ->name(t("Fix Album/Photo hierarchy"))
-                 ->description(t("Fix problems where your album/photo breadcrumbs are out of " .
-                                 "sync with your actual hierarchy."))
-                 ->severity(log::SUCCESS),
-
-                 Task_Definition::factory()
                  ->callback("rescue_task::fix_internet_addresses")
                  ->name(t("Fix internet addresses"))
                  ->description(t("Fix internet addresses broken when upgrading to Beta 3"))
@@ -37,56 +27,6 @@ class rescue_task_Core {
                  );
   }
 
-  static function fix_mptt($task) {
-    $start = microtime(true);
-
-    $total = $task->get("total");
-    if (empty($total)) {
-      $task->set("total", $total = db::build()->count_records("items"));
-      $task->set("stack", "1:" . self::LEFT);
-      $task->set("ptr", 1);
-      $task->set("completed", 0);
-    }
-
-    $ptr = $task->get("ptr");
-    $stack = explode(" ", $task->get("stack"));
-    $completed = $task->get("completed");
-
-    // Implement a depth-first tree walk using a stack.  Not the most efficient, but it's simple.
-    while ($stack && microtime(true) - $start < 1.5) {
-      list($id, $state) = explode(":", array_pop($stack));
-      switch ($state) {
-      case self::LEFT:
-        self::set_left($id, $ptr++);
-        $item = ORM::factory("item", $id);
-        array_push($stack, $id . ":" . self::RIGHT);
-        foreach (self::children($id) as $child) {
-          array_push($stack, $child->id . ":" . self::LEFT);
-        }
-        break;
-
-      case self::RIGHT:
-        self::set_right($id, $ptr++);
-        $completed++;
-        break;
-      }
-    }
-
-    $task->set("stack", implode(" ", $stack));
-    $task->set("ptr", $ptr);
-    $task->set("completed", $completed);
-
-    if ($total == $completed) {
-      $task->done = true;
-      $task->state = "success";
-      $task->percent_complete = 100;
-    } else {
-      $task->percent_complete = round(100 * $completed / $total);
-    }
-    $task->status = t2("One row updated", "%count / %total rows updated", $completed,
-                       array("total" => $total));
-  }
-
   static function fix_internet_addresses($task) {
     $start = microtime(true);
 
@@ -131,29 +71,4 @@ class rescue_task_Core {
     $task->status = t2("One row updated", "%count / %total rows updated", $completed,
                        array("total" => $total));
   }
-
-  static function children($parent_id) {
-    return db::build()
-      ->select("id")
-      ->from("items")
-      ->where("parent_id", "=", $parent_id)
-      ->order_by("left_ptr", "ASC")
-      ->execute();
-  }
-
-  static function set_left($id, $value) {
-    db::build()
-      ->update("items")
-      ->set("left_ptr", $value)
-      ->where("id", "=", $id)
-      ->execute();
-  }
-
-  static function set_right($id, $value) {
-    db::build()
-      ->update("items")
-      ->set("right_ptr", $value)
-      ->where("id", "=", $id)
-      ->execute();
-  }
 }