rgutierrez/resq
1
0
Fork 0
Code Issues 126 Pull requests 49 Projects Releases 4 Packages Wiki Activity Actions

Merge branch '447-resq-landmark-required-in-job-order' into '453-resq-july-30-release'

Browse source 
Make Landmark field in JO required. #447

See merge request jankstudio/resq!526
This commit is contained in:
Korina Cordero 2020-07-28 10:11:30 +00:00
commit 84042a6bb9
1 changed files with 4 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/Service/JobOrderHandler/ResqJobOrderHandler.php
View file

@ -320,6 +320,10 @@ class ResqJobOrderHandler implements JobOrderHandlerInterface
} }
} }
// check if landmark is set
if (empty($req->request->get('landmark')))
$error_array['landmark'] = 'Landmark is required.';
// TODO: check status before saving since JO might already // TODO: check status before saving since JO might already
// have a status that needs to be retained // have a status that needs to be retained