Use is_descendant() API inside move_to() for clarity.

gallery · Aug 30, 2009 · dee0abf · dee0abf
1 parent 53390ba
commit dee0abf
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/modules/gallery/libraries/ORM_MPTT.php b/modules/gallery/libraries/ORM_MPTT.php
@@ -212,8 +212,7 @@ function descendants_count($where=array()) {
    * @return  ORM_MTPP
    */
   function move_to($target) {
-    if ($this->left_ptr <= $target->left_ptr &&
-        $this->right_ptr >= $target->right_ptr) {
+    if ($this->is_descendant($target)) {
       throw new Exception("@todo INVALID_TARGET can't move item inside itself");
     }