Fix JO saving when plate number is blank. #424
This commit is contained in:
parent
a65211a709
commit
be52d089d2
3 changed files with 16 additions and 12 deletions
|
|
@ -454,8 +454,8 @@ class CMBJobOrderHandler implements JobOrderHandlerInterface
|
||||||
$error_array['customer_phone_mobile'] = 'Invalid mobile phone number.';
|
$error_array['customer_phone_mobile'] = 'Invalid mobile phone number.';
|
||||||
|
|
||||||
// check if plate number is in request
|
// check if plate number is in request
|
||||||
if (empty(trim($req->request->get('cv_plate'))))
|
if (empty(trim($req->request->get('plate_number'))))
|
||||||
$error_array['cv_plate'] = 'Plate number is required.';
|
$error_array['plate_number'] = 'Plate number is required.';
|
||||||
|
|
||||||
// find the vehicle using vid
|
// find the vehicle using vid
|
||||||
$new_vehicle = $em->getRepository(Vehicle::class)->find($req->request->get('vid'));
|
$new_vehicle = $em->getRepository(Vehicle::class)->find($req->request->get('vid'));
|
||||||
|
|
@ -479,7 +479,7 @@ class CMBJobOrderHandler implements JobOrderHandlerInterface
|
||||||
|
|
||||||
$new_cv->setCustomer($new_cust)
|
$new_cv->setCustomer($new_cust)
|
||||||
->setVehicle($new_vehicle)
|
->setVehicle($new_vehicle)
|
||||||
->setPlateNumber(trim($req->request->get('cv_plate')))
|
->setPlateNumber(trim($req->request->get('plate_number')))
|
||||||
->setModelYear($req->request->get('cv_year'))
|
->setModelYear($req->request->get('cv_year'))
|
||||||
->setColor('')
|
->setColor('')
|
||||||
->setStatusCondition('')
|
->setStatusCondition('')
|
||||||
|
|
@ -648,6 +648,7 @@ class CMBJobOrderHandler implements JobOrderHandlerInterface
|
||||||
|
|
||||||
// add errors to list
|
// add errors to list
|
||||||
foreach ($errors as $error) {
|
foreach ($errors as $error) {
|
||||||
|
error_log($error->getPropertyPath());
|
||||||
$error_array[$error->getPropertyPath()] = $error->getMessage();
|
$error_array[$error->getPropertyPath()] = $error->getMessage();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
@ -2490,8 +2491,8 @@ class CMBJobOrderHandler implements JobOrderHandlerInterface
|
||||||
$error_array['customer_phone_mobile'] = 'Invalid mobile phone number.';
|
$error_array['customer_phone_mobile'] = 'Invalid mobile phone number.';
|
||||||
|
|
||||||
// check if plate number is in request
|
// check if plate number is in request
|
||||||
if (empty(trim($req->request->get('cv_plate'))))
|
if (empty(trim($req->request->get('plate_number'))))
|
||||||
$error_array['cv_plate'] = 'Plate number is required.';
|
$error_array['plate_number'] = 'Plate number is required.';
|
||||||
|
|
||||||
// find the vehicle using vid
|
// find the vehicle using vid
|
||||||
$new_vehicle = $em->getRepository(Vehicle::class)->find($req->request->get('vid'));
|
$new_vehicle = $em->getRepository(Vehicle::class)->find($req->request->get('vid'));
|
||||||
|
|
@ -2516,7 +2517,7 @@ class CMBJobOrderHandler implements JobOrderHandlerInterface
|
||||||
|
|
||||||
$new_cv->setCustomer($new_cust)
|
$new_cv->setCustomer($new_cust)
|
||||||
->setVehicle($new_vehicle)
|
->setVehicle($new_vehicle)
|
||||||
->setPlateNumber(trim($req->request->get('cv_plate')))
|
->setPlateNumber(trim($req->request->get('plate_number')))
|
||||||
->setModelYear($req->request->get('cv_year'))
|
->setModelYear($req->request->get('cv_year'))
|
||||||
->setColor('')
|
->setColor('')
|
||||||
->setStatusCondition('')
|
->setStatusCondition('')
|
||||||
|
|
@ -2650,6 +2651,7 @@ class CMBJobOrderHandler implements JobOrderHandlerInterface
|
||||||
|
|
||||||
// add errors to list
|
// add errors to list
|
||||||
foreach ($errors as $error) {
|
foreach ($errors as $error) {
|
||||||
|
error_log($error->getPropertyPath());
|
||||||
$error_array[$error->getPropertyPath()] = $error->getMessage();
|
$error_array[$error->getPropertyPath()] = $error->getMessage();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -184,11 +184,11 @@
|
||||||
</div>
|
</div>
|
||||||
<div class="form-group m-form__group row">
|
<div class="form-group m-form__group row">
|
||||||
<div class="col-lg-3">
|
<div class="col-lg-3">
|
||||||
<label data-field="cv_plate">Plate #
|
<label data-field="plate_number">Plate #
|
||||||
<span style="color:red"> *</span>
|
<span style="color:red"> *</span>
|
||||||
</label>
|
</label>
|
||||||
<input type="text" name="cv_plate" id="cv-plate" class="form-control m-input cv_field" value="{{ obj.getCustomerVehicle.getPlateNumber|default('') }}" data-vehicle-field="1" disabled>
|
<input type="text" name="plate_number" id="cv-plate" class="form-control m-input cv_field" value="{{ obj.getCustomerVehicle.getPlateNumber|default('') }}" data-vehicle-field="1" disabled>
|
||||||
<div class="form-control-feedback hide" data-field="cv_plate"></div>
|
<div class="form-control-feedback hide" data-field="plate_number"></div>
|
||||||
</div>
|
</div>
|
||||||
<!--
|
<!--
|
||||||
<div class="col-lg-3">
|
<div class="col-lg-3">
|
||||||
|
|
|
||||||
|
|
@ -184,11 +184,11 @@
|
||||||
</div>
|
</div>
|
||||||
<div class="form-group m-form__group row">
|
<div class="form-group m-form__group row">
|
||||||
<div class="col-lg-3">
|
<div class="col-lg-3">
|
||||||
<label data-field="cv_plate">Plate #
|
<label data-field="plate_number">Plate #
|
||||||
<span style="color:red"> *</span>
|
<span style="color:red"> *</span>
|
||||||
</label>
|
</label>
|
||||||
<input type="text" name="cv_plate" id="cv-plate" class="form-control m-input cv_field" value="{{ obj.getCustomerVehicle.getPlateNumber|default('') }}" data-vehicle-field="1" disabled>
|
<input type="text" name="plate_number" id="cv-plate" class="form-control m-input cv_field" value="{{ obj.getCustomerVehicle.getPlateNumber|default('') }}" data-vehicle-field="1" disabled>
|
||||||
<div class="form-control-feedback hide" data-field="cv_plate"></div>
|
<div class="form-control-feedback hide" data-field="plate_number"></div>
|
||||||
</div>
|
</div>
|
||||||
<!--
|
<!--
|
||||||
<div class="col-lg-3">
|
<div class="col-lg-3">
|
||||||
|
|
@ -616,6 +616,8 @@ $(function() {
|
||||||
|
|
||||||
// display errors contextually
|
// display errors contextually
|
||||||
$.each(errors, function(field, msg) {
|
$.each(errors, function(field, msg) {
|
||||||
|
console.log(field);
|
||||||
|
console.log(msg);
|
||||||
var formfield = $("[name='" + field + "'], [data-name='" + field + "']");
|
var formfield = $("[name='" + field + "'], [data-name='" + field + "']");
|
||||||
var label = $("label[data-field='" + field + "']");
|
var label = $("label[data-field='" + field + "']");
|
||||||
var msgbox = $(".form-control-feedback[data-field='" + field + "']");
|
var msgbox = $(".form-control-feedback[data-field='" + field + "']");
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue