Use $value in valid_xxx() functions instead of the member field. They're equivalent, but it's more intuitive this way.

1 files changed, 2 insertions, 2 deletions

diff --git a/modules/gallery/models/item.php b/modules/gallery/models/item.php
index 19bdf655..9edc65ce 100644
--- a/modules/gallery/models/item.php
+++ b/modules/gallery/models/item.php
@@ -666,7 +666,7 @@ class Item_Model extends ORM_MPTT {
         ->from("items")
         ->where("parent_id", "=", $this->parent_id)
         ->where("id", "<>", $this->id)
-        ->where("slug", "=", $this->slug)
+        ->where("slug", "=", $value)
         ->count_records()) {
       $v->add_error("slug", "conflict");
     }
@@ -685,7 +685,7 @@ class Item_Model extends ORM_MPTT {
                ->from("items")
                ->where("parent_id", "=", $this->parent_id)
                ->where("id", "<>", $this->id)
-               ->where("name", "=", $this->name)
+               ->where("name", "=", $value)
                ->count_records()) {
       $v->add_error("name", "conflict");
     }