Merge pull request #1 from jules2689/master

Fix authenticating user inside of an engine
brandnewbox · Jan 13, 2023 · d1e90d2 · d1e90d2
2 parents c68d9ea + 4f6491a
commit d1e90d2
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/lib/two_factor_authentication/controllers/helpers.rb b/lib/two_factor_authentication/controllers/helpers.rb
@@ -35,8 +35,13 @@ def handle_failed_second_factor(scope)
 
       def two_factor_authentication_path_for(resource_or_scope = nil)
         scope = Devise::Mapping.find_scope!(resource_or_scope)
+        namespace = if Devise.available_router_name
+          send(Devise.available_router_name)
+        else
+          self
+        end
         change_path = "#{scope}_two_factor_authentication_path"
-        send(change_path)
+        namespace.send(change_path)
       end
 
     end

diff --git a/lib/two_factor_authentication/models/two_factor_authenticatable.rb b/lib/two_factor_authentication/models/two_factor_authenticatable.rb
@@ -101,7 +101,7 @@ def generate_totp_secret
         def create_direct_otp(options = {})
           # Create a new random OTP and store it in the database
           digits = options[:length] || self.class.direct_otp_length || 6
-          update_attributes(
+          update_columns(
             direct_otp: random_base10(digits),
             direct_otp_sent_at: Time.now.utc
           )
@@ -122,7 +122,7 @@ def direct_otp_expired?
         end
 
         def clear_direct_otp
-          update_attributes(direct_otp: nil, direct_otp_sent_at: nil)
+          update_columns(direct_otp: nil, direct_otp_sent_at: nil)
         end
       end