Merge pull request #178 from vkononov/bug-support-email-array

Fix an issue with validating an email array

lib/valid_email2/email_validator.rb

@@ -12,8 +12,7 @@ def validate_each(record, attribute, value)
       return unless value.present?
       options = default_options.merge(self.options)
 
-      value_spitted = options[:multiple] ? value.split(',').map(&:strip) : [value]
-      addresses = value_spitted.map { |v| ValidEmail2::Address.new(v) }
+      addresses = sanitized_values(value).map { |v| ValidEmail2::Address.new(v) }
 
       error(record, attribute) && return unless addresses.all?(&:valid?)
 
@@ -54,6 +53,18 @@ def validate_each(record, attribute, value)
       end
     end
 
+    def sanitized_values(input)
+      options = default_options.merge(self.options)
+
+      if options[:multiple]
+        email_list = input.is_a?(Array) ? input : input.split(',')
+      else
+        email_list = [input]
+      end
+
+      email_list.reject(&:empty?).map(&:strip)
+    end
+
     def error(record, attribute)
       record.errors.add(attribute, options[:message] || :invalid)
     end

spec/valid_email2_spec.rb

@@ -312,6 +312,11 @@ def set_whitelist
       expect(user.valid?).to be_truthy
     end
 
+    it "tests each address from an array" do
+      user = TestUserMultiple.new(email: %w[foo@gmail.com bar@gmail.com])
+      expect(user.valid?).to be_truthy
+    end
+
     context 'when one address is invalid' do
       it "fails for all" do
         user = TestUserMultiple.new(email: "foo@gmail.com, bar@123")