Skip to content

Commit 4b40b94

Browse files
committed
Merge branch 'fix/user_permission_removed_on_edit' into staging
2 parents 9276fbf + 8aa4fe3 commit 4b40b94

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

app/admin/user.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
qc1_observer_ids: [], qc2_observer_ids: [],
1111
managed_observer_ids: [],
1212
responsible_for_country_ids: [],
13-
user_permission_attributes: [:user_role]
13+
user_permission_attributes: [:id, :user_role]
1414

1515
filter :country, as: :select, collection: -> { Country.joins(:users).by_name_asc }
1616
filter :operator

app/controllers/application_controller.rb

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,9 @@ def user_for_paper_trail
4343
private
4444

4545
def backoffice_user?
46-
current_user.user_permission.present? && current_user.is_active &&
47-
%w[admin bo_manager].include?(current_user.user_permission.user_role)
46+
current_user.is_active && %w[admin bo_manager].include?(current_user.user_permission.user_role)
47+
rescue => e
48+
Sentry.capture_exception(e)
49+
raise SecurityError
4850
end
4951
end

0 commit comments

Comments
 (0)