OpenNeo/impress
OpenNeo/impress
1
0
Fork 1
Code Pull requests Activity

Use strong parameters for User

Browse source
This commit is contained in:
Matchu 2023-07-29 11:07:14 -07:00 committed by Matchu
parent 523137253c
commit 248e710fcb
2 changed files with 6 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
app/controllers/users_controller.rb
View file

@ -17,7 +17,7 @@ class UsersController < ApplicationController
end end
def update def update
success = @user.update_attributes params[:user] success = @user.update_attributes user_params
respond_to do |format| respond_to do |format|
format.html { format.html {
if success if success
@ -41,6 +41,11 @@ class UsersController < ApplicationController
protected protected
def user_params
params.require(:user).permit(:owned_closet_hangers_visibility,
:wanted_closet_hangers_visibility, :contact_neopets_connection_id)
end
def find_and_authorize_user! def find_and_authorize_user!
if current_user.id == params[:id].to_i if current_user.id == params[:id].to_i
@user = current_user @user = current_user

3
app/models/user.rb
View file

@ -21,9 +21,6 @@ class User < ActiveRecord::Base
devise :rememberable devise :rememberable
attr_accessible :owned_closet_hangers_visibility,
:wanted_closet_hangers_visibility, :contact_neopets_connection_id
def admin? def admin?
name == 'matchu' # you know that's right. name == 'matchu' # you know that's right.
end end