Only run email checks if email changed

codidact · Aug 6, 2020 · 2e85331 · 2e85331
1 parent 239a4f3
commit 2e85331
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/app/models/user.rb b/app/models/user.rb
@@ -125,6 +125,7 @@ def username_not_fake_admin
 
   def email_domain_not_blocklisted
     return unless File.exist?(Rails.root.join('../.qpixel-domain-blocklist.txt'))
+    return unless saved_changes.include? 'email'
 
     blocklist = File.read(Rails.root.join('../.qpixel-domain-blocklist.txt')).split("\n")
     email_domain = email.split('@')[-1]
@@ -158,6 +159,7 @@ def is_not_blocklisted
 
   def email_not_bad_pattern
     return unless File.exist?(Rails.root.join('../.qpixel-email-patterns.txt'))
+    return unless saved_changes.include? 'email'
 
     patterns = File.read(Rails.root.join('../.qpixel-email-patterns.txt')).split("\n")
     matched = patterns.select { |p| email.match? Regexp.new(p) }