Updating model_config and require confirmable hook with last changes.
This commit is contained in:
parent
63d5f9dc82
commit
697a17686e
|
@ -68,9 +68,6 @@ module Devise
|
||||||
include Devise::Models.const_get(m.to_s.classify)
|
include Devise::Models.const_get(m.to_s.classify)
|
||||||
end
|
end
|
||||||
|
|
||||||
# Assert valid keys after including modules to ensure MODEL_CONFIG is fully loaded
|
|
||||||
options.assert_valid_keys(:except, *Devise::MODEL_CONFIG)
|
|
||||||
|
|
||||||
# Convert new keys to methods which overwrites Devise defaults
|
# Convert new keys to methods which overwrites Devise defaults
|
||||||
options.each { |key, value| send(:"#{key}=", value) }
|
options.each { |key, value| send(:"#{key}=", value) }
|
||||||
end
|
end
|
||||||
|
|
|
@ -1,3 +1,5 @@
|
||||||
|
require 'devise/hooks/confirmable'
|
||||||
|
|
||||||
module Devise
|
module Devise
|
||||||
module Models
|
module Models
|
||||||
|
|
||||||
|
@ -27,7 +29,6 @@ module Devise
|
||||||
# User.find(1).send_confirmation_instructions # manually send instructions
|
# User.find(1).send_confirmation_instructions # manually send instructions
|
||||||
# User.find(1).reset_confirmation! # reset confirmation status and send instructions
|
# User.find(1).reset_confirmation! # reset confirmation status and send instructions
|
||||||
module Confirmable
|
module Confirmable
|
||||||
Devise.model_config(self, :confirm_in, 0.days)
|
|
||||||
|
|
||||||
def self.included(base)
|
def self.included(base)
|
||||||
base.class_eval do
|
base.class_eval do
|
||||||
|
@ -158,6 +159,8 @@ module Devise
|
||||||
confirmable
|
confirmable
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
Devise.model_config(self, :confirm_in, 0.days)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
|
@ -32,7 +32,6 @@ module Devise
|
||||||
# # lookup the user based on the incoming cookie information
|
# # lookup the user based on the incoming cookie information
|
||||||
# User.serialize_from_cookie(cookie_string)
|
# User.serialize_from_cookie(cookie_string)
|
||||||
module Rememberable
|
module Rememberable
|
||||||
Devise.model_config(self, :remember_for, 2.weeks)
|
|
||||||
|
|
||||||
def self.included(base)
|
def self.included(base)
|
||||||
base.class_eval do
|
base.class_eval do
|
||||||
|
@ -63,12 +62,12 @@ module Devise
|
||||||
|
|
||||||
# Checks whether the incoming token matches or not with the record token.
|
# Checks whether the incoming token matches or not with the record token.
|
||||||
def valid_remember_token?(token)
|
def valid_remember_token?(token)
|
||||||
!remember_expired? && remember_token == token
|
remember_token? && !remember_expired? && remember_token == token
|
||||||
end
|
end
|
||||||
|
|
||||||
# Remember token should be expired if expiration time not overpass now.
|
# Remember token should be expired if expiration time not overpass now.
|
||||||
def remember_expired?
|
def remember_expired?
|
||||||
!remember_token? || remember_expires_at <= Time.now.utc
|
remember_expires_at <= Time.now
|
||||||
end
|
end
|
||||||
|
|
||||||
# Remember token expires at created time + remember_for configuration
|
# Remember token expires at created time + remember_for configuration
|
||||||
|
@ -89,8 +88,9 @@ module Devise
|
||||||
rememberable = find_by_id(rememberable_id) if rememberable_id
|
rememberable = find_by_id(rememberable_id) if rememberable_id
|
||||||
rememberable if rememberable.try(:valid_remember_token?, remember_token)
|
rememberable if rememberable.try(:valid_remember_token?, remember_token)
|
||||||
end
|
end
|
||||||
|
|
||||||
end
|
end
|
||||||
|
|
||||||
|
Devise.model_config(self, :remember_for, 2.weeks)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
|
@ -108,11 +108,6 @@ class RememberableTest < ActiveSupport::TestCase
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
test 'remember should be expired without remember token' do
|
|
||||||
user = create_user
|
|
||||||
assert user.remember_expired?
|
|
||||||
end
|
|
||||||
|
|
||||||
test 'remember expires at should sum date of creation with remember for configuration' do
|
test 'remember expires at should sum date of creation with remember for configuration' do
|
||||||
Devise.remember_for = 3.days
|
Devise.remember_for = 3.days
|
||||||
user = create_user
|
user = create_user
|
||||||
|
|
Loading…
Reference in New Issue