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

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

ただいまの
回答率

87.94%

ユーザー情報が更新されない。

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 493

score 2

前提・実現したいこと

ユーザー登録情報を変更した後に、変更内容をupdateさせたい。updateをしようとしてもrenderでeditページに戻ってしまう。

発生している問題・エラーメッセージ

Started PATCH "/users/3" for ::1 at 2020-09-20 16:15:02 +0900
Processing by UsersController#update as HTML
  Parameters: {"authenticity_token"=>"Qc88vYX/mt2UR6csI9ahU6M+S795ElzNGqB9ldSs8YU+GXzSSv7W08vsRJlvww5ZQwdsVd1z1KbJnLhmKK8ooQ==", "user"=>{"name"=>"Ns.jimin", "email"=>"a@a"}, "commit"=>"Change your account", "id"=>"3"}
  User Load (0.4ms)  SELECT `users`.* FROM `users` WHERE `users`.`id` = 3 ORDER BY `users`.`id` ASC LIMIT 1
   (0.2ms)  BEGIN
  ↳ app/controllers/users_controller.rb:6:in `update'
  User Exists? (0.5ms)  SELECT 1 AS one FROM `users` WHERE `users`.`name` = BINARY 'Ns.jimin' AND `users`.`id` != 3 LIMIT 1
  ↳ app/controllers/users_controller.rb:6:in `update'
   (0.2ms)  ROLLBACK
  ↳ app/controllers/users_controller.rb:6:in `update'
  Rendering users/edit.html.erb within layouts/application
  Rendered users/edit.html.erb within layouts/application (Duration: 1.0ms | Allocations: 445)
[Webpacker] Everything's up-to-date. Nothing to do
Completed 200 OK in 25ms (Views: 16.2ms | ActiveRecord: 1.2ms | Allocations: 22272)

該当のソースコード

class UsersController < ApplicationController
  def edit
  end

  def update
    if current_user.update(user_params)
      redirect_to root_path
    else
      render :edit
    end
  end

  private

  def user_params
    params.require(:user).permit(:name, :email)
  end
end
app/views/users/edit.html.erb
<div class="login-view">
  <div class="login-form">
    <%= form_with model: current_user, url: user_path, local: true do |f|%>
    <div class="login-text">
      <%= link_to "finish the work", destroy_user_session_path, method: :delete, class: 'login-submit'%>
      <%= link_to "Back to the hospital", :back, class: 'login-submit'%>
    </div>
    <div class='input-field'>
      <%= f.text_field :name, class:"login-username", autofocus: true, placeholder:"Name" %>
      <%= f.email_field :email, class:"login-username", autofocus: true, placeholder:"Email" %>
    </div>
    <%= f.submit "Change your account", class: 'login-submit' %>
    <% end %>
  </div>
  <div class="underlay-logout-photo"></div>
  <div class="underlay-black"></div>
</div>
        Prefix Verb   URI Pattern                                                                              Controller#Action
                     new_user_session GET    /users/sign_in(.:format)                                                                 devise/sessions#new
                         user_session POST   /users/sign_in(.:format)                                                                 devise/sessions#create
                 destroy_user_session DELETE /users/sign_out(.:format)                                                                devise/sessions#destroy
                    new_user_password GET    /users/password/new(.:format)                                                            devise/passwords#new
                   edit_user_password GET    /users/password/edit(.:format)                                                           devise/passwords#edit
                        user_password PATCH  /users/password(.:format)                                                                devise/passwords#update
                                      PUT    /users/password(.:format)                                                                devise/passwords#update
                                      POST   /users/password(.:format)                                                                devise/passwords#create
             cancel_user_registration GET    /users/cancel(.:format)                                                                  devise/registrations#cancel
                new_user_registration GET    /users/sign_up(.:format)                                                                 devise/registrations#new
               edit_user_registration GET    /users/edit(.:format)                                                                    devise/registrations#edit
                    user_registration PATCH  /users(.:format)                                                                         devise/registrations#update
                                      PUT    /users(.:format)                                                                         devise/registrations#update
                                      DELETE /users(.:format)                                                                         devise/registrations#destroy
                                      POST   /users(.:format)                                                                         devise/registrations#create
                       patients_index GET    /patients/index(.:format)                                                                patients#index
                                 root GET    /                                                                                        patients#index
                            edit_user GET    /users/:id/edit(.:format)                                                                users#edit
                                 user PATCH  /users/:id(.:format)                                                                     users#update
                                      PUT    /users/:id(.:format)                                                                     users#update
ActiveRecord::RecordInvalid in UsersController#update
Validation failed: Password
class User < ApplicationRecord
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable

  has_many :patient_users
  has_many :patients, through: :patient_users

  with_options presence: true do
    validates :name, uniqueness: { case_sensitive: true }, length: { maximum: 15 }
  end
  PASSWORD_REGEX = /\A(?=.*?[a-z])(?=.*?[\d])[a-z\d]+\z/i.freeze
    validates_format_of :password, with: PASSWORD_REGEX, message: ''
end

試したこと

binding.pryをコントローラーにかけて見るが、user_paramsは正常に送られていた。

補足情報(FW/ツールのバージョンなど)

rails6.0

初心者です。よろしくお願いいたします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+1

if current_user.update(user_params)

if current_user.update!(user_params)
にして試してください。エラーの原因が表示されます。
多分passwordがないというえらーだと予想されます。
そこ確認できたら、
モデルUserのvalidationの定義を載せてください

追記
validates_format_of :password に on: :create オプションを追加してください。
登録の時に必須で、それ以外は無くてもよい、とするため

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2020/09/21 20:28

    無事update出来ました!ありがとうございます!
    update時にはパスワードの入力は不要なため、with_options presence: trueはパスワードにつけていなかったのですが、何故反応してしまったのでしょうか?

    キャンセル

  • 2020/09/22 03:48

    validates_format_of :password が動いたのです。
    空なのでformatにマッチしなかったのでエラー

    キャンセル

  • 2020/09/22 08:49

    ご回答有り難うございます!
    なるほど、format_ofオプション時にはallow_nilやallow_blankなどをかけない限りonオプションでバリデーションのタイミングをしてして置く事が必要そうですね!
    有り難うございました!

    キャンセル

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

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

関連した質問

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