2015-09-29 27 views
6

Come posso modificare il codice seguente in modo che un utente che accede con facebook salti la conferma con ideata confermabile? Ho provato ad aggiungere user.skip_confirmation! sotto la riga user.email... ma questo non ha funzionato.Devise salta conferma quando si utilizza 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 

risposta

8

Prova questo con 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

realtà ho aggiunto la linea 'user.confirmed_at = Time.zone. ora e sembra funzionare. Riesci a vedere qualche problema in questo modo rispetto al tuo suggerimento? Grazie – user4584963

+1

puoi usare 'user.confirm' @ user4584963 http://www.rubydoc.info/github/plataformatec/devise/Devise/Models/Confirmable#confirm-instance_method – illusionist

+0

@ user4584963 È praticamente la stessa cosa, ma' skip_confirmation! 'è stato creato per un motivo. Per quanto ne so, una migliore pratica di codifica. Vedi [docs for it here] (http://www.rubydoc.info/github/plataformatec/devise/Devise%2FModels%2FConfirmable%3Askip_confirmation%21) –

Problemi correlati