From 0008d9bc6f8094eaeb43ecf6a9d590aa57094b05 Mon Sep 17 00:00:00 2001 From: David Taylor Date: Tue, 20 Nov 2018 14:19:26 +0000 Subject: [PATCH] FIX: Correct order of operations in `if` statement --- plugin.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/plugin.rb b/plugin.rb index c493b23..39bd8a0 100644 --- a/plugin.rb +++ b/plugin.rb @@ -34,7 +34,7 @@ class Auth::ManagedAuthenticator < Auth::Authenticator association = UserAssociatedAccount.find_by(provider_name: auth_token[:provider], provider_uid: auth_token[:uid]) - if match_by_email && association.nil? && user = User.find_by_email(email) && !UserAssociatedAccount.exists?(user: user, provider_name: auth_token[:provider]) + if match_by_email && association.nil? && (user = User.find_by_email(email)) && !UserAssociatedAccount.exists?(user: user, provider_name: auth_token[:provider]) association = UserAssociatedAccount.create!(user: user, provider_name: auth_token[:provider], provider_uid: auth_token[:uid], info: auth_token[:info], credentials: auth_token[:credentials], extra: auth_token[:extra]) end