Set the asset host for person notifier mailing

Victor Costa
1 parent 2bdb49ac
Showing 3 changed files with 3 additions and 39 deletions Show diff stats
app/helpers/person_notifier_helper.rb
app/models/person_notifier.rb
test/unit/person_notifier_helper_test.rb
@@ -1,15 +0,0 @@
-module PersonNotifierHelper
-
-  include ApplicationHelper
-
-  private
-
-  def path_to_image(source)
-    top_url + source
-  end
-
-  def top_url
-    top_url = @profile.environment ? @profile.environment.top_url : ''
-  end
-
-end
@@ -67,13 +67,15 @@ class PersonNotifier
  
   class Mailer < ActionMailer::Base
  
-    add_template_helper(PersonNotifierHelper)
+    add_template_helper(ApplicationHelper)
  
     def session
       {:theme => nil}
     end
  
     def content_summary(person, notifications)
+      ActionMailer::Base.asset_host = person.environment.top_url if person.environment
+
       @current_theme = 'default'
       @profile = person
       @recipient = @profile.nickname || @profile.name
@@ -1,23 +0,0 @@
-require File.dirname(__FILE__) + '/../test_helper'
-
-class PersonNotifierHelperTest < ActionView::TestCase
-
-  include PersonNotifierHelper
-
-  def setup
-    @profile = mock
-    @env = Environment.new
-  end
-  attr_reader :profile, :env
-
-  should 'append top url of environment at image path' do
-    profile.expects(:environment).returns(env).at_least_once
-    assert_match /src="http:\/\/localhost\/image.png"/, image_tag("/image.png")
-  end
-
-  should 'return original path if do not have an environment' do
-    profile.expects(:environment).returns(nil).at_least_once
-    assert_match /src="\/image.png"/, image_tag("/image.png")
-  end
-
-end
...	...	@@ -1,15 +0,0 @@
1		-module PersonNotifierHelper
2		-
3		- include ApplicationHelper
4		-
5		- private
6		-
7		- def path_to_image(source)
8		- top_url + source
9		- end
10		-
11		- def top_url
12		- top_url = @profile.environment ? @profile.environment.top_url : ''
13		- end
14		-
15		-end
...	...	@@ -67,13 +67,15 @@ class PersonNotifier
67	67
68	68	class Mailer < ActionMailer::Base
69	69
70		- add_template_helper(PersonNotifierHelper)
	70	+ add_template_helper(ApplicationHelper)
71	71
72	72	def session
73	73	{:theme => nil}
74	74	end
75	75
76	76	def content_summary(person, notifications)
	77	+ ActionMailer::Base.asset_host = person.environment.top_url if person.environment
	78	+
77	79	@current_theme = 'default'
78	80	@profile = person
79	81	@recipient = @profile.nickname \|\| @profile.name
...	...
...	...	@@ -1,23 +0,0 @@
1		-require File.dirname(__FILE__) + '/../test_helper'
2		-
3		-class PersonNotifierHelperTest < ActionView::TestCase
4		-
5		- include PersonNotifierHelper
6		-
7		- def setup
8		- @profile = mock
9		- @env = Environment.new
10		- end
11		- attr_reader :profile, :env
12		-
13		- should 'append top url of environment at image path' do
14		- profile.expects(:environment).returns(env).at_least_once
15		- assert_match /src="http:\/\/localhost\/image.png"/, image_tag("/image.png")
16		- end
17		-
18		- should 'return original path if do not have an environment' do
19		- profile.expects(:environment).returns(nil).at_least_once
20		- assert_match /src="\/image.png"/, image_tag("/image.png")
21		- end
22		-
23		-end