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/items.php | 140 +++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 140 insertions(+) create mode 100644 core/controllers/items.php (limited to 'core/controllers/items.php') diff --git a/core/controllers/items.php b/core/controllers/items.php new file mode 100644 index 00000000..6c202b30 --- /dev/null +++ b/core/controllers/items.php @@ -0,0 +1,140 @@ +type}s}/$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}s/{$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}s/{$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