質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.91%

【Rails】Devise アカウント編集 パスワードなしで更新

受付中

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 503
退会済みユーザー

退会済みユーザー

RailsのDeviseを使って、アカウント編集の実装を行っています。
以下の内容を参考に実装しているのですが、現在のパスワードの入力は不要になりましたが、毎回パスワード変更を行わないと正しく更新されないようになってしまいます。
https://kossy-web-engineer.hatenablog.com/entry/2018/11/06/102047

イメージ説明
上記のようにパスワードの変更を行わないとエラーが起きてしまいます。
https://kossy-web-engineer.hatenablog.com/entry/2018/11/06/102047

#registrations/edit.html.haml(現在のパスワードは入力なしでも更新できます。)

.useredits
    .useredits-line
      .useredits-line_prof
        .useredits-line_prof_main
          .useredits-line_prof_main_user
            .useredits-line_prof_main_user_into
              -# %h2.useredits-line_prof_main_user_into_acount
              -#   アカウント編集
                -# Edit #{resource_name.to_s.humanize}
              = form_for(resource, as: resource_name, url: registration_path(resource_name), html: { method: :put }) do |f|
                = render "devise/shared/error_messages", resource: resource
                .edituserid
                  .edituserid_sub
                    %h2 ユーザID
                  .edituserid_main
                    = f.text_field :useruserid, autofocus: true, placeholder: "ユーザID"
                .editusername
                  .editusername_sub
                    = f.label :ユーザ名
                  .editusername_main
                    = f.text_field :name, autofocus: true, placeholder: "ユーザ名"
                .editusermail
                  .editusermail_sub
                    = f.label :メールアドレス
                  .editusermail_main  
                    = f.email_field :email, autofocus: true, autocomplete: "email", placeholder: "メールアドレス"
                - if devise_mapping.confirmable? && resource.pending_reconfirmation?
                  %div
                    Currently waiting confirmation for: #{resource.unconfirmed_email}
                  .edituserform_password
                    .edituserform_password_sub
                      = f.label :パスワード
                    .edituserform_password_main
                      = f.password_field :password, autocomplete: "new-password", placeholder: "パスワード"
                  .edituserform_passconf
                    .edituserform_passconf_sub
                      = f.label :パスワード確認
                    .edituserform_passconf_main 
                      = f.password_field :password_confirmation, autocomplete: "new-password", placeholder: "パスワード確認"
                  .edituserform_passcurent
                    .edituserform_passcurent_sub
                      = f.label :現在のパスワード
                    .edituserform_passcurent_main
                      = f.password_field :current_password, autocomplete: "current-password", placeholder: "現在のパスワード"
                  .edituserform_submit  
                    .edituserform_submit_sub
                      = f.submit "更新", { :class => 'submit-btnaa' }


#user.rb

class User < ApplicationRecord
  # Include default devise modules. Others available are:
  # :confirmable, :lockable, :timeoutable, :trackable and :omniauthable
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable

         validates :name,:useruserid, presence: true #追記
        #  @([a-zA-Z0-9_-])
         mount_uploader :image, ImageUploader
         validates :useruserid,           presence: true, uniqueness: true, format: { with: /\A@[\w+\-.]+\z/i }
         validates :email,           presence: true, uniqueness: true, format: { with: /\A[\w+\-.]+@[a-z\d\-.]+\.[a-z]+\z/i }
         validates :password,        presence: true, format: { with: /\A(?=.*?[a-z])(?=.*?\d)[a-z\d]{6,40}+\z/i }
  # validates :profile, length: { maximum: 200 } #追記
  has_many :comments
  has_many :tweets, dependent: :destroy
  has_many :likes, dependent: :destroy
  has_many :liked_tweets, through: :likes, source: :tweet

  has_many :following_relationships, foreign_key: "follower_id", class_name: "Relationship", dependent: :destroy

  has_many :followings, through: :following_relationships

  has_many :follower_relationships, foreign_key: "following_id", class_name: "Relationship", dependent: :destroy

  has_many :followers, through: :follower_relationships

  # has_many :temps

  def following?(other_user)
    following_relationships.find_by(following_id: other_user.id)
  end

  def follow!(other_user)
    following_relationships.create!(following_id: other_user.id)
  end

  def unfollow!(other_user)
    following_relationships.find_by(following_id: other_user.id).destroy
  end

  def already_liked?(tweet)
    self.likes.exists?(tweet_id: tweet.id)
  end
  def self.search(search)
    if search
      User.where('name LIKE(?) or sex LIKE(?) ',  "%#{search}%","%#{search}%")
    else
      User.all
    end
  end

  def update_without_current_password(params, *options)
    params.delete(:current_password)

    if params[:password].blank? && params[:password_confirmation].blank?
      params.delete(:password)
      params.delete(:password_confirmation)
    end

    binding.pry
    result = update_attributes(params, *options)
    clean_up_passwords
    result
  end

end




#routes.rb

devise_for :users,
  controllers: { registrations: 'registrations' }

#app/controllers/registrations_controller.rb

class RegistrationsController < Devise::RegistrationsController

  protected

  def update_resource(resource, params)
    resource.update_without_current_password(params)
  end
end

