can_has_validations 0.4.2 → 0.4.3

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    ZTc3ZDIyYWM3Y2ExNWY3YWNjZmM4MDZlZWM3Y2RmNDkzZjA0MDUxNA==
+    ZjBjODY1MTlkYjZlYzA1NmU4MWI3NWRlZWU2OWMxNzE2MDI1OTA2OA==
   data.tar.gz: !binary |-
-    OGEyMmM2YjM2OWJlYzAzNTM1ODExMDM2YzJkZDA2MDU5ODNjZTk4MA==
+    YzJiYzNiYmMyMjZmOWNlNjNlN2JkOTI2ODRhMWI5YmFlMjJhZTY0Yg==
 SHA512:
   metadata.gz: !binary |-
-    N2FjMGUzNjkxOTA2ODQ0N2ViOTQxYmQ3NDZiZWEyM2Y3ZDg4MGNmMDY4N2Y2
-    NDM3MjkxZTdhMDVlMWU1NTNlMjAyZDQzZmNkMTQ0MGU4Y2UzZjFkMjg2YTFl
-    NzMwY2ZjZDlhY2I0M2VjOThlNzJmYmMyNDM2OTQ4YzVkOWJjOTA=
+    OWYzMDhjMTc3MjgxMjBhMDY4M2MwYTQ5ZTI1MjIzNmU4MTcxNjg1NGZhNWFm
+    ZjRmOWNlYjc4YTkxNjNmYThiZTkyODA4NzA0YTU4ZjhmOTVhYzhkM2Q1MWZh
+    ZGFhNTdmNThhMjZmMDkwMTcyY2FkYzRiYWM4YTMyOThjNmZjMWM=
   data.tar.gz: !binary |-
-    ZDk1NGM3NTZkY2NiZjQwNDQ1NDRiOTQ5OGI1ZDk3MmIyOGRhOTQ3ZmZlYjZi
-    YzY0OTIzOTcyM2JmNWMxZWRkOThmMTBhOTMzMTQ5Mzc2NDFhMmZkYTY3NWU1
-    YzA1NjIwMGE3NjkwMWJhMmY5N2U4YWI2M2Q2YTgzNzJkOWJiNDg=
+    M2ZlYTc5OWJjODNlMGEzNDZjZjBlMzk0ZGM0NzljYzA2NjZiODg3YTNiMWY2
+    OGIzOTAzY2Y1M2E5MTEwOWU1M2FmOTNiMjNlOTM4MTU3YzliYjVhMTYyY2U4
+    ZDliMTUzNzFjNmM3NTRmNjgxYmZlZTY5N2EyMzcxY2JhOTE3NzE=

data/lib/can_has_validations/validators/write_once_validator.rb

@@ -2,7 +2,9 @@
 #   Allows a value to be set to a non-nil value once, and then makes it immutable.
 #   Combine with :existence=>true to accomplish the same thing as attr_readonly,
 #   except with error messages (instead of silently refusing to save the change).
-# eg: validates :user_id, :write_once=>true
+# eg: validates :user_id, write_once: true
+#   Optionally refuses changing from nil => non-nil, always making field immutable.
+# eg: validates :source, write_once: {immutable_nil: true}
 
 module ActiveModel::Validations
   class WriteOnceValidator < ActiveModel::EachValidator
@@ -16,8 +18,10 @@ module ActiveModel::Validations
     end
 
     def validate_each(record, attribute, value)
-      if record.persisted? && record.send("#{attribute}_changed?") && !record.send("#{attribute}_was").nil?
-        record.errors.add(attribute, :unchangeable, options)
+      if record.persisted? && record.send("#{attribute}_changed?")
+        if options[:immutable_nil] || !record.send("#{attribute}_was").nil?
+          record.errors.add(attribute, :unchangeable, options)
+        end
       end
     end
   end

data/lib/can_has_validations/version.rb

@@ -1,3 +1,3 @@
 module CanHasValidations
-  VERSION = '0.4.2'
+  VERSION = '0.4.3'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: can_has_validations
 version: !ruby/object:Gem::Version
-  version: 0.4.2
+  version: 0.4.3
 platform: ruby
 authors:
 - thomas morgan
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-02-13 00:00:00.000000000 Z
+date: 2016-08-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -60,7 +60,6 @@ files:
 - lib/can_has_validations/validators/existence_validator.rb
 - lib/can_has_validations/validators/grandparent_validator.rb
 - lib/can_has_validations/validators/ordering_validator.rb
-- lib/can_has_validations/validators/safe_uniqueness_validator.rb
 - lib/can_has_validations/validators/url_validator.rb
 - lib/can_has_validations/validators/write_once_validator.rb
 - lib/can_has_validations/version.rb

data/lib/can_has_validations/validators/safe_uniqueness_validator.rb

@@ -1,15 +0,0 @@
-# better behaved version of the uniqueness validator
-#   Difference is it doesn't puke if the foreign_key is an invalid type (for
-#   example, an invalid string being provided for UUID type key). Instead,
-#   returns an error about being unable to find the key value.
-# eg: validates :user_id, :safe_uniqueness=>true
-
-class SafeUniquenessValidator < ActiveRecord::Validations::UniquenessValidator
-  def validate_each(record, attribute, value)
-    record.transaction(:requires_new=>true) do
-      super
-    end
-  rescue ActiveRecord::StatementInvalid
-    record.errors.add(attribute, "%{attribute} %{value} not found", options.except(:case_sensitive, :scope).merge(:value=>value))
-  end  
-end