From 9c4636f5c4dd5d3b7ff12effb553172f8235cfe6 Mon Sep 17 00:00:00 2001
From: msquare <msquare@notrademark.de>
Date: Thu, 17 Nov 2016 14:35:04 +0100
Subject: reduce complexity of user driver license

---
 includes/view/UserDriverLicenses_view.php | 30 +++++++++++++++---------------
 1 file changed, 15 insertions(+), 15 deletions(-)

(limited to 'includes/view/UserDriverLicenses_view.php')

diff --git a/includes/view/UserDriverLicenses_view.php b/includes/view/UserDriverLicenses_view.php
index c07e5e1f..5a3effeb 100644
--- a/includes/view/UserDriverLicenses_view.php
+++ b/includes/view/UserDriverLicenses_view.php
@@ -2,16 +2,16 @@
 
 /**
  * Edit a user's driving license information.
+ *
  * @param User $user_source
+ *          The user
  * @param bool $wants_to_drive
- * @param bool $has_car
- * @param bool $has_license_car
- * @param bool $has_license_3_5t_transporter
- * @param bool $has_license_7_5t_truck
- * @param bool $has_license_12_5t_truck
- * @param bool $has_license_forklift
+ *          true, if the user wants to drive
+ * @param UserDriverLicense $user_driver_license
+ *          The user driver license
+ * @return string
  */
-function UserDriverLicense_edit_view($user_source, $wants_to_drive, $has_car, $has_license_car, $has_license_3_5t_transporter, $has_license_7_5t_truck, $has_license_12_5t_truck, $has_license_forklift) {
+function UserDriverLicense_edit_view($user_source, $wants_to_drive, $user_driver_license) {
   return page_with_title(sprintf(_("Edit %s driving license information"), User_Nick_render($user_source)), [
       buttons([
           button(user_link($user_source), _("Back to profile"), 'back') 
@@ -22,17 +22,17 @@ function UserDriverLicense_edit_view($user_source, $wants_to_drive, $has_car, $h
           form_checkbox('wants_to_drive', _("I am willing to operate cars for the PL"), $wants_to_drive),
           div('panel panel-default', [
               div('panel-body', [
-                  form_checkbox('has_car', _("I have my own car with me and am willing to use it for the PL (You'll get reimbursed for fuel)"), $has_car),
+                  form_checkbox('has_car', _("I have my own car with me and am willing to use it for the PL (You'll get reimbursed for fuel)"), $user_driver_license['has_car']),
                   heading(_("Driver license"), 3),
-                  form_checkbox('has_license_car', _("Car"), $has_license_car),
-                  form_checkbox('has_license_3_5t_transporter', _("Transporter 3,5t"), $has_license_3_5t_transporter),
-                  form_checkbox('has_license_7_5t_truck', _("Truck 7,5t"), $has_license_7_5t_truck),
-                  form_checkbox('has_license_12_5t_truck', _("Truck 12,5t"), $has_license_12_5t_truck),
-                  form_checkbox('has_license_forklift', _("Forklift"), $has_license_forklift) 
+                  form_checkbox('has_license_car', _("Car"), $user_driver_license['has_license_car']),
+                  form_checkbox('has_license_3_5t_transporter', _("Transporter 3,5t"), $user_driver_license['has_license_3_5t_transporter']),
+                  form_checkbox('has_license_7_5t_truck', _("Truck 7,5t"), $user_driver_license['has_license_7_5t_truck']),
+                  form_checkbox('has_license_12_5t_truck', _("Truck 12,5t"), $user_driver_license['has_license_12_5t_truck']),
+                  form_checkbox('has_license_forklift', _("Forklift"), $user_driver_license['has_license_forklift']) 
               ]) 
           ], 'driving_license'),
           form_submit('submit', _("Save")) 
-      ]) ,
+      ]),
       '<script type="text/javascript">
         $(function() {
           if($("#wants_to_drive").is(":checked"))
@@ -49,7 +49,7 @@ function UserDriverLicense_edit_view($user_source, $wants_to_drive, $has_car, $h
             }
           );
         });
-      </script>'
+      </script>' 
   ]);
 }
 
-- 
cgit v1.2.3-54-g00ecf