From 5713e3c66ed8bab3b86490d7b6e90f7947d53429 Mon Sep 17 00:00:00 2001
From: Bharat Mediratta <bharat@menalto.com>
Date: Tue, 23 Dec 2008 04:36:09 +0000
Subject: Change photo::create() and album::create() to take ORM instances
 instead of ids.

---
 core/helpers/photo.php | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

(limited to 'core/helpers/photo.php')

diff --git a/core/helpers/photo.php b/core/helpers/photo.php
index 8c826e1c..d8fc2595 100644
--- a/core/helpers/photo.php
+++ b/core/helpers/photo.php
@@ -33,7 +33,11 @@ class photo_Core {
    * @param string  $description (optional) the longer description of this photo
    * @return Item_Model
    */
-  static function create($parent_id, $filename, $name, $title, $description=null, $owner_id=null) {
+  static function create($parent, $filename, $name, $title, $description=null, $owner_id=null) {
+    if (!$parent->loaded || $parent->type != "album") {
+      throw new Exception("@todo INVALID_PARENT");
+    }
+
     if (!is_file($filename)) {
       throw new Exception("@todo MISSING_IMAGE_FILE");
     }
@@ -63,7 +67,7 @@ class photo_Core {
 
     // Randomize the name if there's a conflict
     while (ORM::Factory("item")
-           ->where("parent_id", $parent_id)
+           ->where("parent_id", $parent->id)
            ->where("name", $photo->name)
            ->find()->id) {
       // @todo Improve this.  Random numbers are not user friendly
@@ -71,11 +75,6 @@ class photo_Core {
     }
 
     // This saves the photo
-    $parent = ORM::factory("item", $parent_id);
-    if (!$parent->loaded) {
-      throw new Exception("@todo INVALID_PARENT_ID");
-    }
-
     $photo->add_to_parent($parent);
     copy($filename, $photo->file_path());
 
@@ -91,6 +90,8 @@ class photo_Core {
       $parent->save();
       graphics::generate($parent);
     }
+
+    return $photo;
   }
 
   static function get_add_form($parent) {
-- 
cgit v1.2.3