Commit 8a3a71500d90fe3c99a102beff10e84afc32fe45
1 parent
78d45848
Exists in
theme-brasil-digital-from-staging
and in
9 other branches
Removes 2 duplicates methods and fixes 1
Showing
1 changed file
with
2 additions
and
14 deletions
Show diff stats
app/models/user.rb
| ... | ... | @@ -128,7 +128,8 @@ class User < ActiveRecord::Base |
| 128 | 128 | end |
| 129 | 129 | |
| 130 | 130 | #FIXME make this test |
| 131 | - def private_token_expired? | |
| 131 | + def private_token_expired? | |
| 132 | + return true if self.private_token_generated_at.nil? | |
| 132 | 133 | self.generate_private_token! if self.private_token.nil? || (self.private_token_generated_at + 2.weeks < DateTime.now) |
| 133 | 134 | end |
| 134 | 135 | |
| ... | ... | @@ -354,19 +355,6 @@ class User < ActiveRecord::Base |
| 354 | 355 | @is_password_required = false |
| 355 | 356 | end |
| 356 | 357 | |
| 357 | - #FIXME make this test | |
| 358 | - def generate_private_token! | |
| 359 | - self.private_token = SecureRandom.hex | |
| 360 | - self.private_token_generated_at = DateTime.now | |
| 361 | - save(:validate => false) | |
| 362 | - end | |
| 363 | - | |
| 364 | - #FIXME make this test | |
| 365 | - def private_token_expired? | |
| 366 | - self.generate_private_token! if self.private_token.nil? || (self.private_token_generated_at + 2.weeks < DateTime.now) | |
| 367 | - end | |
| 368 | - | |
| 369 | - | |
| 370 | 358 | protected |
| 371 | 359 | |
| 372 | 360 | def normalize_email | ... | ... |