diff options
| author | mo khan <mo@mokhan.ca> | 2015-11-14 21:19:58 -0700 |
|---|---|---|
| committer | mo khan <mo@mokhan.ca> | 2015-11-14 21:19:58 -0700 |
| commit | d311e696b9689c8c8eaf8eb13dd6773243eba2c8 (patch) | |
| tree | f812771f88d02e4d9fcf4073f3026dc4c5b2ff3e /app/controllers/profiles_controller.rb | |
| parent | f28ddb69d7cca6bce073d82391bca5a323db0d82 (diff) | |
remove redundant code from profiles controller.
Diffstat (limited to 'app/controllers/profiles_controller.rb')
| -rw-r--r-- | app/controllers/profiles_controller.rb | 30 |
1 files changed, 11 insertions, 19 deletions
diff --git a/app/controllers/profiles_controller.rb b/app/controllers/profiles_controller.rb index 185b218..9abd1b4 100644 --- a/app/controllers/profiles_controller.rb +++ b/app/controllers/profiles_controller.rb @@ -1,33 +1,25 @@ class ProfilesController < ApplicationController - def show @user = User.find_by(username: params[:id]) @profile = @user.profile @program = Program.stronglifts end - + def edit - @user = @current_user - @profile = @user.profile + @profile = @current_user.profile @program = Program.stronglifts end - + def update - if @current_user - @profile = @current_user.profile - @profile.update_attributes(profile_params) - flash[:notice] = t("profiles.edit.profile_update_success") - redirect_to profile_path(@profile) - else - flash[:notice] = t("profiles.edit.profile_update_error") - render 'edit' - end + profile = @current_user.profile + profile.update_attributes(profile_params) + flash[:notice] = t("profiles.edit.profile_update_success") + redirect_to profile_path(profile) end - + private - def profile_params - params.require(:profile).permit(:gender, :social_tolerance) - end - + def profile_params + params.require(:profile).permit(:gender, :social_tolerance) + end end |
