Download as
Browse Code »

Commit eb4178406f1c00d43543884c734efe486fe66c7b

Authored by Daniela Feitosa
1 parent 4dd11514
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

Remove "Manage Roles" option for person 

Only organizations can manage roles
Showing 2 changed files with 7 additions and 1 deletions   Show diff stats
app/views/profile_editor/index.html.erb
  Diff comments   View file @eb41784
@@ -28,7 +28,7 @@ @@ -28,7 +28,7 @@
28 28
29 <%= control_panel_button(_('Manage Content'), 'cms', :controller => 'cms') %> 29 <%= control_panel_button(_('Manage Content'), 'cms', :controller => 'cms') %>
30 30
31 - <%= control_panel_button(_('Manage Roles'), 'roles', :controller => 'profile_roles') %> 31 + <%= control_panel_button(_('Manage Roles'), 'roles', :controller => 'profile_roles') if profile.organization? %>
32 32
33 <% unless profile.enterprise? %> 33 <% unless profile.enterprise? %>
34 <%= case profile.blogs.count 34 <%= case profile.blogs.count
test/functional/profile_editor_controller_test.rb
  Diff comments   View file @eb41784
@@ -1210,4 +1210,10 @@ class ProfileEditorControllerTest &lt; ActionController::TestCase @@ -1210,4 +1210,10 @@ class ProfileEditorControllerTest &lt; ActionController::TestCase
1210 get :header_footer, :profile => user.identifier 1210 get :header_footer, :profile => user.identifier
1211 assert_response :success 1211 assert_response :success
1212 end 1212 end
  1213 +
  1214 + should 'not display button to manage roles on control panel of person' do
  1215 + get :index, :profile => profile.identifier
  1216 + assert_no_tag :tag => 'a', :attributes => { :href => "/myprofile/default_user/profile_roles" }
  1217 + end
  1218 +
1213 end 1219 end