From fd2fe4fb6fcf8387868ce71cca7766e220fad396 Mon Sep 17 00:00:00 2001 From: Korina Cordero Date: Mon, 3 Feb 2020 08:59:22 +0000 Subject: [PATCH] Add routes and UI elements to add compatible battery to vehicle. #323 --- config/routes/battery.yaml | 5 ++ src/Controller/BatteryController.php | 26 ++++++ src/Controller/VehicleController.php | 8 ++ templates/vehicle/form.html.twig | 115 +++++++++++++++++++++------ 4 files changed, 130 insertions(+), 24 deletions(-) diff --git a/config/routes/battery.yaml b/config/routes/battery.yaml index 04a9d952..f8c31836 100644 --- a/config/routes/battery.yaml +++ b/config/routes/battery.yaml @@ -39,6 +39,11 @@ battery_delete: controller: App\Controller\BatteryController::destroy methods: [DELETE] +battery_get: + path: /ajax/battery_find + controller: App\Controller\BatteryController::getBattery + methods: [GET] + # battery manufacturers bmfg_list: diff --git a/src/Controller/BatteryController.php b/src/Controller/BatteryController.php index 059d8b57..bea9eafd 100644 --- a/src/Controller/BatteryController.php +++ b/src/Controller/BatteryController.php @@ -415,6 +415,32 @@ class BatteryController extends Controller ]); } + public function getBattery(Request $req) + { + // no access check, grant all for this ajax call + + error_log($req->query->get('id')); + // parse the id: model size + $batt_specs = explode(" ", $req->query->get('id')); + $bmodel_id = $batt_specs[0]; + $bsize_id = $batt_specs[1]; + + // find the battery using model and size + $em = $this->getDoctrine()->getManager(); + $query = $em->createQuery('select b from App\Entity\Battery b join b.model bm + join b.size bs + where bm.id = :bm_id and bs.id = :bs_id') + ->setParameter('bm_id', $bmodel_id) + ->setParameter('bs_id', $bsize_id); + + $result = $query->getResult(); + + foreach ($result as $row) + { + error_log('id ' . $row->getID()); + } + } + // check if datatable filter is present and append to query protected function setQueryFilters($datatable, &$query) { if (isset($datatable['query']['data-rows-search']) && !empty($datatable['query']['data-rows-search'])) { diff --git a/src/Controller/VehicleController.php b/src/Controller/VehicleController.php index 2df6ec74..f7f8c625 100644 --- a/src/Controller/VehicleController.php +++ b/src/Controller/VehicleController.php @@ -4,7 +4,10 @@ namespace App\Controller; use App\Entity\Vehicle; use App\Entity\VehicleManufacturer; +use App\Entity\BatteryManufacturer; use App\Entity\Battery; +use App\Entity\BatteryModel; +use App\Entity\BatterySize; use Doctrine\ORM\Query; use Symfony\Component\HttpFoundation\Request; @@ -225,6 +228,11 @@ class VehicleController extends Controller $params['years'] = $this->generateYearOptions(); $params['obj'] = $row; + // get battery information for compatible batteries + $params['bmfgs'] = $em->getRepository(BatteryManufacturer::class)->findAll(); + $params['bmodels'] = $em->getRepository(BatteryModel::class)->findAll(); + $params['bsizes'] = $em->getRepository(BatterySize::class)->findAll(); + // response return $this->render('vehicle/form.html.twig', $params); } diff --git a/templates/vehicle/form.html.twig b/templates/vehicle/form.html.twig index 4669d1c2..fdfa1a64 100644 --- a/templates/vehicle/form.html.twig +++ b/templates/vehicle/form.html.twig @@ -96,23 +96,50 @@ {% if mode == 'update' %} -
- -
-
-

- Compatible Batteries -

-
-
-
-
-
-
-
+
+
+
+

+ Compatible Batteries +

+
+
+
+
+
+
+
+ +
+ +
+
+ +
+
+ +
+
+ +
+
+
{% endif %} - - +
@@ -201,19 +228,22 @@ $(function() { }); }); + // initialize battery arrays var battRows = []; var batteryIds = []; + var battModelSize = [] {% for batt in obj.getBatteries %} - trow = { - id: "{{ batt.getID }}", - model: "{{ batt.getModel.getName|default('') }}", - size: "{{ batt.getSize.getName|default('') }}", - sell_price: "{{ batt.getSellingPrice }}" - }; + trow = { + id: "{{ batt.getID }}", + model: "{{ batt.getModel.getName|default('') }}", + size: "{{ batt.getSize.getName|default('') }}", + sell_price: "{{ batt.getSellingPrice }}" + }; - battRows.push(trow); - batteryIds.push({{ batt.getID }}); + battRows.push(trow); + batteryIds.push({{ batt.getID }}); + battModelSize.push({{ batt.getModel.getID }} + ' ' + {{ batt.getSize.getID }}); {% endfor %} // remove battery from table @@ -237,6 +267,43 @@ $(function() { battTable.reload(); }); + // add a battery to the table + $("#btn-add-battery").click(function() { + var bmfg_id = $("#bmfg").val(); + var bmodel_id = $("#bmodel").val(); + var bsize_id = $("#bsize").val(); + + var bmodelsize = (bmodel_id + ' ' + bsize_id); + + console.log(bmodelsize); + + if (battModelSize.indexOf(bmodelsize) !== -1) { + swal({ + title: 'Whoops', + text: 'This battery is already on the list.', + type: 'warning' + }); + return true; + } + + // get the battery given the model and size + $.ajax({ + method: "GET", + url: "{{ url('battery_get') }}", + data: {id: bmodelsize} + }).done(function(response) { + + }) + + // add battery to arrays + battModelSize.push(bmodelsize); + + // refresh the data table + battTable.originalDataSet = battRows; + battTable.reload(); + + }); + // battery data table var battOptions = { data: {