2015-09-29 27 views
6

Jak mogę zmienić poniższy kod, aby użytkownik logujący się na Facebooku pomijał potwierdzenie za pomocą funkcji potwierdzania? Próbowałem dodać user.skip_confirmation! pod linią user.email..., ale to nie zadziałało.Potwierdzenie pomijania pominięcia podczas korzystania z omniauth

user.rb:

# Finds or creates user based on omniauth hash from given provider 
    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 
     user.first_name = auth.info.first_name 
     user.last_name = auth.info.last_name 
     user.email = auth.info.email 
    end 
    end 

    # Overrides class method 'new_with_session' to persist and validate attributes 
    def self.new_with_session(params, session) 
    if session["devise.user_attributes"] 
     new(session["devise.user_attributes"], without_protection: true) do |user| 
     user.attributes = params 
     user.valid? 
     end 
    else 
     super 
    end 
    end 

omniauth_callbacks_controller.rb

class Users::OmniauthCallbacksController < Devise::OmniauthCallbacksController 

    def all 
    user = User.from_omniauth(request.env["omniauth.auth"]) 
    if user.persisted? 
     flash.notice = "Signed in!" 
     sign_in_and_redirect user 
    else 
     session["devise.user_attributes"] = user.attributes 
     redirect_to new_user_registration_url 
    end 
    end 
    alias_method :facebook, :all 
end 

Odpowiedz

8

Spróbuj tego z first_or_initialize:

def self.from_omniauth(auth) 
    where(provider: auth.provider, uid: auth.uid). first_or_initialize do |user| 
     user.provider = auth.provider 
     user.uid = auth.uid 
     user.first_name = auth.info.first_name 
     user.last_name = auth.info.last_name 
     user.email = auth.info.email 
     user.skip_confirmation! 
     user.save! 
    end 
    end 
+0

I rzeczywiście dodaje linię 'user.confirmed_at = Time.zone. teraz "i wydaje się działać. Czy widzisz jakieś problemy robiące to w ten sposób w porównaniu do twojej sugestii? Dzięki – user4584963

+1

możesz użyć 'user.confirm' @ user4584963 http://www.rubydoc.info/github/plataformatec/devise/Devise/Models/Confirmable#confirm-instance_method – illusionist

+0

@ user4584963 Jest to praktycznie to samo, ale' skip_confirmation! 'został stworzony z jakiegoś powodu. O ile mi wiadomo, lepsza praktyka kodowania. Zobacz [dokumenty na ten temat tutaj] (http://www.rubydoc.info/github/plataformatec/devise/Devise%2FModels%2FConfirmable%3Askip_confirmation%21) –