actionmailer 0.6.1 → 0.7.0

data/CHANGELOG

@@ -1,3 +1,25 @@
+*0.7.0* (24th February, 2005)
+
+* Added support for charsets for both subject and body. The default charset is now UTF-8 #673 [Jamis Buck]. Examples:
+
+    def iso_charset(recipient)
+      @recipients = recipient
+      @subject    = "testing iso charsets"
+      @from       = "system@loudthinking.com"
+      @body       = "Nothing to see here."
+      @charset    = "iso-8859-1"
+    end
+    
+    def unencoded_subject(recipient)
+      @recipients = recipient
+      @subject    = "testing unencoded subject"
+      @from       = "system@loudthinking.com"
+      @body       = "Nothing to see here."
+      @encode_subject = false
+      @charset    = "iso-8859-1"
+    end
+    
+
 *0.6.1* (January 18th, 2005)
 
 * Fixed sending of emails to use Tmail#from not the deprecated Tmail#from_address

data/lib/action_mailer.rb

@@ -24,9 +24,12 @@
 begin
   require 'action_controller'
 rescue LoadError
-  # Action Pack is not already available, try RubyGems
-  require 'rubygems'	
-  require_gem 'actionpack', '>= 0.9.0'
+  begin
+    require File.dirname(__FILE__) + '/../../actionpack/lib/action_controller'
+  rescue LoadError
+    require 'rubygems'
+    require_gem 'actionpack', '>= 0.9.0'
+  end
 end
 
 $:.unshift(File.dirname(__FILE__) + "/action_mailer/vendor/")

data/lib/action_mailer/base.rb

@@ -55,6 +55,11 @@ module ActionMailer #:nodoc:
   #
   # * <tt>deliveries</tt> - Keeps an array of all the emails sent out through the Action Mailer with delivery_method :test. Most useful
   #   for unit and functional testing.
+  #
+  # * <tt>default_charset</tt> - The default charset used for the body and to encode the subject. Defaults to UTF-8. You can also 
+  #    pick a different charset from inside a method with <tt>@encoding</tt>.
+  #
+  # * <tt>encode_subject</tt> - Whether or not to encode the subject with the active charset. Defaults to true.
   class Base
     private_class_method :new #:nodoc:
 
@@ -83,10 +88,18 @@ module ActionMailer #:nodoc:
     @@deliveries = []
     cattr_accessor :deliveries
 
-    attr_accessor :recipients, :subject, :body, :from, :sent_on, :headers, :bcc, :cc
+    @@default_charset = "utf-8"
+    cattr_accessor :default_charset
+
+    @@encode_subject = true
+    cattr_accessor :encode_subject
+
+    attr_accessor :recipients, :subject, :body, :from, :sent_on, :headers, :bcc, :cc, :charset, :encode_subject
 
     def initialize
       @bcc = @cc = @from = @recipients = @sent_on = @subject = @body = nil
+      @charset = @@default_charset.dup
+      @encode_subject = @@encode_subject
       @headers = {}
     end
 
@@ -104,14 +117,22 @@ module ActionMailer #:nodoc:
         end        
       end
 
-      def mail(to, subject, body, from, timestamp = nil, headers = nil) #:nodoc:
-        deliver(create(to, subject, body, from, timestamp, headers))
+      def mail(to, subject, body, from, timestamp = nil, headers = {},
+               encode = @@encode_subject, charset = @@default_charset
+      ) #:nodoc:
+        deliver(create(to, subject, body, from, timestamp, headers, charset))
       end
 
-      def create(to, subject, body, from, timestamp = nil, headers = nil) #:nodoc:
+      def create(to, subject, body, from, timestamp = nil, headers = {},
+                 encode = @@encode_subject, charset = @@default_charset
+      ) #:nodoc:
         m = TMail::Mail.new
-        m.to, m.subject, m.body, m.from = to, subject, body, from
+        m.to, m.subject, m.body, m.from = to,
+          ( encode ? quoted_printable(subject,charset) : subject ), body, from
         m.date = timestamp.respond_to?("to_time") ? timestamp.to_time : (timestamp || Time.now)    
+
+        m.set_content_type "text", "plain", { "charset" => charset }
+
         headers.each do |k, v|
           m[k] = v
         end
