persist?がfalseになる原因。

Question

以下、①、②以外の原因を教えていただきたいです。

###前提・実現したいこと
persist?の結果をtrueにしたい。

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

persist?が返す値が、毎度falseになってしまいます。
調べたところ、falseを返す原因は大きく分けて2つだとわかりました。
①モデル構築しか行われていない(save or createがない)
②validationが働いて①の妨げになってしまっている。

①、②の問題が発生しているか確認をしましたが、
どうやらないようです。

persist?が他にfalseになる原因が他にもあるのかをこの質問を読んでくださった方にお聞きしたいです。

###該当のソースコード

【users_controller.rb】

  def facebook
    user = User.from_omniauth(request.env["omniauth.auth"])←★from_omniauthに原因があるはず。★
      if user.persisted? ←★falseになり、elseの処理しか行われない。★
        sign_in_and_redirect user
        set_flash_message(:notice, :success, kind: __callee__.to_s.capitalize) if is_navigational_format?
      else←★いつもこの処理が実行されてしまう。★
        session["devise.user_attributes"] = user.attributes
        redirect_to new_user_registration_url(from_omniauth_callback: "１")
      end
  end

【user.rb】

  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :trackable, :validatable

  def self.from_omniauth(auth)
    where(provider: auth.provider, uid: auth.uid).first_or_create do |user| ←first_or_createがあるので①の問題は発生していない…？
      user.provider = auth.provider
      user.uid      = auth.uid
    end
  end

【omniauth.rb】

Rails.application.config.middleware.use OmniAuth::Builder do
  require 'omniauth/strategies/facebook_another'

  provider :facebook, "ID", "KEY"

  provider :facebook_another, "ID", "KEY"

end

###試したこと
①【user.rb】のself.from_omniauthメソッドでデータがきちんと保存されているかを確認するため条件分岐を書きましたが、エラーがでてできませんでした…

###補足情報(言語/FW/ツール等のバージョンなど)
①user.rbにはvalidationを書いていないので②の問題も発生していないと思われます。
②Userテーブルにはproviderとuidカラムは存在します。
③deviseで複数モデルを実装してます。deviseでのOmniAuthは単一モデルにしか対応できないそうなので。
④関連する質問はこちら

###情報追加

hana-daさんへ

見づらくて申し訳ありません((+_+))

class User < ActiveRecord::Base
  # Include default devise modules. Others available are:
  
  #microposts
  has_many :microposts, dependent: :destroy
  
  #plugin(SNS omuniauth)
  has_many :social_profiles, dependent: :destroy
  
  
  ##コメント機能
  
  #comment(user-to-micropost.user)
  has_many :comments
  
  #call(user-to-micropost.shop)
  has_many :calls
  
  #says(shop-to-micropost.shop)
  has_many :says
  
  #insists(shop-to-micropost.user)
  has_many :insists



  #relationship(users-to-users)
  has_many :active_relationships, class_name: "Relationship", foreign_key: "follower_id", dependent: :destroy
  has_many :passive_relationships, class_name: "Relationship", foreign_key: "followed_id", dependent: :destroy
  
  has_many :following, through: :active_relationships, source: :followed #フォローしている人を表示できるメソッドを関連づける
  has_many :followers, through: :passive_relationships, source: :follower #フォローされている人を表示できるメソッドを関連付ける

  #associates(user-to-shop)
  has_many :shops
  has_many :associates, dependent: :destroy
  has_many :associate_shops, through: :associates, source: :shop
  
  #makes(shop-to-user)
  has_many :makes, dependent: :destroy
  has_many :make_shops, through: :makes, source: :shop
  
  #praises(user-to-micropost)
  has_many :praises, dependent: :destroy
  has_many :praise_microposts, through: :praises, source: :micropost
  
  #evaluates(user-to-micropost)
  has_many :evaluates, dependent: :destroy
  has_many :evaluate_microposts, through: :evaluates, source: :micropost

  
  devise :database_authenticatable, :registerable,
         :recoverable, :rememberable, :trackable, :validatable
         
  def self.from_omniauth(auth)★★
    where(provider: auth.provider, uid: auth.uid).first_or_create do |user| ←first_or_create
      user.provider = auth.provider
      user.uid      = auth.uid
    end
  end

    user
  end

  
  def password_required?
    super && provider.blank?  # provider属性に値があればパスワード入力免除
  end
  
  def self.new_with_session(params, session)
    super.tap do |user|
      if data = session["devise.facebook_data"] && session["devise.facebook_data"]["extra"]["raw_info"]
          user.email = data["email"] if user.email.blank?
      end
    end
  end
  
  def feed
    following_ids = "SELECT followed_id FROM relationships WHERE follower_id = :user_id"
    associate_shops_ids = "SELECT shop_id FROM associates WHERE user_id = :user_id"
    Micropost.where("user_id IN (#{following_ids}) OR shop_id IN (#{associate_shops_ids}) OR user_id = :user_id", user_id: id)
  end
  
  #follow
  
  def follow(other_user)
    active_relationships.create(followed_id: other_user.id)
  end
  
  def unfollow(other_user)
    active_relationships.find_by(followed_id: other_user.id).destroy
  end
  
  def following?(other_user)
    following.include?(other_user)
  end
  
  #お気に入りしているかどうかを、返すメソッド(shop-to-user)
  def maked_by? shop
     makes.where(shop_id: shop.id).exists?
  end
  


