PPHA-528: Age when started smoking page

Author: jamiefalcus

features/questionnaire.feature

@@ -1,3 +1,4 @@
+@AgeWhenStartedSmoking
 Feature: Questionnaire
   Scenario: Cannot change responses once submitted
     Given I am logged in
@@ -40,7 +41,6 @@ Feature: Questionnaire
     Then I am on "/education"
     When I fill in and submit my education with "A-levels"
 
-
     Then I am on "/respiratory-conditions"
     When I fill in and submit my respiratory conditions with "Pneumonia" and "Emphysema"
 
@@ -56,8 +56,11 @@ Feature: Questionnaire
     Then I am on "/relatives-age-when-diagnosed"
     When I fill in and submit my relatives age when diagnosed with "Yes, they were younger than 60"
 
+    Then I am on "/age-when-started-smoking"
+    When I fill in "How old were you when you started smoking?" as "18" and submit
+
     Then I am on "/check-your-answers"
-    And I see a back link to "/relatives-age-when-diagnosed"
+    And I see a back link to "/age-when-started-smoking"
 
     And I see "Yes, I used to smoke" as a response to "Have you ever smoked tobacco?" under "Eligibility"
     And I see a date 55 years ago as a response to "Date of birth" under "Eligibility"

features/relatives_age_when_diagnosed.feature

@@ -17,14 +17,15 @@ Feature: Relatives age when diagnosed page
     And I see a form error "Select if your relatives were younger than 60 when they were diagnosed with lung cancer"
     And there are no accessibility violations
 
+  @AgeWhenStartedSmoking
   Scenario: Navigating backwards and forwards
     Given I am logged in
     When I go to "/family-history-lung-cancer"
     And I fill in and submit my family history lung cancer with "Yes"
     Then I am on "/relatives-age-when-diagnosed"
     And I see a back link to "/family-history-lung-cancer"
     When I fill in and submit my relatives age when diagnosed with "Yes, they were younger than 60"
-    Then I am on "/check-your-answers"
+    Then I am on "/age-when-started-smoking"
 
   Scenario: Redirecting if they have no family history of lung cancer
     Given I am logged in

features/steps/form_steps.py

@@ -9,6 +9,11 @@ def when_i_check_value_and_submit(context, value):
     context.page.get_by_label(value, exact=True).check()
     when_i_submit_the_form(context)
 
+@when("I fill in \"{field}\" as \"{value}\" and submit")
+def when_i_enter_value_and_submit(context, field, value):
+    context.page.get_by_label(field).fill(value)
+    when_i_submit_the_form(context)
+
 @when("I take a screenshot")
 @when("I take a screenshot {value}")
 def screenshot(context, value=""):

lung_cancer_screening/nhsuk_forms/integer_field.py

@@ -7,15 +7,18 @@ def __init__(
         *args,
         hint=None,
         label_classes=None,
+        label_is_page_heading=False,
         classes=None,
         **kwargs,
     ):
         kwargs["template_name"] = "input.jinja"
 
         self.suffix = kwargs.pop("suffix", None)
+        self.prefix = kwargs.pop("prefix", None)
         self.hint = hint
         self.classes = classes
         self.label_classes = label_classes
+        self.label_is_page_heading = label_is_page_heading
 
         super().__init__(*args, **kwargs)
 

lung_cancer_screening/nhsuk_forms/jinja2/input.jinja

@@ -10,12 +10,14 @@
     "label": {
       "text": field.label,
       "html": label_html if label_html,
-      "classes": unbound_field.label_classes if unbound_field.label_classes
+      "classes": unbound_field.label_classes if unbound_field.label_classes,
+      "isPageHeading": unbound_field.label_is_page_heading if unbound_field.label_is_page_heading
     },
     "hint": {
       "text": unbound_field.hint
     } if unbound_field.hint,
     "suffix": unbound_field.suffix if unbound_field.suffix,
+    "prefix": unbound_field.prefix if unbound_field.prefix,
     "id": field.auto_id,
     "name": field.html_name,
     "value": field.value() or "",

lung_cancer_screening/questions/forms/age_when_started_smoking_form.py