@@ -124,7 +145,12 @@ module ActionMailer #:nodoc:
         send("perform_delivery_#{delivery_method}", mail) if perform_deliveries
       end
 
-      private      
+      def quoted_printable(text, charset)#:nodoc:
+        text = text.gsub( /[^a-z ]/i ) { "=%02x" % $&[0] }.gsub( / /, "_" )
+        "=?#{charset}?Q?#{text}?="
+      end
+
+      private
         def perform_delivery_smtp(mail)
           Net::SMTP.start(server_settings[:address], server_settings[:port], server_settings[:domain], 
               server_settings[:user_name], server_settings[:password], server_settings[:authentication]) do |smtp|
@@ -153,7 +179,8 @@ module ActionMailer #:nodoc:
           end
 
           mail = create(mailer.recipients, mailer.subject, mailer.body,
-                        mailer.from, mailer.sent_on, mailer.headers)
+                        mailer.from, mailer.sent_on, mailer.headers,
+                        mailer.encode_subject, mailer.charset)
 
           mail.bcc = mailer.bcc unless mailer.bcc.nil?
           mail.cc  = mailer.cc  unless mailer.cc.nil?

data/rakefile

@@ -8,7 +8,7 @@ require 'rake/contrib/rubyforgepublisher'
 
 PKG_BUILD     = ENV['PKG_BUILD'] ? '.' + ENV['PKG_BUILD'] : ''
 PKG_NAME      = 'actionmailer'
-PKG_VERSION   = '0.6.1' + PKG_BUILD
+PKG_VERSION   = '0.7.0' + PKG_BUILD
 PKG_FILE_NAME = "#{PKG_NAME}-#{PKG_VERSION}"
 
 desc "Default Task"
@@ -48,9 +48,9 @@ spec = Gem::Specification.new do |s|
   s.author = "David Heinemeier Hansson"
   s.email = "david@loudthinking.com"
   s.rubyforge_project = "actionmailer"
-  s.homepage = "http://actionmailer.rubyonrails.org"
+  s.homepage = "http://www.rubyonrails.org"
 
-  s.add_dependency('actionpack', '>= 0.9.5')
+  s.add_dependency('actionpack', '= 1.5.0' + PKG_BUILD)
 
   s.has_rdoc = true
   s.requirements << 'none'

data/test/mail_service_test.rb

@@ -31,12 +31,46 @@ class TestMailer < ActionMailer::Base
     @body       = "Nothing to see here."
   end
 
+  def iso_charset(recipient)
+    @recipients = recipient
+    @subject    = "testing iso charsets"
+    @from       = "system@loudthinking.com"
+    @sent_on    = Time.local 2004, 12, 12
+    @cc         = "nobody@loudthinking.com"
+    @bcc        = "root@loudthinking.com"
+    @body       = "Nothing to see here."
+    @charset    = "iso-8859-1"
+  end
+
+  def unencoded_subject(recipient)
+    @recipients = recipient
+    @subject    = "testing unencoded subject"
+    @from       = "system@loudthinking.com"
+    @sent_on    = Time.local 2004, 12, 12
+    @cc         = "nobody@loudthinking.com"
+    @bcc        = "root@loudthinking.com"
+    @body       = "Nothing to see here."
+    @encode_subject = false
+  end
+
 end
 
 TestMailer.template_root = File.dirname(__FILE__) + "/fixtures"
 
 class ActionMailerTest < Test::Unit::TestCase
 
+  def encode( text, charset="utf-8" )
+    ActionMailer::Base.quoted_printable( text, charset )
+  end
+
+  def new_mail( charset="utf-8" )
+    mail = TMail::Mail.new
+    if charset
+      mail.set_content_type "text", "plain", { "charset" => charset }
+    end
+    mail
+  end
+
   def setup
     ActionMailer::Base.delivery_method = :test
     ActionMailer::Base.perform_deliveries = true
@@ -46,9 +80,9 @@ class ActionMailerTest < Test::Unit::TestCase
   end
 
   def test_signed_up
-    expected = TMail::Mail.new
+    expected = new_mail
     expected.to      = @recipient
