Allow multiple question to disable the new report form

This changes the existing logic to account for the fact that multiple
questions might have the disable form checkbox ticked for one of their
options.

At the moment this just displays one of the messages at a time, with the
first one taking priority. So if you have a "Is this an emergency"
question and a "Is this on private land" question which both disable the
form, and the user has selected "Yes" to both, then only the message for
the first question is displayed. This can potentially be improved in the
future, but seemed out of scope for this change.

1 files changed, 7 insertions, 3 deletions

diff --git a/t/app/controller/report_new_open311.t b/t/app/controller/report_new_open311.t
index 3488b96ce..dcad10899 100644
--- a/t/app/controller/report_new_open311.t
+++ b/t/app/controller/report_new_open311.t
@@ -378,9 +378,13 @@ subtest "Category extras includes form disabling string" => sub {
             my $output = $json->{by_category} ? $json->{by_category}{Pothole}{disable_form} : $json->{disable_form};
             is_deeply $output, {
                 all => 'Please ring us!',
-                message => 'Please please ring',
-                code => 'dangerous',
-                answers => [ 'yes' ],
+                questions => [
+                    {
+                        message => 'Please please ring',
+                        code => 'dangerous',
+                        answers => [ 'yes' ],
+                    },
+                ],
             };
         }