diff options
| author | Eugen Rochko <eugen@zeonfederated.com> | 2022-08-25 20:39:40 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-08-25 20:39:40 +0200 |
| commit | 0396acf39ea902688374fac65fa7ef5dc4c05512 (patch) | |
| tree | f5349303e930976f8accd22453dd4bc5024c32f5 /app/controllers | |
| parent | 99aed9069d4319d53779c483142e6649f3fef17b (diff) | |
Add audit log entries for user roles (#19040)
* Refactor audit log schema * Add audit log entries for user roles
Diffstat (limited to 'app/controllers')
| -rw-r--r-- | app/controllers/admin/roles_controller.rb | 3 | ||||
| -rw-r--r-- | app/controllers/admin/users/roles_controller.rb | 1 | ||||
| -rw-r--r-- | app/controllers/concerns/accountable_concern.rb | 8 |
3 files changed, 10 insertions, 2 deletions
diff --git a/app/controllers/admin/roles_controller.rb b/app/controllers/admin/roles_controller.rb index 3e502ccc4..d76aa745b 100644 --- a/app/controllers/admin/roles_controller.rb +++ b/app/controllers/admin/roles_controller.rb @@ -23,6 +23,7 @@ module Admin @role.current_account = current_account if @role.save + log_action :create, @role redirect_to admin_roles_path else render :new @@ -39,6 +40,7 @@ module Admin @role.current_account = current_account if @role.update(resource_params) + log_action :update, @role redirect_to admin_roles_path else render :edit @@ -48,6 +50,7 @@ module Admin def destroy authorize @role, :destroy? @role.destroy! + log_action :destroy, @role redirect_to admin_roles_path end diff --git a/app/controllers/admin/users/roles_controller.rb b/app/controllers/admin/users/roles_controller.rb index 0db50cee9..f5dfc643d 100644 --- a/app/controllers/admin/users/roles_controller.rb +++ b/app/controllers/admin/users/roles_controller.rb @@ -14,6 +14,7 @@ module Admin @user.current_account = current_account if @user.update(resource_params) + log_action :change_role, @user redirect_to admin_account_path(@user.account_id), notice: I18n.t('admin.accounts.change_role.changed_msg') else render :show diff --git a/app/controllers/concerns/accountable_concern.rb b/app/controllers/concerns/accountable_concern.rb index 87d62478d..c1349915f 100644 --- a/app/controllers/concerns/accountable_concern.rb +++ b/app/controllers/concerns/accountable_concern.rb @@ -3,7 +3,11 @@ module AccountableConcern extend ActiveSupport::Concern - def log_action(action, target, options = {}) - Admin::ActionLog.create(account: current_account, action: action, target: target, recorded_changes: options.stringify_keys) + def log_action(action, target) + Admin::ActionLog.create( + account: current_account, + action: action, + target: target + ) end end |