Fix bug where JO goes back to assigned when updated. #424
This commit is contained in:
parent
d61d71804d
commit
ed53e494b4
1 changed files with 6 additions and 1 deletions
|
|
@ -595,7 +595,6 @@ class CMBJobOrderHandler implements JobOrderHandlerInterface
|
|||
->setServiceType($stype)
|
||||
->setWarrantyClass($req->request->get('warranty_class'))
|
||||
->setSource($req->request->get('source'))
|
||||
->setStatus(JOStatus::ASSIGNED)
|
||||
->setDeliveryInstructions($req->request->get('delivery_instructions'))
|
||||
->setTier1Notes($req->request->get('tier1_notes'))
|
||||
->setTier2Notes($req->request->get('tier2_notes'))
|
||||
|
|
@ -611,6 +610,12 @@ class CMBJobOrderHandler implements JobOrderHandlerInterface
|
|||
$jo->addMeta('discount', $discount);
|
||||
$jo->addMeta('service_charges', $service_charges);
|
||||
|
||||
// retain old jo status if it's an update JO
|
||||
if ($old_jo_status != null)
|
||||
$jo->setStatus($old_jo_status);
|
||||
else
|
||||
$jo->setStatus(JOStatus::ASSIGNED);
|
||||
|
||||
// check if user is null, meaning call to create came from API
|
||||
if ($user != null)
|
||||
{
|
||||
|
|
|
|||
Loading…
Reference in a new issue