From ffa531f3115dc4ad9b52388fdb65014b53c7e11a Mon Sep 17 00:00:00 2001
From: Xu <git@xumail.de>
Date: Fri, 29 Dec 2023 13:32:00 +0100
Subject: [PATCH] disable shirt size edit for user in settings if got shirt

---
 resources/views/macros/form.twig                | 1 +
 resources/views/pages/settings/profile.twig     | 1 +
 src/Controllers/Admin/UserWorkLogController.php | 4 ++--
 src/Controllers/SettingsController.php          | 1 +
 4 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/resources/views/macros/form.twig b/resources/views/macros/form.twig
index eaa0c83a..e577a2dc 100644
--- a/resources/views/macros/form.twig
+++ b/resources/views/macros/form.twig
@@ -193,6 +193,7 @@ Renders a select element wrapped in a DIV with mb-3.
         {% endif %}
         <select id="{{ name }}" name="{{ name }}"
             class="form-control {%- if opt.class is defined %} {{ opt.class }}{% endif %}"
+            {%- if opt.disabled|default(false) %} disabled{%- endif -%}
             {%- if opt.required|default(false) %} required{%- endif -%}>
             {%- if opt.default_option is defined %}
                 <option value="">{{ opt.default_option }}</option>
diff --git a/resources/views/pages/settings/profile.twig b/resources/views/pages/settings/profile.twig
index eaa3b608..c1f26e7c 100644
--- a/resources/views/pages/settings/profile.twig
+++ b/resources/views/pages/settings/profile.twig
@@ -154,6 +154,7 @@
                         'required': isTShirtSizeRequired,
                         'required_icon': isTShirtSizeRequired,
                         'default_option': __('form.select_placeholder'),
+                        'disabled': user.state.got_shirt,
                     }) }}
                 </div>
             {% endif %}
diff --git a/src/Controllers/Admin/UserWorkLogController.php b/src/Controllers/Admin/UserWorkLogController.php
index 7d5b671f..89c315ba 100644
--- a/src/Controllers/Admin/UserWorkLogController.php
+++ b/src/Controllers/Admin/UserWorkLogController.php
@@ -86,7 +86,7 @@ class UserWorkLogController extends BaseController
         $worklog->save();
 
         $this->log->info(
-            'Added worklog for {name} ({id}) at {time} about {hours}h: {text}',
+            'Added worklog for {name} ({id}) at {time} spanning {hours}h: {text}',
             [
                 'name' => $user->name,
                 'id' => $user->id,
@@ -132,7 +132,7 @@ class UserWorkLogController extends BaseController
         $worklog->delete();
 
         $this->log->info(
-            'Deleted worklog for {name} ({id}) at {time} about {hours}h: {text}',
+            'Deleted worklog for {name} ({id}) at {time} spanning {hours}h: {text}',
             [
                 'name' => $worklog->user->name,
                 'id' => $worklog->user->id,
diff --git a/src/Controllers/SettingsController.php b/src/Controllers/SettingsController.php
index a00620eb..cc7c6c15 100644
--- a/src/Controllers/SettingsController.php
+++ b/src/Controllers/SettingsController.php
@@ -110,6 +110,7 @@ class SettingsController extends BaseController
         if (
             $goodie_tshirt
             && isset(config('tshirt_sizes')[$data['shirt_size']])
+            && !$user->state->got_shirt
         ) {
             $user->personalData->shirt_size = $data['shirt_size'];
         }