#ApplicationController.rb
class ApplicationController < ActionController::Base
  protect_from_forgery with: :exception
  before_action :authenticate_user!
  before_action :configure_permitted_parameters, if: :devise_controller?

  protected

  def configure_permitted_parameters
    devise_parameter_sanitizer.permit(:sign_up, keys: [:name,:useruserid])
    devise_parameter_sanitizer.permit(:account_update, keys: [:name, :profile,:image,:sex,:age,:tall])
  end

  def log_in(user)
    session[:user_id] = user.id
  end

end


現在のパスワードは空でも更新できますが、パスワード変更なしの記事が見当たらなかったので質問させていただきました。原因特定できる記事もなかったためアドバイスまたは提案をお願いします

result = update_attributes(params, *options)
    clean_up_passwords
    result

#上記の部分でエラーが起きていました。
エラー文は以下のものになります。
Unpermitted parameter: :useruserid
   (1.0ms)  BEGIN
  ↳ app/models/user.rb:59
  User Exists (0.6ms)  SELECT  1 AS one FROM `users` WHERE `users`.`useruserid` = BINARY '@aaa' AND `users`.`id` != 1 LIMIT 1
  ↳ app/models/user.rb:59
  User Exists (11.9ms)  SELECT  1 AS one FROM `users` WHERE `users`.`email` = BINARY 'aaa@gmail.com' AND `users`.`id` != 1 LIMIT 1
  ↳ app/models/user.rb:59
   (3.6ms)  ROLLBACK
  ↳ app/models/user.rb:59
#元々あったregistrations/edit.html.haml

<h2>Edit <%= resource_name.to_s.humanize %></h2>

<%= form_for(resource, as: resource_name, url: registration_path(resource_name), html: { method: :put }) do |f| %>
  <%= render "devise/shared/error_messages", resource: resource %>

  <div class="field">
    <%= f.label :email %><br />
    <%= f.email_field :email, autofocus: true, autocomplete: "email" %>
  </div>

  <% if devise_mapping.confirmable? && resource.pending_reconfirmation? %>
    <div>Currently waiting confirmation for: <%= resource.unconfirmed_email %></div>
  <% end %>

  <div class="field">
    <%= f.label :password %> <i>(leave blank if you don't want to change it)</i><br />
    <%= f.password_field :password, autocomplete: "new-password" %>
    <% if @minimum_password_length %>
      <br />
      <em><%= @minimum_password_length %> characters minimum</em>
    <% end %>
  </div>

  <div class="field">
    <%= f.label :password_confirmation %><br />
    <%= f.password_field :password_confirmation, autocomplete: "new-password" %>
  </div>

  <div class="field">
    <%= f.label :current_password %> <i>(we need your current password to confirm your changes)</i><br />
    <%= f.password_field :current_password, autocomplete: "current-password" %>
  </div>

  <div class="actions">
    <%= f.submit "Update" %>
  </div>
<% end %>

<h3>Cancel my account</h3>

<p>Unhappy? <%= button_to "Cancel my account", registration_path(resource_name), data: { confirm: "Are you sure?" }, method: :delete %></p>

<%= link_to "Back", :back %>
#編集後(useruserid追加)
エラー画面

 ↳ app/models/user.rb:60
  User Exists (0.4ms)  SELECT  1 AS one FROM `users` WHERE `users`.`useruserid` = BINARY '@aaaa' AND `users`.`id` != 1 LIMIT 1
  ↳ app/models/user.rb:60
  User Exists (0.4ms)  SELECT  1 AS one FROM `users` WHERE `users`.`email` = BINARY 'aaa@gmail.com' AND `users`.`id` != 1 LIMIT 1
  ↳ app/models/user.rb:60
   (0.2ms)  ROLLBACK
  ↳ app/models/user.rb:60

result = update_attributes(params, *options)
上記のコードでエラーが起きました。


binding.pryを実行した結果
イメージ説明

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

0

deviseの編集画面では、もともとパスワード変更を入力しなくても現在のパスワードだけ入力すれば更新が可能になっているはずです。作業の中でその辺りの変更を行なった記憶はありませんか?

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/05/13 01:47

    あまり覚えがありません。元の作成したファイルを追加記載しましたので気づきがあれば返答をお願い致します。

    キャンセル

0

立て続けに失礼します。

エラーの内容を見るとUnpermitted parameter: :useruseridとありますので、
どうやらuseruseridというカラムがdeviseに許可されていないようです。

#ApplicationController.rb
  def configure_permitted_parameters
    devise_parameter_sanitizer.permit(:sign_up, keys: [:name,:useruserid])
    devise_parameter_sanitizer.permit(:account_update, keys: [:name, :profile,:image,:sex,:age,:tall])
  end


devise_parameter_sanitizer.permit(:account_update, keys: [:name, :profile,:image,:sex,:age,:tall])
:useruseridが抜けていますので、恐らくこれが原因かと思います。

:account_updateつまり更新処理の際に許可するパラメーターにuseruseridが無いということですね。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2020/05/13 12:47

    はい、ファイル、binding.pryの内容追加記載しました。

    キャンセル

  • 2020/05/14 08:48

    内容記載しましたので確認の程お願いします

    キャンセル

  • 2020/05/14 11:06

    原因がよく分かりませんね...

    [:name, :profile,:image,:sex,:age,:tall,:useruserid]の項目の中でDBで入力必須のものはありますか?nilはダメなど。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.91%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る