effective_resources 2.11.3 → 2.11.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 812cfa2dff0975f437ae27401f0514ffda1e9e0f63656ed9a108d36994e6f54e
-  data.tar.gz: 3abb1076d4711aacaeeec65dd3cc0c37d5e9c10d07ee254f6a8b870a626e115a
+  metadata.gz: 0db1af1cd9888460bd6ba110a0d972925ad88c0221bc77d6fc98a15b44175b31
+  data.tar.gz: acb21c1d6a77daec5e2401ee22aa17b60005a46dfc708b06e8b1a5c0b5437196
 SHA512:
-  metadata.gz: bd3bf2b4b9073a4097c86a4cc8ba7e9199080f4ce835431be533e20227728fc389b72d5fdf4aeccb2f71818bb312db477df0db42d2f3b4f5017ce527cb964b86
-  data.tar.gz: ab42b89c90b2bd741e325148dab189fc600e49da80dec0596dfeb238ada7daa5f9edef3a8bf497d7bdce0004edbcf98d9a06b8a03afcf1b62272da398e4f97f5
+  metadata.gz: a62c534e47770dda823fb51cc30834f1de229c61dae266d1de6f20031c6137ce57782c11f512d83a82bb0a90ca7983b439c94dd55298cdbba5466367d7e057a4
+  data.tar.gz: ade2cf04ecca9009abe9b2883252f5064843e83f76232d5ab8e51418370a0297bbe436ea83700d33c8840778cad272f0ffc447930fa10c631b92788c961aa3bc

data/app/models/concerns/acts_as_statused.rb

@@ -96,7 +96,7 @@ module ActsAsStatused
           id = status_steps[sym_by_id]
           klass = status_steps[sym_by_type]
 
-          klass.constantize.find(id) if id.present? && klass.present?
+          klass.constantize.find_by_id(id) if id.present? && klass.present?
         end
       end
 

data/app/models/concerns/effective_devise_user.rb

@@ -48,12 +48,17 @@ module EffectiveDeviseUser
     # Devise invitable ignores model validations, so we manually check for duplicate email addresses.
     before_save(if: -> { new_record? && try(:invitation_sent_at).present? }) do
       if email.blank?
-        self.errors.add(:email, "can't be blank")
+        errors.add(:email, "can't be blank")
         raise("email can't be blank")
       end
 
-      if self.class.where(email: email.downcase.strip).exists?
-        self.errors.add(:email, 'has already been taken')
+      if self.class.where(email: value).present?
+        errors.add(:email, 'has already been taken')
+        raise("email has already been taken")
+      end
+
+      if respond_to?(:alternate_email) && self.class.where(alternate_email: value).present?
+        errors.add(:email, 'has already been taken')
         raise("email has already been taken")
       end
     end
@@ -63,25 +68,29 @@ module EffectiveDeviseUser
       assign_attributes(provider: nil, access_token: nil, refresh_token: nil, token_expires_at: nil)
     end
 
+    validate(if: -> { email.present? && try(:alternate_email).present? }) do
+      errors.add(:alternate_email, 'cannot be the same as email') if email.strip.downcase == alternate_email.strip.downcase
+    end
+
     # Uniqueness validation of emails and alternate emails across all users
     validate(if: -> { respond_to?(:alternate_email) }) do
-      records = self.class.where.not(id: self.id) # exclude self
-      email_duplicates = records.where("lower(email) = :email OR lower(alternate_email) = :email", email: email.to_s.strip.downcase)
-      alternate_email_duplicates = records.where("lower(email) = :alternate_email OR lower(alternate_email) = :alternate_email", alternate_email: alternate_email.to_s.strip.downcase)
+      records = self.class.where.not(id: id)
 
-      # Check if a uniqueness validation was already performed before triggering the exists query
-      if !self.errors.added?(:email, 'has already been taken') && email_duplicates.exists?
-        self.errors.add(:email, 'has already been taken')
+      # Validate email uniqueness
+      if (value = email.to_s.strip.downcase).present? && !errors.added?(:email, 'has already been taken')
+        existing = records.where("email = :value OR alternate_email = :value", value: value)
+        errors.add(:email, 'has already been taken') if existing.present?
       end
 
-      # Check if the alternate email is set before triggering the exists query
-      if try(:alternate_email).present? && alternate_email_duplicates.exists?
-        self.errors.add(:alternate_email, 'has already been taken')
+      # Validate alternate_email uniqueness
+      if (value = alternate_email.to_s.strip.downcase).present?
+        existing = records.where("email = :value OR alternate_email = :value", value: value)
+        errors.add(:alternate_email, 'has already been taken') if existing.present?
       end
     end
 
     with_options(if: -> { respond_to?(:alternate_email) }) do
-      validates :alternate_email, email: true, allow_blank: true
+      validates :alternate_email, email: true
     end
 
   end

data/lib/effective_resources/engine.rb

@@ -6,13 +6,11 @@ module EffectiveResources
 
     config.autoload_paths += Dir[
       "#{config.root}/jobs/",
-      "#{config.root}/lib/validators/",
       "#{config.root}/app/controllers/concerns/"
     ]
 
     config.eager_load_paths += Dir[
       "#{config.root}/jobs/",
-      "#{config.root}/lib/validators/",
       "#{config.root}/app/controllers/concerns/"
     ]
 

data/lib/effective_resources/version.rb

@@ -1,3 +1,3 @@
 module EffectiveResources
-  VERSION = '2.11.3'.freeze
+  VERSION = '2.11.5'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: effective_resources
 version: !ruby/object:Gem::Version
-  version: 2.11.3
+  version: 2.11.5
 platform: ruby
 authors:
 - Code and Effect
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-09-25 00:00:00.000000000 Z
+date: 2023-10-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -223,8 +223,12 @@ files:
 - app/models/effective/resources/relation.rb
 - app/models/effective/resources/sql.rb
 - app/models/effective/resources/tenants.rb
+- app/validators/boolean_validator.rb
 - app/validators/content_type_validator.rb
+- app/validators/email_cc_validator.rb
+- app/validators/email_validator.rb
 - app/validators/size_validator.rb
+- app/validators/url_validator.rb
 - app/views/application/_flash.html.haml
 - app/views/application/create.js.erb
 - app/views/application/destroy.js.erb
@@ -251,10 +255,6 @@ files:
 - lib/effective_resources/engine.rb
 - lib/effective_resources/version.rb
 - lib/generators/effective_resources/install_generator.rb
-- lib/validators/boolean_validator.rb
-- lib/validators/email_cc_validator.rb
-- lib/validators/email_validator.rb
-- lib/validators/url_validator.rb
 homepage: https://github.com/code-and-effect/effective_resources
 licenses:
 - MIT