Commit 30f99608 authored by Jose Ivan Vargas's avatar Jose Ivan Vargas

Fixed some missing permission conditions

parent f4794659
- can_admin_group = can?(current_user, :admin_group, @group)
- can_edit = can?(current_user, :admin_group, @group)
= content_for :sub_nav do = content_for :sub_nav do
.scrolling-tabs-container.sub-nav-scroll .scrolling-tabs-container.sub-nav-scroll
= render 'shared/nav_scroll' = render 'shared/nav_scroll'
...@@ -8,7 +10,8 @@ ...@@ -8,7 +10,8 @@
%span %span
Projects Projects
= nav_link(path: 'groups#edit') do - if can_edit && can_admin_group
= link_to edit_group_path(@group), title: 'Edit Group' do = nav_link(path: 'groups#edit') do
%span = link_to edit_group_path(@group), title: 'Edit Group' do
Edit Group %span
Edit Group
- can_admin_group = can?(current_user, :admin_group, @group) - can_admin_group = can?(current_user, :admin_group, @group)
- can_edit = can?(current_user, :admin_group, @group)
.scrolling-tabs-container{ class: nav_control_class } .scrolling-tabs-container{ class: nav_control_class }
.fade-left .fade-left
= icon('angle-left') = icon('angle-left')
...@@ -26,9 +25,8 @@ ...@@ -26,9 +25,8 @@
= link_to group_group_members_path(@group), title: 'Members' do = link_to group_group_members_path(@group), title: 'Members' do
%span %span
Members Members
- if current_user - if current_user && can_admin_group
- if can_admin_group || can_edit = nav_link(path: %w[groups#projects groups#edit]) do
= nav_link(path: %w[groups#projects groups#edit]) do = link_to projects_group_path(@group), title: 'Settings' do
= link_to projects_group_path(@group), title: 'Settings' do %span
%span Settings
Settings
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment