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

Question

前提・実現したいこと

ユーザー登録情報を変更した後に、変更内容を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

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

Answer 1

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

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

Comments

[Misa0]

ご回答有り難うございました！
仰られた通り、パスワードエラーが出ました！
```
ActiveRecord::RecordInvalid in UsersController#update
Validation failed: Password
```
```ruby
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
```
バリデーションが悪さしてたんですかね。パスワードのバリデーションのかけ方がおかしいのでしょうか？

[winterboum]

プログラムを質問欄に<code>で書いてください。コメント欄だとベタなので読みにくい

[Misa0]

失礼しました。
質問欄に追記しましたので、よろしくお願い致します。

[Misa0]

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

[winterboum]

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

[Misa0]

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