@@ -0,0 +1,27 @@
+from django import forms
+from ...nhsuk_forms.integer_field import IntegerField
+from ..models.age_when_started_smoking_response import AgeWhenStartedSmokingResponse
+
+
+class AgeWhenStartedSmokingForm(forms.ModelForm):
+    def __init__(self, *args, **kwargs):
+        super().__init__(*args, **kwargs)
+
+        self.fields["value"] = IntegerField(
+            label="How old were you when you started smoking?",
+            label_classes="nhsuk-fieldset__legend--l",
+            label_is_page_heading=True,
+            classes="nhsuk-input--width-2",
+            hint="Give an estimate if you are not sure",
+            prefix="Age",
+            error_messages={
+                'required': 'Enter your age when you started smoking',
+                'invalid': 'Enter your age when you started smoking',
+                'zero_entered':'The age you started smoking must be between 1 and your current age',
+                'old_current_age':'The age you started smoking must be the same as, or younger than your current age'
+            }
+        )
+
+    class Meta:
+        model = AgeWhenStartedSmokingResponse
+        fields = ['value']

lung_cancer_screening/questions/forms/metric_height_form.py

@@ -16,7 +16,7 @@ def __init__(self, *args, **kwargs):
         self.fields["metric"] = DecimalField(
             decimal_places=1,
             label="Centimetres",
-            classes="nhsuk-input--width-4",
+            classes="nhsuk-input--width-2",
             error_messages={
                 'required': 'Enter your height',
                 "max_decimal_places": (

lung_cancer_screening/questions/migrations/0043_agewhenstartedsmokingresponse.py

@@ -0,0 +1,27 @@
+# Generated by Django 5.2.10 on 2026-01-15 08:59
+
+import django.db.models.deletion
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('questions', '0042_alter_educationresponse_value'),
+    ]
+
+    operations = [
+        migrations.CreateModel(
+            name='AgeWhenStartedSmokingResponse',
+            fields=[
+                ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
+                ('created_at', models.DateTimeField(auto_now_add=True)),
+                ('updated_at', models.DateTimeField(auto_now=True)),
+                ('value', models.PositiveIntegerField()),
+                ('response_set', models.OneToOneField(on_delete=django.db.models.deletion.CASCADE, related_name='age_when_started_smoking_response', to='questions.responseset')),
+            ],
+            options={
+                'abstract': False,
+            },
+        ),
+    ]

lung_cancer_screening/questions/models/__init__.py

@@ -2,6 +2,7 @@
 from .response_set import ResponseSet # noqa: F401
 from .user import User # noqa: F401
 
+from .age_when_started_smoking_response import AgeWhenStartedSmokingResponse # noqa: F401
 from .asbestos_exposure_response import AsbestosExposureResponse # noqa: F401
 from .cancer_diagnosis_response import CancerDiagnosisResponse # noqa: F401
 from .check_need_appointment_response import CheckNeedAppointmentResponse # noqa: F401

lung_cancer_screening/questions/models/age_when_started_smoking_response.py

@@ -0,0 +1,53 @@
+import json
+
+from django.db import models
+from django.forms import ValidationError
+from django.utils import timezone
+
+from .base import BaseModel
+from .response_set import ResponseSet
+
+from django.core.validators import MaxValueValidator, MinValueValidator
+
+
+
+# def validate_less_than_age(self, value):
+#     print(f"self : {self}")
+#     print(f"value : {self.value}")
+#     print("what")
+#     print(f"dob : {self.response_set.date_of_birth}")
+#     print(hasattr(self.response_set, "date_of_birth"))
+#     if hasattr(self.response_set, "date_of_birth") :
+#         if value < self.get_age :
+#             raise ValidationError(
+#                 "Must be less than your age",
+#                 code="less_than_age",
+#             )
+
+class AgeWhenStartedSmokingResponse(BaseModel):
+    response_set = models.OneToOneField(ResponseSet, on_delete=models.CASCADE, related_name='age_when_started_smoking_response')
+    value = models.PositiveIntegerField(validators=[
+        MinValueValidator(1, message="The age you started smoking must be between 1 and your current age"),
+        # validate_less_than_age
+    ]
+    )
+
+    # def clean(self):
+    #     super().clean()
+    #     print(f"clean 1:{self.response_set}")
+    #     if hasattr(self.response_set, "date_of_birth_response") :
+    #         print(f"clean 2:{self.response_set.date_of_birth_response.value}")
+    #         today = timezone.now()
+    #         born = self.response_set.date_of_birth_response.value
+    #         age = today.year - born.year - ((today.month, today.day) < (born.month, born.day))
+    #         print(f"value : {self.value}")
+    #         print(f"Age {age}")
+    #         if (self.value and self.value < age):
+    #             raise ValidationError(
+    #                 "The age you started smoking must be the same as, or younger than your current age"
+    #         )
+            #if submitted_response_sets_in_last_year:
+            #    raise ValidationError(
+            #        "Responses have already been submitted for this user"
+            #    )
+