Fix issue when saving a model without any changes

ruhlio · May 25, 2018 · a87091f · a87091f
1 parent fd89cfb
commit a87091f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/spec/model/model_spec.cr b/spec/model/model_spec.cr
@@ -197,7 +197,7 @@ module ModelSpec
           u.id.should eq(1)
 
           u = User.query.find_or_create({last_name: "Henry"}) do |u|
-            u.first_name = "King" #<< This should not be triggered since we found the row
+            u.first_name = "Thierry" #<< This should not be triggered since we found the row
           end
           u.first_name.should eq("Thierry")
           u.last_name.should eq("Henry")

diff --git a/src/clear/model/modules/has_saving.cr b/src/clear/model/modules/has_saving.cr
@@ -13,7 +13,7 @@ module Clear::Model::HasSaving
     raise Clear::Model::ReadOnlyModelError.new("The model is read-only") if self.class.read_only?
 
     with_triggers(:save) do
-      if valid?
+      if valid? && changed?
         if persisted?
           Clear::SQL.update(self.class.table).set(update_h).where { var("#{self.class.pkey}") == pkey }.execute
         else