diff --git a/modules/user_homes/helpers/user_homes_event.php b/modules/user_homes/helpers/user_homes_event.php index 00842f23..bf586827 100644 --- a/modules/user_homes/helpers/user_homes_event.php +++ b/modules/user_homes/helpers/user_homes_event.php @@ -1,5 +1,5 @@ where("id", $user->id)->find(); - if ($home) { - if ($home->home!=0) { - $session = Session::instance(); - $session->set("redirect_home",$home->home); - } + if ($home->loaded && $home->home != 0) { + Session::instance()->set("redirect_home", $home->home); } } @@ -44,101 +40,99 @@ class user_homes_event_Core { $session = Session::instance(); $home = $session->get("redirect_home"); if ($home) { - // remove from session to ensure redirect does not - // occur again + // Remove from session to ensure redirect does not occur again $session->set("redirect_home",null); url::redirect("albums/$home"); } } /** - * called just before a user is deleted. This will remove the user from + * Called just before a user is deleted. This will remove the user from * the user_homes directory. */ static function user_before_delete($user) { ORM::factory("user_home") - ->where("id", $user->id) - ->delete_all(); + ->where("id", $user->id) + ->delete_all(); } /** - * called when admin is adding a user + * Called when admin is adding a user */ static function user_add_form_admin($user, $form) { $form->add_user->dropdown("user_home") - ->label(t("Home Gallery")) - ->options(self::createGalleryArray()) - ->selected(0); + ->label(t("Home Gallery")) + ->options(self::createGalleryArray()) + ->selected(0); } /** - * called after a user has been added + * Called after a user has been added */ static function user_add_form_admin_completed($user, $form) { $home = ORM::factory("user_home")->where("id", $user->id)->find(); - $home->id=$user->id; - $home->home=$form->add_user->user_home->value; + $home->id = $user->id; + $home->home = $form->add_user->user_home->value; $home->save(); } /** - * called when admin is editing a user + * Called when admin is editing a user */ static function user_edit_form_admin($user, $form) { $home = ORM::factory("user_home")->where("id", $user->id)->find(); - - if ($home) { + if ($home->loaded) { $selected = $home->home; } else { $selected = 0; } $form->edit_user->dropdown("user_home") - ->label(t("Home Gallery")) - ->options(self::createGalleryArray()) - ->selected($selected); + ->label(t("Home Gallery")) + ->options(self::createGalleryArray()) + ->selected($selected); } /** - * called after a user had been edited by the admin + * Called after a user had been edited by the admin */ static function user_edit_form_admin_completed($user, $form) { $home = ORM::factory("user_home")->where("id", $user->id)->find(); - $home->id=$user->id; - $home->home=$form->edit_user->user_home->value; + $home->id = $user->id; + $home->home = $form->edit_user->user_home->value; $home->save(); } /** - * called when user is editing their own form + * Called when user is editing their own form */ static function user_edit_form($user, $form) { $home = ORM::factory("user_home")->where("id", $user->id)->find(); - if ($home) { + if ($home->loaded) { $selected = $home->home; } else { $selected = 0; } $form->edit_user->dropdown("user_home") - ->label(t("Home Gallery")) - ->options(self::createGalleryArray()) - ->selected($selected); + ->label(t("Home Gallery")) + ->options(self::createGalleryArray()) + ->selected($selected); } /** - * called after a user had been edited by the user + * Called after a user had been edited by the user */ static function user_edit_form_completed($user, $form) { $home = ORM::factory("user_home")->where("id", $user->id)->find(); - $home->id=$user->id; - $home->home=$form->edit_user->user_home->value; + $home->id = $user->id; + $home->home = $form->edit_user->user_home->value; $home->save(); } /** - * creates an array of galleries + * Creates an array of galleries */ static function createGalleryArray() { $array[0] = "none"; @@ -158,9 +152,9 @@ class user_homes_event_Core { } $albums = ORM::factory("item") - ->where(array("parent_id" => $parent->id, "type" => "album")) - ->orderby(array("title" => "ASC")) - ->find_all(); + ->where(array("parent_id" => $parent->id, "type" => "album")) + ->orderby(array("title" => "ASC")) + ->find_all(); foreach ($albums as $album) { self::tree($album, "-$dashes", $array); } diff --git a/modules/user_homes/helpers/user_homes_installer.php b/modules/user_homes/helpers/user_homes_installer.php index e29e6844..0e80e6c1 100644 --- a/modules/user_homes/helpers/user_homes_installer.php +++ b/modules/user_homes/helpers/user_homes_installer.php @@ -1,5 +1,5 @@