From b2772f5a050351129a64b966b127e39cf76c80b5 Mon Sep 17 00:00:00 2001 From: Jozef Selesi Date: Wed, 19 Nov 2008 00:12:25 +0000 Subject: * Renamed the album, item and photo controllers to albums, items and photos in order to follow the convention that controllers that refer to a collection of resources have plural names. * Added a bug workaround to routes.php --- core/controllers/item.php | 140 ---------------------------------------------- 1 file changed, 140 deletions(-) delete mode 100644 core/controllers/item.php (limited to 'core/controllers/item.php') diff --git a/core/controllers/item.php b/core/controllers/item.php deleted file mode 100644 index f5c5f9c9..00000000 --- a/core/controllers/item.php +++ /dev/null @@ -1,140 +0,0 @@ -type}/$item->id"); - } - - public function _update($item) { - // @todo Productionize this code - // 1) Add security checks - // 2) Support owner_ids properly - - $user = Session::instance()->get("user"); - $owner_id = $user ? $user->id : $item->owner_id; - - switch ($this->input->post("type")) { - case "album": - $album = album::create( - $item->id, - $this->input->post("name"), - $this->input->post("title", $this->input->post("name")), - $this->input->post("description"), - $owner_id); - url::redirect("album/{$album->id}"); - break; - - case "photo": - if (is_array($_FILES["file"]["name"])) { - for ($i = 0; $i < count($_FILES["file"]["name"]) - 1; $i++) { - if ($_FILES["file"]["error"][$i] == 0) { - $photo = photo::create( - $item->id, - $_FILES["file"]["tmp_name"][$i], - $_FILES["file"]["name"][$i], - $_FILES["file"]["name"][$i], - "", $owner_id); - } else { - throw new Exception("@todo ERROR_IN_UPLOAD_FILE"); - } - } - url::redirect("album/{$item->id}"); - } else { - $photo = photo::create( - $item->id, - $_FILES["file"]["tmp_name"], - $_FILES["file"]["name"], - $this->input->post("title", $this->input->post("name")), - $this->input->post("description"), - $owner_id); - url::redirect("{$new_item->type}/{$new_item->id}"); - } - break; - } - } - - public function _delete($item) { - // @todo Production this code - // 1) Add security checks - $parent = $item->parent(); - if ($parent->id) { - $item->delete(); - } - url::redirect("{$parent->type}/{$parent->id}"); - } - - public function _create($item) { - // @todo Productionize this - // 1) Figure out how to do the right validation here. Validate the form input and apply it to - // the model as appropriate. - // 2) Figure out how to dispatch according to the needs of the client. Ajax requests from - // jeditable will want the changed field back, and possibly the whole item in json. - // - // For now let's establish a simple protocol where the client passes in a __return parameter - // that specifies which field it wants back from the item. Later on we can expand that to - // include a data format, etc. - - // These fields are safe to change - $post = $this->input->post(); - foreach ($post as $key => $value) { - switch ($key) { - case "title": - case "description": - $item->$key = $value; - break; - } - } - - // @todo Support additional fields - // These fields require additional work if you change them - // parent_id, owner_id - - $item->save(); - if (array_key_exists("__return", $post)) { - print $item->{$post["__return"]}; - } - } -} -- cgit v1.2.3