From e1b458829f2a1cf2dece755fd13e705c9c950b64 Mon Sep 17 00:00:00 2001 From: samyou-softwire Date: Tue, 21 Apr 2026 16:16:05 +0100 Subject: [PATCH] CLDC-4330: Update confirmation comment --- app/controllers/auth/confirmations_controller.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/app/controllers/auth/confirmations_controller.rb b/app/controllers/auth/confirmations_controller.rb index c9586fb73..c08af04bf 100644 --- a/app/controllers/auth/confirmations_controller.rb +++ b/app/controllers/auth/confirmations_controller.rb @@ -5,8 +5,8 @@ class Auth::ConfirmationsController < Devise::ConfirmationsController yield resource if block_given? if resource.errors.empty? - # previously we reset sign in count to indicate that a user was deactivated and so needs to reset their password on confirming their email post reactivation. - # now we have a specific flag for this. + # previously we reset sign_in_count to indicate that a user was deactivated and so needs to reset their password on confirming their email post reactivation. + # now we have a specific flag for this as resetting sign in count was difficult for auditing. # though for backwards compatability we need to ensure previous users with a reset sign in count still will see the password reset screen if resource.reset_password_on_confirmation || resource.sign_in_count.zero? token = resource.send(:set_reset_password_token)