Hook should be on lockable not recoverable and removed an useless comment

2022-11-09 12:18:31 -05:00 · 2012-03-19 18:02:33 -03:00 · 2012-03-19 18:02:33 -03:00 · e6af9763fd
commit e6af9763fd
parent ca73dae1f0
3 changed files with 2 additions and 3 deletions
--- a/lib/devise/hooks/recoverable.rb
+++ b/lib/devise/hooks/recoverable.rb
--- a/lib/devise/models/lockable.rb
+++ b/lib/devise/models/lockable.rb
@ -1,3 +1,5 @@
+require "devise/hooks/lockable"
+
 module Devise
  module Models
    # Handles blocking a user access after a certain number of attempts.
@ -88,7 +90,6 @@ module Devise
        # if the user can login or not (wrong password, etc)
        unlock_access! if lock_expired?

-        # As failed_attempts is always 0 after sign_in, there is no need to set it to 0
        if super && !access_locked?
          true
        else
--- a/lib/devise/models/recoverable.rb
+++ b/lib/devise/models/recoverable.rb
@ -1,5 +1,3 @@
-require "devise/hooks/recoverable"
-
 module Devise
  module Models