Merge pull request #4483 from opensafely-core/iaindillingham/add-new-…

…question-to-study-information-page

Add a new question to the "Study information" page

applications/form_specs.py

@@ -113,6 +113,11 @@
                         help_text=snippet("study_purpose-help_text"),
                         template_name="form_textarea",
                     ),
+                    Field(
+                        name="read_analytic_methods_policy",
+                        label="Have all applicants for OpenSAFELY read and agreed to the OpenSAFELY Analytic Methods Policy?",
+                        help_text=snippet("read_analytic_methods_policy-help_text"),
+                    ),
                 ],
             ),
         ],

applications/forms.py

@@ -12,6 +12,7 @@ def __init__(self, *args, **kwargs):
         # which uses choices.
         radio_fields = [
             "all_applicants_completed_getting_started",
+            "read_analytic_methods_policy",
             "need_record_level_data",
             "is_approved",
         ]

applications/migrations/0003_add_read_analytic_methods_policy.py

@@ -0,0 +1,20 @@
+# Generated by Django 5.0.7 on 2024-07-23 15:10
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("applications", "0002_add_short_data_report"),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="studyinformationpage",
+            name="read_analytic_methods_policy",
+            field=models.BooleanField(
+                choices=[(True, "Yes"), (False, "No")], null=True
+            ),
+        ),
+    ]

applications/models.py

@@ -218,6 +218,9 @@ class CommercialInvolvementPage(AbstractPage):
 class StudyInformationPage(AbstractPage):
     study_name = models.TextField(blank=True)
     study_purpose = models.TextField(blank=True)
+    read_analytic_methods_policy = models.BooleanField(
+        null=True, choices=YES_NO_CHOICES
+    )
 
 
 class StudyPurposePage(AbstractPage):

snippets/read_analytic_methods_policy-help_text.md

@@ -0,0 +1,3 @@
+The [OpenSAFELY Analytic Methods Policy][1] can be found on the OpenSAFELY website.
+
+[1]: https://www.opensafely.org/policies-for-researchers/#opensafely-analytic-methods-policy

tests/factories/application.py

@@ -75,6 +75,7 @@ class Meta:
 
     study_name = factory.Sequence(lambda n: f"study {n}")
     study_purpose = factory.Sequence(lambda n: f"purpose {n}")
+    read_analytic_methods_policy = factory.fuzzy.FuzzyChoice([True, False])
 
 
 class StudyPurposePageFactory(AbstractPageFactory):

tests/integration/test_applications.py

@@ -82,6 +82,7 @@ def test_successful_application(client, mailoutbox, slack_messages):
     data = {
         "study_name": "Title for this study",
         "study_purpose": "Long form technical message about what this study is about",
+        "read_analytic_methods_policy": "True",
     }
     response = client.post(url("study-information"), data, follow=True)
     assert response.status_code == 200