diff --git a/app/controllers/users/settings/teams_controller.rb b/app/controllers/users/settings/teams_controller.rb index 43b55723a..dce1104b2 100644 --- a/app/controllers/users/settings/teams_controller.rb +++ b/app/controllers/users/settings/teams_controller.rb @@ -28,8 +28,6 @@ module Users members ) - before_action :check_read_permissions, only: :show - before_action :check_create_team_permission, only: %i(new create) @@ -140,10 +138,6 @@ module Users render_403 unless can_create_teams? end - def check_read_permissions - render_403 unless can_read_team?(@team) - end - def load_user @user = current_user end diff --git a/app/controllers/users/settings/user_groups_controller.rb b/app/controllers/users/settings/user_groups_controller.rb index 92dc0c1e5..845b27ec5 100644 --- a/app/controllers/users/settings/user_groups_controller.rb +++ b/app/controllers/users/settings/user_groups_controller.rb @@ -7,8 +7,8 @@ module Users before_action :set_breadcrumbs_items, only: %i(index show) before_action :check_user_groups_enabled before_action :load_user_group, except: %i(index unassigned_users actions_toolbar create) - before_action :check_read_permissions, only: %i(index show unassigned_users actions_toolbar users) - before_action :check_manage_permissions, except: %i(index show unassigned_users actions_toolbar users) + before_action :check_read_permissions, only: %i(users) + before_action :check_manage_permissions, except: %i(users) def index respond_to do |format| diff --git a/app/views/users/settings/teams/_header.html.erb b/app/views/users/settings/teams/_header.html.erb index d0b90bdf4..2dcd619be 100644 --- a/app/views/users/settings/teams/_header.html.erb +++ b/app/views/users/settings/teams/_header.html.erb @@ -18,7 +18,7 @@
- <%= link_to t("users.settings.teams.navigation.details"), team_path(@team), class: "p-2.5 hover:no-underline #{ 'disabled' unless can_read_team?(@team) } #{ @active_tab == :details ? "text-sn-blue" : "text-sn-grey" }"%> + <%= link_to t("users.settings.teams.navigation.details"), team_path(@team), class: "p-2.5 hover:no-underline #{ @active_tab == :details ? "text-sn-blue" : "text-sn-grey" }"%> <%= link_to t("users.settings.teams.navigation.members"), members_users_settings_team_path(@team), class: "p-2.5 hover:no-underline #{ @active_tab == :members ? "text-sn-blue" : "text-sn-grey" }"%> - <%= link_to t("users.settings.teams.navigation.groups"), users_settings_team_user_groups_path(@team), class: "p-2.5 hover:no-underline #{ 'disabled' unless can_read_team?(@team) } #{ @active_tab == :user_groups ? "text-sn-blue" : "text-sn-grey" }"%> + <%= link_to t("users.settings.teams.navigation.groups"), users_settings_team_user_groups_path(@team), class: "p-2.5 hover:no-underline #{ 'disabled' unless can_manage_team?(@team) } #{ @active_tab == :user_groups ? "text-sn-blue" : "text-sn-grey" }"%>