Only validate has_link if body/desc is present

SumOfUs · Mar 1, 2017 · a135db5 · a135db5
1 parent 5cff047
commit a135db5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/models/share/email.rb b/app/models/share/email.rb
@@ -17,7 +17,7 @@ class Share::Email < ActiveRecord::Base
   include Share::Variant
 
   validates :subject, :body, presence: true
-  validate :has_link
+  validate :has_link, unless: -> { body.nil? }
 
   def has_link
     errors.add(:body, 'does not contain {LINK}') unless body.match?(/\{LINK\}/)

diff --git a/app/models/share/twitter.rb b/app/models/share/twitter.rb
@@ -17,7 +17,7 @@ class Share::Twitter < ActiveRecord::Base
   include Share::Variant
 
   validates :description, presence: true
-  validate :has_link
+  validate :has_link, unless: -> { description.nil? }
 
   def has_link
     errors.add(:description, 'does not contain {LINK}') unless description.match?(/\{LINK\}/)