Rename file::unlink() to dir::unlink(). This creates an API that is

clearer about its actions, otherwise we may accidentally blow away an
entire hierarchy when we think we're deleting a single file (there's a
reason why rm won't let you delete dirs that aren't empty!)

Name it "dir" instead of "directory" because apparently "directory" is
already reserved.

Move the delete call out of Items_Controller and into the model.  The
model is responsible for keeping state integrity, not the controller.

1 files changed, 0 insertions, 1 deletions

diff --git a/core/controllers/items.php b/core/controllers/items.php
index 6ba7db65..f10a30c3 100644
--- a/core/controllers/items.php
+++ b/core/controllers/items.php
@@ -102,7 +102,6 @@ class Items_Controller extends REST_Controller {
       module::event("{$item->type}_before_delete", $item);
 
       $item->delete();
-      file::unlink($item->file_path());
     }
 
     url::redirect("{$parent->type}s/{$parent->id}");