From 834ecddbb7a9297289aae8707a51049f38142bad Mon Sep 17 00:00:00 2001
From: Ramon Gutierrez <arcticzero@gmail.com>
Date: Tue, 16 Jan 2018 17:53:16 +0800
Subject: [PATCH] Set year range to current + 1, change customer vehicle year
 selection to dropdown

---
 src/Controller/CustomerController.php | 10 ++++++++++
 src/Controller/VehicleController.php  |  2 +-
 templates/customer/form.html.twig     |  7 ++++++-
 3 files changed, 17 insertions(+), 2 deletions(-)

diff --git a/src/Controller/CustomerController.php b/src/Controller/CustomerController.php
index 1be54743..57b1971e 100644
--- a/src/Controller/CustomerController.php
+++ b/src/Controller/CustomerController.php
@@ -142,6 +142,8 @@ class CustomerController extends BaseController
         $params['bmfgs'] = $em->getRepository(BatteryManufacturer::class)->findAll();
         $params['vmfgs'] = $em->getRepository(VehicleManufacturer::class)->findAll();
 
+        $params['years'] = $this->generateYearOptions();
+
         // response
         return $this->render('customer/form.html.twig', $params);
     }
@@ -302,6 +304,8 @@ class CustomerController extends BaseController
         $params['bmfgs'] = $em->getRepository(BatteryManufacturer::class)->findAll();
         $params['vmfgs'] = $em->getRepository(VehicleManufacturer::class)->findAll();
 
+        $params['years'] = $this->generateYearOptions();
+
         $params['obj'] = $row;
 
         // response
@@ -527,6 +531,12 @@ class CustomerController extends BaseController
         $response->send();
     }
 
+    protected function generateYearOptions()
+    {
+        $start_year = 1950;
+        return range($start_year, date("Y") + 1);
+    }
+
     // 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 d2568fd0..cadc7857 100644
--- a/src/Controller/VehicleController.php
+++ b/src/Controller/VehicleController.php
@@ -296,7 +296,7 @@ class VehicleController extends BaseController
     protected function generateYearOptions()
     {
         $start_year = 1950;
-        return range($start_year, date("Y"));
+        return range($start_year, date("Y") + 1);
     }
 
     // check if datatable filter is present and append to query
diff --git a/templates/customer/form.html.twig b/templates/customer/form.html.twig
index 6f76c35b..79a4bdb8 100644
--- a/templates/customer/form.html.twig
+++ b/templates/customer/form.html.twig
@@ -165,7 +165,12 @@
 								</div>
 								<div class="col-lg-3">
 									<label for="vehicle-year" data-field="model_year">Year</label>
-									<input type="text" name="model_year" id="vehicle-year" class="form-control m-input" data-required="1">
+									<select class="form-control m-input" id="vehicle-year" name="model_year" data-required="1">
+										<option value=""></option>
+										{% for year in years %}
+											<option value="{{ year }}">{{ year }}</option>
+										{% endfor %}
+									</select>
 									<div class="form-control-feedback hide" data-field="model_year"></div>
 								</div>
 							</div>