diff --git a/core/controllers/items.php b/core/controllers/items.php index 6cf27fbf..6ba7db65 100644 --- a/core/controllers/items.php +++ b/core/controllers/items.php @@ -100,7 +100,9 @@ class Items_Controller extends REST_Controller { $parent = $item->parent(); if ($parent->id) { module::event("{$item->type}_before_delete", $item); + $item->delete(); + file::unlink($item->file_path()); } url::redirect("{$parent->type}s/{$parent->id}"); diff --git a/core/helpers/MY_file.php b/core/helpers/MY_file.php new file mode 100644 index 00000000..89e81204 --- /dev/null +++ b/core/helpers/MY_file.php @@ -0,0 +1,43 @@ +isDot()) { + unset($resource); + continue; + } else if ($resource->isFile()) { + self::unlink($resource->getPathName()); + } else if ($resource->isDir()) { + self::unlink($resource->getRealPath()); + } + unset($resource); + } + return @rmdir($path); + } + return false; + } + + +} diff --git a/core/tests/File_Helper_Test.php b/core/tests/File_Helper_Test.php new file mode 100644 index 00000000..dd71aabb --- /dev/null +++ b/core/tests/File_Helper_Test.php @@ -0,0 +1,40 @@ +assert_boolean(!file_exists($filename), "File not deleted"); + } + + public function remove_album_test() { + $dirname = (VARPATH . "albums/testdir"); + mkdir($dirname, 0777, true); + + $filename = tempnam($dirname, "file"); + touch($filename); + + file::unlink($dirname); + $this->assert_boolean(!file_exists($filename), "File not deleted"); + $this->assert_boolean(!file_exists($dirname), "Directory not deleted"); + } +}