Changed the check in organize_Controller::move_to:

1) check that target and source are not the same.
2) check that the source is not already a child of the target (use to be a
   descendant) Fixes #741.

1 files changed, 1 insertions, 1 deletions

diff --git a/modules/organize/controllers/organize.php b/modules/organize/controllers/organize.php
index 27299e85..0611c334 100644
--- a/modules/organize/controllers/organize.php
+++ b/modules/organize/controllers/organize.php
@@ -47,7 +47,7 @@ class Organize_Controller extends Controller {
     $album = ORM::factory("item", $album_id);
     foreach ($this->input->post("source_ids") as $source_id) {
       $source = ORM::factory("item", $source_id);
-      if (!$album->is_descendant($source)) {
+      if ($album->id != $source->parent_id && $album->id != $source->id) {
         item::move($source, $album);
       }
     }