Refactored according to line notes by rodrigoflores

brynary · Jul 11, 2012 · 42a0c30 · 42a0c30
1 parent 87f2fa9
commit 42a0c30
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/lib/devise/models/confirmable.rb b/lib/devise/models/confirmable.rb
@@ -178,7 +178,7 @@ def confirmation_period_expired?
 
         # Checks whether the record requires any confirmation.
         def pending_any_confirmation
-          if !confirmation_period_expired? && (!confirmed? || pending_reconfirmation?)
+          unless confirmation_period_expired? || (confirmed? && !pending_reconfirmation?)
             yield
           else
             # TODO: cache this call or not?

diff --git a/lib/generators/templates/devise.rb b/lib/generators/templates/devise.rb
@@ -97,7 +97,7 @@
   # within 3 days after the mail was sent, but on the fourth day their account can't be
   # confirmed with the token any more
   # Default is nil, meaning there is no restriction on how long a user can take before
-  # comfirming their account.
+  # confirming their account.
   # config.expire_confirmation_token_after = 3.days
 
   # If true, requires any email changes to be confirmed (exactly the same way as

diff --git a/test/models/confirmable_test.rb b/test/models/confirmable_test.rb
@@ -236,7 +236,7 @@ def setup
     end
   end
 
-  def confirm_user_by_token_with_confirmation_sent_at confirmation_sent_at
+  def confirm_user_by_token_with_confirmation_sent_at(confirmation_sent_at)
     user = create_user
     user.confirmation_sent_at = confirmation_sent_at
     confirmed_user = User.confirm_by_token(user.confirmation_token)