Fixed: TMail 1.2.7 malforms To header when number of recipients more then 8 (#8751).

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@7863 e93f8b46-1217-0410-a6f0-8f06a7374b81

diff --git a/config/initializers/10-patches.rb b/config/initializers/10-patches.rb
index f409f6611394345db9f399bc46dbda1a7e1e5e99..f525e59464a1c2e5ec9b9e3cc30f4fd964073044 100644 (file)
--- a/config/initializers/10-patches.rb
+++ b/config/initializers/10-patches.rb
@@ -86,14 +86,21 @@ end
 
 ActionMailer::Base.send :include, AsynchronousMailer
 
-# TMail::Unquoter.convert_to_with_fallback_on_iso_8859_1 introduced in TMail 1.2.7
-# triggers a test failure in test_add_issue_with_japanese_keywords(MailHandlerTest)
 module TMail
+  # TMail::Unquoter.convert_to_with_fallback_on_iso_8859_1 introduced in TMail 1.2.7
+  # triggers a test failure in test_add_issue_with_japanese_keywords(MailHandlerTest)
   class Unquoter
     class << self
       alias_method :convert_to, :convert_to_without_fallback_on_iso_8859_1
     end
   end
+
+  # Patch for TMail 1.2.7. See http://www.redmine.org/issues/8751
+  class Encoder
+    def puts_meta(str)
+      add_text str
+    end
+  end
 end
 
 module ActionController

diff --git a/test/unit/mailer_test.rb b/test/unit/mailer_test.rb
index c819d272fc5b792a7301d89f277e970c24265e98..2a6e9b75829028d7cd7bc0813a0e5a5131ee18eb 100644 (file)
--- a/test/unit/mailer_test.rb
+++ b/test/unit/mailer_test.rb
@@ -472,6 +472,14 @@ class MailerTest < ActiveSupport::TestCase
     # should restore perform_deliveries
     assert ActionMailer::Base.perform_deliveries
   end
+  
+  def test_tmail_to_header_field_should_not_include_blank_lines
+    mail = TMail::Mail.new
+    mail.to = ["a.user@example.com", "v.user2@example.com", "e.smith@example.com", "info@example.com", "v.pupkin@example.com",
+      "b.user@example.com", "w.user2@example.com", "f.smith@example.com", "info2@example.com", "w.pupkin@example.com"]
+    
+    assert !mail.encoded.strip.split("\r\n").detect(&:blank?), "#{mail.encoded} malformed"
+  end
 
   context "layout" do
     should "include the emails_header" do