Merge branch '447-resq-landmark-required-in-job-order' into '453-resq-july-30-release'
Make Landmark field in JO required. #447 See merge request jankstudio/resq!526
This commit is contained in:
commit
84042a6bb9
1 changed files with 4 additions and 0 deletions
|
|
@ -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
|
||||
// have a status that needs to be retained
|
||||
|
||||
|
|
|
|||
Loading…
Reference in a new issue