pundit: convert passwords to pundit.
This commit is contained in:
@@ -1,31 +1,20 @@
|
||||
class PasswordsController < ApplicationController
|
||||
before_action :member_only
|
||||
respond_to :html, :xml, :json
|
||||
|
||||
def edit
|
||||
@user = User.find(params[:user_id])
|
||||
check_privilege(@user)
|
||||
|
||||
@user = authorize User.find(params[:user_id]), policy_class: PasswordPolicy
|
||||
respond_with(@user)
|
||||
end
|
||||
|
||||
def update
|
||||
@user = User.find(params[:user_id])
|
||||
check_privilege(@user)
|
||||
|
||||
@user = authorize User.find(params[:user_id]), policy_class: PasswordPolicy
|
||||
@user.update(user_params)
|
||||
flash[:notice] = @user.errors.none? ? "Password updated" : @user.errors.full_messages.join("; ")
|
||||
|
||||
respond_with(@user, location: @user)
|
||||
end
|
||||
|
||||
private
|
||||
|
||||
def check_privilege(user)
|
||||
raise User::PrivilegeError unless user.id == CurrentUser.id || CurrentUser.is_admin?
|
||||
end
|
||||
|
||||
def user_params
|
||||
params.require(:user).permit(%i[signed_user_id old_password password password_confirmation])
|
||||
params.fetch(:user, {}).permit(policy(:password).permitted_attributes)
|
||||
end
|
||||
end
|
||||
|
||||
9
app/policies/password_policy.rb
Normal file
9
app/policies/password_policy.rb
Normal file
@@ -0,0 +1,9 @@
|
||||
class PasswordPolicy < ApplicationPolicy
|
||||
def update?
|
||||
record.id == user.id || user.is_admin?
|
||||
end
|
||||
|
||||
def permitted_attributes
|
||||
[:signed_user_id, :old_password, :password, :password_confirmation]
|
||||
end
|
||||
end
|
||||
Reference in New Issue
Block a user