renames verify? methods [#395]
This commit is contained in:
@@ -25,7 +25,7 @@ class Verification::LetterController < ApplicationController
|
||||
|
||||
def update
|
||||
@letter = Verification::Letter.new(letter_params.merge(user: current_user))
|
||||
if @letter.verify?
|
||||
if @letter.verified?
|
||||
current_user.update(verified_at: Time.now)
|
||||
redirect_to account_path, notice: t('verification.letter.update.flash.success')
|
||||
else
|
||||
|
||||
@@ -25,7 +25,7 @@ class Verification::SmsController < ApplicationController
|
||||
|
||||
def update
|
||||
@sms = Verification::Sms.new(sms_params.merge(user: current_user))
|
||||
if @sms.verify?
|
||||
if @sms.verified?
|
||||
current_user.update(confirmed_phone: current_user.unconfirmed_phone)
|
||||
ahoy.track :level_2_user, user_id: current_user.id
|
||||
|
||||
|
||||
@@ -21,7 +21,7 @@ class Verification::Letter
|
||||
user.update(letter_requested_at: Time.now, letter_verification_code: generate_verification_code)
|
||||
end
|
||||
|
||||
def verify?
|
||||
def verified?
|
||||
validate_letter_sent
|
||||
validate_correct_code
|
||||
errors.blank?
|
||||
|
||||
@@ -35,7 +35,7 @@ class Verification::Sms
|
||||
user.update(sms_confirmation_tries: user.sms_confirmation_tries += 1)
|
||||
end
|
||||
|
||||
def verify?
|
||||
def verified?
|
||||
user.sms_confirmation_code == confirmation_code
|
||||
end
|
||||
|
||||
|
||||
Reference in New Issue
Block a user