fix: restore "No (Operator to post)" radio for internal caseworker fo…

…rm. (#338)
dvsa · Sep 19, 2024 · bdd53bf · bdd53bf
1 parent 6b46419
commit bdd53bf
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 2 deletions.
diff --git a/app/internal/module/Olcs/src/FormService/Form/Lva/OperatingCentre/LvaOperatingCentre.php b/app/internal/module/Olcs/src/FormService/Form/Lva/OperatingCentre/LvaOperatingCentre.php
@@ -48,6 +48,13 @@ public function alterForm(Form $form, array $params)
         $advertisements = $form->get('advertisements');
         $advertisements->setLabel('application_operating-centres_authorisation-sub-action.advertisements.adPlaced');
 
+        $adPlaced = $advertisements->get('radio');
+
+        // Add operator to post option back in just for caseworker form: https://dvsa.atlassian.net/browse/VOL-5814
+        $valueOptions = $adPlaced->getValueOptions();
+        $valueOptions['adSendByPost'] = 'No (operator to post)';
+        $adPlaced->setValueOptions($valueOptions);
+
         $form->get('data')->get('guidance')->setValue('lva-operating-centre-newspaper-advert');
         $form->get('data')->get('permission')->setLabel('');
 

diff --git a/app/internal/test/Olcs/src/FormService/Form/Lva/OperatingCentre/LvaOperatingCentreTest.php b/app/internal/test/Olcs/src/FormService/Form/Lva/OperatingCentre/LvaOperatingCentreTest.php
@@ -64,13 +64,20 @@ public function testAlterForm($appliedVia)
                         m::mock(ElementInterface::class)
                         ->shouldReceive('getValueOptions')
                         ->andReturn(['foo' => 'bar', OperatingCentreMapper::VALUE_OPTION_AD_UPLOAD_LATER => 'cake'])
-                        ->once()
+                        ->twice()
                         ->shouldReceive('setValueOptions')
                         ->with(['foo' => 'bar'])
                         ->once()
+                        ->shouldReceive('setValueOptions')
+                        ->with([
+                            'foo' => 'bar',
+                            'adPlacedLater' => 'cake',
+                            'adSendByPost' => 'No (operator to post)'
+                        ])
+                        ->once()
                         ->getMock()
                     )
-                    ->once()
+                    ->twice()
                     ->shouldReceive('setLabel')
                     ->with('application_operating-centres_authorisation-sub-action.advertisements.adPlaced')
                     ->once()