-    expected.subject = "[Signed up] Welcome #{@recipient}"
+    expected.subject = encode "[Signed up] Welcome #{@recipient}"
     expected.body    = "Hello there, \n\nMr. #{@recipient}"
     expected.from    = "system@loudthinking.com"
     expected.date    = Time.local(2004, 12, 12)
@@ -64,9 +98,9 @@ class ActionMailerTest < Test::Unit::TestCase
   end
   
   def test_cancelled_account
-    expected = TMail::Mail.new
+    expected = new_mail
     expected.to      = @recipient
-    expected.subject = "[Cancelled] Goodbye #{@recipient}"
+    expected.subject = encode "[Cancelled] Goodbye #{@recipient}"
     expected.body    = "Goodbye, Mr. #{@recipient}"
     expected.from    = "system@loudthinking.com"
     expected.date    = Time.local(2004, 12, 12)
@@ -82,9 +116,9 @@ class ActionMailerTest < Test::Unit::TestCase
   end
   
   def test_cc_bcc
-    expected = TMail::Mail.new
+    expected = new_mail
     expected.to      = @recipient
-    expected.subject = "testing bcc/cc"
+    expected.subject = encode "testing bcc/cc"
     expected.body    = "Nothing to see here."
     expected.from    = "system@loudthinking.com"
     expected.cc      = "nobody@loudthinking.com"
@@ -106,6 +140,56 @@ class ActionMailerTest < Test::Unit::TestCase
     assert_equal expected.encoded, ActionMailer::Base.deliveries.first.encoded
   end
 
+  def test_iso_charset
+    expected = new_mail( "iso-8859-1" )
+    expected.to      = @recipient
+    expected.subject = encode "testing iso charsets", "iso-8859-1"
+    expected.body    = "Nothing to see here."
+    expected.from    = "system@loudthinking.com"
+    expected.cc      = "nobody@loudthinking.com"
+    expected.bcc     = "root@loudthinking.com"
+    expected.date    = Time.local 2004, 12, 12
+
+    created = nil
+    assert_nothing_raised do
+      created = TestMailer.create_iso_charset @recipient
+    end
+    assert_not_nil created
+    assert_equal expected.encoded, created.encoded
+
+    assert_nothing_raised do
+      TestMailer.deliver_iso_charset @recipient
+    end
+
+    assert_not_nil ActionMailer::Base.deliveries.first
+    assert_equal expected.encoded, ActionMailer::Base.deliveries.first.encoded
+  end
+
+  def test_unencoded_subject
+    expected = new_mail
+    expected.to      = @recipient
+    expected.subject = "testing unencoded subject"
+    expected.body    = "Nothing to see here."
+    expected.from    = "system@loudthinking.com"
+    expected.cc      = "nobody@loudthinking.com"
+    expected.bcc     = "root@loudthinking.com"
+    expected.date    = Time.local 2004, 12, 12
+
+    created = nil
+    assert_nothing_raised do
+      created = TestMailer.create_unencoded_subject @recipient
+    end
+    assert_not_nil created
+    assert_equal expected.encoded, created.encoded
+
+    assert_nothing_raised do
+      TestMailer.deliver_unencoded_subject @recipient
+    end
+
+    assert_not_nil ActionMailer::Base.deliveries.first
+    assert_equal expected.encoded, ActionMailer::Base.deliveries.first.encoded
+  end
+
   def test_instances_are_nil
     assert_nil ActionMailer::Base.new
     assert_nil TestMailer.new

metadata

@@ -3,13 +3,13 @@ rubygems_version: 0.8.4
 specification_version: 1
 name: actionmailer
 version: !ruby/object:Gem::Version 
-  version: 0.6.1
-date: 2005-01-18
+  version: 0.7.0
+date: 2005-02-24
 summary: Service layer for easy email delivery and testing.
 require_paths: 
   - lib
 email: david@loudthinking.com
-homepage: http://actionmailer.rubyonrails.org
+homepage: http://www.rubyonrails.org
 rubyforge_project: actionmailer
 description: Makes it trivial to test and deliver emails sent from a single service layer.
 autorequire: action_mailer
@@ -81,7 +81,7 @@ dependencies:
     version_requirements: !ruby/object:Gem::Version::Requirement 
       requirements: 
         - 
-          - ">="
+          - "="
           - !ruby/object:Gem::Version 
-            version: 0.9.5
+            version: 1.5.0
       version: