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

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

ただいまの
回答率

90.01%

条件分岐中のpersisted?をtrueにできない

解決済

回答 2

投稿 編集

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

s.k

score 266

2日悩んでます。

前提・実現したいこと

facebookのログイン認証を実装したのですが、思うような機能をしてくれないので
ご相談させていただきます。

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

下記、コードの条件分岐が何度繰り返してもelseになり、
登録画面にリダイレクトされてしまいます。

どうして、@user.persisted?がfalseになってしまうのかを知りたいです。

①一度、認証リンクを押しました。
そこで、facebookページに飛び、ログインせずにキャンセルを押してしまったので、
elseになってしまうのでしょうか?

②もう一つのモデル(deviseの複数モデルで開発してます)では、facebookページに飛び、ログインをしたのですが、エラーがありうまく認証できませんでした。
もう一度認証を試すと、elseでリダイレクトコースです…

def facebook
      # You need to implement the method below in your model (e.g. app/models/user.rb)
      @user = User.from_omniauth(request.env["omniauth.auth"])

      if @user.persisted? ←★なぜfalseになってしまうのか?★
        sign_in_and_redirect @user, :event => :authentication #this will throw if @user is not activated
        set_flash_message(:notice, :success, :kind => "Facebook") if is_navigational_format?
      else ←★すべて条件分岐先がelseになってしまう。★
        session["devise.facebook_data"] = request.env["omniauth.auth"]
        redirect_to new_user_registration_url
      end
  end

試したこと

①persisted?をリファレンスで確認したところ、新しいレコードかどうかをチェックするとありました。そこで、rake db:migrate:resetでデータベースを削除してもう一度、ログイン認証を試しましたがリダイレクトされてしまいました。
②facebookアプリに何か問題があるのかと思いましたので、別のアプリを製作し、IDとPASSを差し替えましたが効果なしです。
③情報追加部分のメソッド内にuser.saveを追加しました。しかし、リダイレクト現象は解決しませんでした。

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

①開発環境です。
②deviseを使い複数モデルを作っています。
③参考サイト
Rails4でOmniAuthを使用したFacebookログイン機能を実装する
OmniAuthでFacebook認証を複数のモデルで使う
既存のDevise認証アプリに OmniAuth認証を追加する
Railsのログイン認証gemのDeviseとOmniAuth-Twitterの連携(Twitterでログインする)
④validationはかけていません。

よろしくお願いします!

情報追加

★maisumakunさんへ★

【user.rb】

def self.from_omniauth(auth)
    where(provider: auth.provider, uid: auth.uid).first_or_create do |user|
      user.email = auth.info.email
      user.password = Devise.friendly_token[0,20]
    end
end
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

checkベストアンサー

+2

User.from_omniauthの中身がわからないと何とも言えない部分はありますが、その中では「モデル構築だけ」やっているのではないでしょうか。

@user.saveのような形でDBへ保存しなければ、いつまでたっても@user.persisted?はfalseのままです。あと、バリデーションに引っかかったなどで保存されないときも@user.persisted?はfalseです。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/11/05 18:11

    maisumakunさん

    お返事ありがとうございます!
    今、指摘された箇所を確認してみます!

    【User.from_omniauth】

    は追加しておきました!

    キャンセル

  • 2016/11/05 18:16

    新規作成された分にはemailとpasswordの2列があるようですが、バリデーションの方はどうでしょうか。

    キャンセル

  • 2016/11/05 18:24

    validationはかけないで開発をしていました!

    キャンセル

0

maisumakunさん

やはり、validationでした。
詳しくはこちらの質問の自己解決策をご覧ください

ありがとうございました!

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

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