Download as
Browse Code »

Commit fdeb05ebc8bb9ef4a2dd30490152dbb5ed953987

Authored by AntonioTerceiro
1 parent 9b43a97d
Exists in master and in 29 other branches add_member_task_reject_details, admin_visible_profile, article-list-template, community_notifications, contact_admin_translation, event_fixes, fix_comments_pagination, fix_rails4_organization_ratings, fix_sign_up_form, follow_step_fix, forum_topic_creation, mirror_block_improvements, multi_env_on_remote_user, new_security, noosfero_spb_ci, organization_ratings_improvements, organization_ratings_link_to_profile_stable-spb-1.3, organization_ratings_translations_fix, profile_api_improvements, ratings_minor_fixes, remote_user_fix, send_email_to_admins, stable-spb-1.3, stable-spb-1.3-fixes, stable-spb-1.4, stable-spb-1.5, suggest_rejected_value, web_steps_improvements, xss_terminate_custom_options

ActionItem18: creating a proper list on RecentDocumentsBlock 



git-svn-id: https://svn.colivre.coop.br/svn/noosfero/trunk@433 3f533792-8f58-4932-b0fe-aaf55b0a4547
Showing 2 changed files with 4 additions and 1 deletions   Show diff stats
app/models/recent_documents_block.rb
  Diff comments   View file @fdeb05e
@@ -4,7 +4,7 @@ class RecentDocumentsBlock < Design::Block @@ -4,7 +4,7 @@ class RecentDocumentsBlock < Design::Block
4 lambda do 4 lambda do
5 content_tag( 5 content_tag(
6 'ul', 6 'ul',
7 - profile.recent_documents.map {|item| link_to_document(item) }.join("\n") 7 + profile.recent_documents.map {|item| content_tag('li', link_to_document(item)) }.join("\n")
8 ) 8 )
9 end 9 end
10 end 10 end
test/unit/recent_documents_block_test.rb
  Diff comments   View file @fdeb05e
@@ -15,8 +15,11 @@ class RecentDocumentsBlockTest < Test::Unit::TestCase @@ -15,8 +15,11 @@ class RecentDocumentsBlockTest < Test::Unit::TestCase
15 helper = mock 15 helper = mock
16 helper.expects(:profile).returns(profile) 16 helper.expects(:profile).returns(profile)
17 helper.expects(:link_to_document).with(doc1).returns('doc1') 17 helper.expects(:link_to_document).with(doc1).returns('doc1')
  18 + helper.expects(:content_tag).with('li', 'doc1').returns('doc1')
18 helper.expects(:link_to_document).with(doc2).returns('doc2') 19 helper.expects(:link_to_document).with(doc2).returns('doc2')
  20 + helper.expects(:content_tag).with('li', 'doc2').returns('doc2')
19 helper.expects(:link_to_document).with(doc3).returns('doc3') 21 helper.expects(:link_to_document).with(doc3).returns('doc3')
  22 + helper.expects(:content_tag).with('li', 'doc3').returns('doc3')
20 helper.expects(:content_tag).with('ul', "doc1\ndoc2\ndoc3").returns('the_tag') 23 helper.expects(:content_tag).with('ul', "doc1\ndoc2\ndoc3").returns('the_tag')
21 24
22 assert_equal('the_tag', helper.instance_eval(&RecentDocumentsBlock.new.content)) 25 assert_equal('the_tag', helper.instance_eval(&RecentDocumentsBlock.new.content))