Rails deviseのcontroller内でif文を使いたい

Question

前提・実現したいこと

Railsを始めて5日目の初心者です。

deviseを使ってユーザー登録必須のシステムを作っています。
下記3つを実装後、ユーザー情報の更新時にif文を追加するとエラーメッセージが発生しました。

1.ユーザー情報の更新にパスワード入力をしなくてもよいようにする。
https://kossy-web-engineer.hatenablog.com/entry/2018/11/06/102047

2.deviseで作成したUserモデルにnameカラムを追加し名前を登録できるようにする。
https://qiita.com/hz1_d/items/77e440b09a1ddbe56263

3.companyモデルを作成する。

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

NoMethodError in RegistrationsController#update

undefined method `user_url' for #<RegistrationsController:0x00007ffc986c65c8> Did you mean? user_unlock_url

該当のソースコード

routes.rb

Rails.application.routes.draw do

  resources :companies
  root 'home#top'
  
  get 'home/top'
  
  devise_for :users,
  controllers: { registrations: 'registrations' }

end

registrations_controller.rb

class RegistrationsController < Devise::RegistrationsController

  protected
  def update_resource(resource, params)
    resource.update_without_current_password(params)
    if @user.present?
    end
  end

end

user.rb

class User < ApplicationRecord

  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :validatable,
         :confirmable, :lockable, :timeoutable, :trackable
  
  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

    result = update_attributes(params, *options)
    clean_up_passwords
    result
  end

end

application_controller.rb

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

  protected

  def configure_permitted_parameters
    added_attrs = [ :email, :name, :company_id, :password, :password_confirmation ]
    devise_parameter_sanitizer.permit :account_update, keys: added_attrs
  end

end

rails routes

Prefix Verb   URI Pattern                                                                              Controller#Action
                companies GET    /companies(.:format)                                                                     companies#index
                          POST   /companies(.:format)                                                                     companies#create
              new_company GET    /companies/new(.:format)                                                                 companies#new
             edit_company GET    /companies/:id/edit(.:format)                                                            companies#edit
                  company GET    /companies/:id(.:format)                                                                 companies#show
                          PATCH  /companies/:id(.:format)                                                                 companies#update
                          PUT    /companies/:id(.:format)                                                                 companies#update
                          DELETE /companies/:id(.:format)                                                                 companies#destroy
                     root GET    /                                                                                        home#top
                 home_top GET    /home/top(.:format)                                                                      home#top
         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)                                                                  registrations#cancel
    new_user_registration GET    /users/sign_up(.:format)                                                                 registrations#new
   edit_user_registration GET    /users/edit(.:format)                                                                    registrations#edit
        user_registration PATCH  /users(.:format)                                                                         registrations#update
                          PUT    /users(.:format)                                                                         registrations#update
                          DELETE /users(.:format)                                                                         registrations#destroy
                          POST   /users(.:format)                                                                         registrations#create
    new_user_confirmation GET    /users/confirmation/new(.:format)                                                        devise/confirmations#new
        user_confirmation GET    /users/confirmation(.:format)                                                            devise/confirmations#show
                          POST   /users/confirmation(.:format)                                                            devise/confirmations#create
          new_user_unlock GET    /users/unlock/new(.:format)                                                              devise/unlocks#new
              user_unlock GET    /users/unlock(.:format)                                                                  devise/unlocks#show
                          POST   /users/unlock(.:format)                                                                  devise/unlocks#create
       rails_service_blob GET    /rails/active_storage/blobs/:signed_id/*filename(.:format)                               active_storage/blobs#show
rails_blob_representation GET    /rails/active_storage/representations/:signed_blob_id/:variation_key/*filename(.:format) active_storage/representations#show
       rails_disk_service GET    /rails/active_storage/disk/:encoded_key/*filename(.:format)                              active_storage/disk#show
update_rails_disk_service PUT    /rails/active_storage/disk/:encoded_token(.:format)                                      active_storage/disk#update
     rails_direct_uploads POST   /rails/active_storage/direct_uploads(.:format)                                           active_storage/direct_uploads#create

試したこと

if文を削除してcompanyにレコードを追加してみたところエラーは発生しませんでした。

    @company = Company.new
    @company.save

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

ruby 2.6.1p33
rails 5.2.2
devise 4.6.1

噛み砕いた内容でご回答をいただけたら大変助かります。
どうぞよろしくお願いいたします。

Answer 1

Devise::RegistrationsControllerを継承した独自のRegistrationsControllerでupdate_resourceを定義することによって、update_resourceを上書き(オーバーライド)しています。

で、もともとのupdate_resourceの内容はこちらです。
https://github.com/plataformatec/devise/blob/master/app/controllers/devise/registrations_controller.rb#L93
こういった使われ方をしています。
https://github.com/plataformatec/devise/blob/master/app/controllers/devise/registrations_controller.rb#L50
返り値が利用されていますね？(つまりはresource.update_with_password(params)の返り値)

一方で、zilchさんがif文を入れたupdate_resourceの返り値は何になるでしょうか？
irbで、if true; endを実行してみてください。自ずと答えは出ると思います。

Comments

[no1knows]

丁寧に説明いただきありがとうございます！
UPDATEもオーバーライドする形ですすめてみたいと思います。