Skip to content

Commit

Permalink
Inline trainee_teacher? method
Browse files Browse the repository at this point in the history
This was the only method EligibilityCheckable was being pulled in for.
While we're duplicating the definition of a trainee teacher, inlining
the method here will make splitting up the EligibilityCheckable policy
easier.
  • Loading branch information
rjlynch committed Dec 11, 2024
1 parent a71315b commit 2cf0f5e
Showing 1 changed file with 4 additions and 2 deletions.
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
module Journeys
module AdditionalPaymentsForTeaching
class NqtInAcademicYearAfterIttForm < Form
include EligibilityCheckable

attribute :nqt_in_academic_year_after_itt, :boolean

validates :nqt_in_academic_year_after_itt, inclusion: {in: [true, false], message: i18n_error_message(:inclusion)}
Expand All @@ -23,6 +21,10 @@ def save

private

def trainee_teacher?
nqt_in_academic_year_after_itt == false
end

def determine_induction_answer_from_dqt_record
return unless passed_details_check_with_teacher_id?
# We can derive the induction_completed value for current_claim using the
Expand Down

0 comments on commit 2cf0f5e

Please sign in to comment.