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

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

ただいまの
回答率

90.50%

  • Ruby

    9421questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails

    8846questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

  • Ruby on Rails 5

    3173questions

  • Devise

    311questions

    Deviseとは、Ruby-on-Railsの認証機能を追加するプラグインです。

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

解決済

回答 1

投稿 編集

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

zilch

score 54

前提・実現したいこと

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

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

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+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を実行してみてください。自ずと答えは出ると思います。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/02/25 00:49

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

    キャンセル

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

  • Ruby

    9421questions

    Rubyはプログラミング言語のひとつで、オープンソース、オブジェクト指向のプログラミング開発に対応しています。

  • Ruby on Rails

    8846questions

    Ruby on Railsは、オープンソースのWebアプリケーションフレームワークです。「同じことを繰り返さない」というRailsの基本理念のもと、他のフレームワークより少ないコードで簡単に開発できるよう設計されています。

  • Ruby on Rails 5

    3173questions

  • Devise

    311questions

    Deviseとは、Ruby-on-Railsの認証機能を追加するプラグインです。