Merge branch '405-mobile-api-advance-order-issues' into 'master'

Make sure advance order flag sticks #405

Closes #405

See merge request jankstudio/resq!467
This commit is contained in:
Kendrick Chan 2020-05-13 02:29:26 +00:00
commit 48640cbbc2

View file

@ -2360,10 +2360,12 @@ class APIController extends Controller implements LoggedController
$schedule_date = $req->request->get('date_schedule');
$slot_id = $req->request->get('slot_id');
$advance_order = $req->request->get('flag_advance_order');
if ($advance_order)
$flag_advance_order = true;
else
// check for 'false' text
if ($advance_order === false || $advance_order === 0 || $advance_order === '0' || $advance_order == 'false')
$flag_advance_order = false;
else
$flag_advance_order = true;
// $flag_advance_order = $advance_order ? true : false;
$jo = new JobOrder();
@ -2377,7 +2379,7 @@ class APIController extends Controller implements LoggedController
->setDeliveryInstructions($instructions)
// TODO: error check for valid mode of payment
->setModeOfPayment($req->request->get('mode_of_payment'))
->setAdvanceOrder($flag_advance_order);
->setAdvanceOrder($advance_order);
// customer
$cust = $this->session->getCustomer();