Download as
Browse Code »

Commit afcbe97285bdb5fc85c5f2f7bcb690fef9a1b3eb

Authored by AntonioTerceiro
1 parent 179d8266
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

ActionItem457: fixing footer for environment 

don't generate "All content" links when the "Recent Documents" block is
owned by an environment environment


git-svn-id: https://svn.colivre.coop.br/svn/noosfero/trunk@2047 3f533792-8f58-4932-b0fe-aaf55b0a4547
Showing 2 changed files with 10 additions and 0 deletions   Show diff stats
app/models/recent_documents_block.rb
  Diff comments   View file @afcbe97
@@ -16,6 +16,8 @@ class RecentDocumentsBlock < Block @@ -16,6 +16,8 @@ class RecentDocumentsBlock < Block
16 end 16 end
17 17
18 def footer 18 def footer
  19 + return nil unless self.owner.is_a?(Profile)
  20 +
19 profile = self.owner 21 profile = self.owner
20 lambda do 22 lambda do
21 link_to _('All content'), :profile => profile.identifier, :controller => 'profile', :action => 'sitemap' 23 link_to _('All content'), :profile => profile.identifier, :controller => 'profile', :action => 'sitemap'
test/unit/recent_documents_block_test.rb
  Diff comments   View file @afcbe97
@@ -62,4 +62,12 @@ class RecentDocumentsBlockTest < Test::Unit::TestCase @@ -62,4 +62,12 @@ class RecentDocumentsBlockTest < Test::Unit::TestCase
62 instance_eval(&(block.footer)) 62 instance_eval(&(block.footer))
63 end 63 end
64 64
  65 + should 'not display link to sitemap when owner is environment' do
  66 + block = RecentDocumentsBlock.new
  67 + box = mock
  68 + block.expects(:box).returns(box).at_least_once
  69 + box.expects(:owner).returns(Environment.new).at_least_once
  70 + assert_equal nil, block.footer
  71 + end
  72 +
65 end 73 end