Download as
Browse Code »

Commit fe6c058f9a655b5378bb5a3758e1eb687cbbad4c

Authored by Antonio Terceiro
1 parent 3bf272b1
Exists in master and in 27 other branches 1.2+SPB2, 2_split_by_role, add_fields_in_api, api_fixies, api_pagination, api_private_token, api_tasks, article_hit_count, blog_page_bug, checkbox_to_user_can_edit_page, communities_ratings, communities_ratings_moderation, community_ratings_rebase, fix_cloned_article_parent, fix_gallery_image_url, fix_javascript_categories, fix_optional_fields, fix_suggest_article, fix_user_email_validation, highlight_block_fix, notification_plugin, notification_plugin_rebase, script_to_update, split_by_role, temp_ratings, tests, tests_fixies

Remove dead code

Showing 3 changed files with 1 additions and 39 deletions   Show diff stats
app/helpers/person_notifier_helper.rb
View file @3bf272b
@@ -1,15 +0,0 @@ @@ -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  
app/models/person_notifier.rb
  Diff comments   View file @fe6c058
@@ -67,7 +67,7 @@ class PersonNotifier @@ -67,7 +67,7 @@ class PersonNotifier
67 67
68 class Mailer < ActionMailer::Base 68 class Mailer < ActionMailer::Base
69 69
70 - add_template_helper(PersonNotifierHelper) 70 + add_template_helper(ApplicationHelper)
71 71
72 def session 72 def session
73 {:theme => nil} 73 {:theme => nil}
test/unit/person_notifier_helper_test.rb
View file @3bf272b
@@ -1,23 +0,0 @@ @@ -1,23 +0,0 @@
1 -require_relative "../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