end

よろしくお願いします。

Answer 1

トラブルシュートの方法を書いてみます。

プログラムが思い通りの動作をしない場合は、小さい単位で1つづつ思い通りの動作をしているかを確認する必要があります。

  def self.from_omniauth(auth)
    where(provider: auth.provider, uid: auth.uid).first_or_create do |user| ←first_or_createがあるので①の問題は発生していない…？
      user.provider = auth.provider
      user.uid      = auth.uid
    end
  end

①の問題は発生していない…？

ここ。自信が持てないんですよね。
こういうところから確認していくといいです。
(時に自信がある所が思い違いをしている場合もあります。厄介)

さて、どう確認するか?

first_or_create しているので User.from_omniauth の戻りは必ず persist? == true である事が確認できればいいわけです

本当ならテストを書いてそれが通る事を確認するのですが、テストがなくても確認はできます。

  def self.from_omniauth(auth)
    result = where(provider: auth.provider, uid: auth.uid).first_or_create do |user| ←first_or_createがあるので①の問題は発生していない…？
      user.provider = auth.provider
      user.uid      = auth.uid
    end

    raise 'result.persist? == true になってないです!!!' unless result.persist? == true
    
    return result
  end

本当に user.persist? == true になっているのであれば例外は発生しないはずです。
例外が発生したのであれば first_or_createがあるので①の問題は発生していない は偽です。

といった感じで1つづつ潰していってください。

---

じゃぁ何が悪いのか persist? が true でないという事は create に失敗しています。
createに失敗する理由はいくつかありますが create! にする事で create に失敗した時点で例外が発生するので

  def self.from_omniauth(auth)
    where(provider: auth.provider, uid: auth.uid).first_or_create! do |user| ←first_or_createがあるので①の問題は発生していない…？
      user.provider = auth.provider
      user.uid      = auth.uid
    end
  end

として例外メッセージを読めばなぜ first_or_create が失敗したのかわかるはずです。

Comments

[s.k]

hana-daさん

教えていただいたコードを試しました。
raiseで例外が発生したので、どうやら★unless result.persisted? == true★の条件を満たしていないようです！原因の特定はこのようにできるのですね。私はまだテストコードを書けませんので非常に助かりました！

はい。
first_or_create!で試したところ、Validation failed: Email can't be blank
というメッセージが出てきました！

Answer 2

【view/users/registration/new.html.erb】

  <% if f.object.password_required? %> 

がpersisted?をfalseにする原因になっていました。

原因を究明の過程(hana-daさんの指摘をもとに)
①first_or_create→first_or_create!に変更
【user.rb】

  def self.from_omniauth(auth)
    where(provider: auth.provider, uid: auth.uid).first_or_create! do |user|★変更箇所★
      user.provider = auth.provider
      user.uid      = auth.uid
    end
  end

結果、このようなエラーが発生

Validation failed: Email can't be blank

②devise :valitableのvalitableを削除
deviseでバリデーションをかけてないつもりだったので、どこにvalidationがかけられているかを確かめるために削除。
結果、以下の箇所にエラーが発生。

  def password_required?
    super && provider.blank?  # provider属性に値があればパスワード入力免除
  end

どうやら、このメソッドがvalidationをかけていたようです。
削除後persited?はtrueになりましたヽ(^o^)丿

hanadaさん、ありがとうございます！