From d58d4642af39122801fe2e52cc75a0cd89652f83 Mon Sep 17 00:00:00 2001
From: philipp <philipp@hofer.link>
Date: Mon, 15 Apr 2024 11:23:07 +0200
Subject: [PATCH] 1x boats: don't show 'Ruderer: ' in logbook; Fixes #362

---
 templates/includes/forms/log.html.tera | 26 ++++----
 templates/includes/macros.html.tera    | 82 +++++++++++++-------------
 2 files changed, 56 insertions(+), 52 deletions(-)

diff --git a/templates/includes/forms/log.html.tera b/templates/includes/forms/log.html.tera
index d1d46e4..d79b824 100644
--- a/templates/includes/forms/log.html.tera
+++ b/templates/includes/forms/log.html.tera
@@ -218,18 +218,20 @@
                     {% if log.comments %}<span class="text-sm italic">- "{{ log.comments }}"</span>{% endif %}
                 </div>
                 {% if amount_guests > 0 or log.rowers | length > 0 %}
-                    <div class="text-sm text-gray-600 dark:text-gray-100">
-                        Ruderer:
-                        {% for rower in log.rowers -%}
-                            {{ rower.name -}}
-                            {% if not loop.last or amount_guests > 0 and log.boat.name != 'Externes Boot' %},{% endif %}
-                        {% endfor %}
-                        {% if amount_guests > 0 and log.boat.name != 'Externes Boot' %}
-                            Gäste
-                            <small class="text-gray-600 dark:text-gray-100">(ohne Account)</small>:
-                            {{ amount_guests }}
-                        {% endif %}
-                    </div>
+                    {% if not log.boat.amount_seats == 1 %}
+                        <div class="text-sm text-gray-600 dark:text-gray-100">
+                            Ruderer:
+                            {% for rower in log.rowers -%}
+                                {{ rower.name -}}
+                                {% if not loop.last or amount_guests > 0 and log.boat.name != 'Externes Boot' %},{% endif %}
+                            {% endfor %}
+                            {% if amount_guests > 0 and log.boat.name != 'Externes Boot' %}
+                                Gäste
+                                <small class="text-gray-600 dark:text-gray-100">(ohne Account)</small>:
+                                {{ amount_guests }}
+                            {% endif %}
+                        </div>
+                    {% endif %}
                 {% endif %}
             {% endif %}
         </div>
diff --git a/templates/includes/macros.html.tera b/templates/includes/macros.html.tera
index 42c9583..e591143 100644
--- a/templates/includes/macros.html.tera
+++ b/templates/includes/macros.html.tera
@@ -34,18 +34,18 @@
                 </a>
             </div>
             <div>
-                    <a href="#"
-                       class="inline-flex justify-center rounded-md bg-primary-600 mx-1 px-3 py-2 text-sm font-semibold text-white hover:bg-primary-500 focus-visible:outline focus-visible:outline-2 focus-visible:outline-offset-2 focus-visible:outline-primary-600 cursor-pointer"
-                       data-sidebar="true"
-                       data-trigger="sidebar"
-                       data-header="Menü"
-                       data-body="#mobile-menu">
-                        {% include "includes/book" %}
-                        <span class="sr-only">Logbuch</span>
-                    </a>
-                    <div class="hidden">
-                        <div id="mobile-menu">
-			                {% if "Donau Linz" in loggedin_user.roles %}
+                <a href="#"
+                   class="inline-flex justify-center rounded-md bg-primary-600 mx-1 px-3 py-2 text-sm font-semibold text-white hover:bg-primary-500 focus-visible:outline focus-visible:outline-2 focus-visible:outline-offset-2 focus-visible:outline-primary-600 cursor-pointer"
+                   data-sidebar="true"
+                   data-trigger="sidebar"
+                   data-header="Menü"
+                   data-body="#mobile-menu">
+                    {% include "includes/book" %}
+                    <span class="sr-only">Logbuch</span>
+                </a>
+                <div class="hidden">
+                    <div id="mobile-menu">
+                        {% if "Donau Linz" in loggedin_user.roles %}
                             <a href="/planned" class="block w-100 py-2 hover:text-primary-600">Geplante Ausfahrten</a>
                             <a href="/log" class="block w-100 py-2 hover:text-primary-600 border-t">Ausfahrt eintragen</a>
                             <a href="/log/show"
@@ -57,35 +57,37 @@
                                class="block w-100 py-2 hover:text-primary-600 border-t">Bootsschaden</a>
                             <a href="/boatreservation"
                                class="block w-100 py-2 hover:text-primary-600 border-t">Bootsreservierung</a>
-					{% endif %}
-                            {% if loggedin_user.weight and loggedin_user.sex and loggedin_user.dob %}
-                                <a href="/ergo" class="block w-100 py-2 hover:text-primary-600 border-t">Ergo</a>
-                            {% endif %}
-                            {% if "admin" in loggedin_user.roles or "Vorstand" in loggedin_user.roles %}
-                                <a href="/admin/user"
-                                   class="block w-100 py-2 hover:text-primary-600 border-t">Userverwaltung</a>
-                            {% endif %}
-                            {% if "admin" in loggedin_user.roles %}
-                                <a href="/admin/boat"
-                                   class="block w-100 py-2 hover:text-primary-600 border-t">Boote</a>
-                            {% endif %}
-                            <a href="/auth/logout"
-                               class="block w-100 py-2 hover:text-primary-600 border-t">Ausloggen<svg class="inline h-4"
-                         width="24"
-                         height="24"
-                         viewbox="0 0 24 24"
-                         fill="none"
-                         stroke="currentColor"
-                         stroke-width="2"
-                         stroke-linecap="round"
-                         stroke-linejoin="round"
-                         class="feather feather-log-out">
-                        <path d="M9 21H5a2 2 0 0 1-2-2V5a2 2 0 0 1 2-2h4"></path>
-                        <polyline points="16 17 21 12 16 7"></polyline>
-                        <line x1="21" y1="12" x2="9" y2="12"></line>
-                    </svg></a>
-                        </div>
+                        {% endif %}
+                        {% if loggedin_user.weight and loggedin_user.sex and loggedin_user.dob %}
+                            <a href="/ergo" class="block w-100 py-2 hover:text-primary-600 border-t">Ergo</a>
+                        {% endif %}
+                        {% if "admin" in loggedin_user.roles or "Vorstand" in loggedin_user.roles %}
+                            <a href="/admin/user"
+                               class="block w-100 py-2 hover:text-primary-600 border-t">Userverwaltung</a>
+                        {% endif %}
+                        {% if "admin" in loggedin_user.roles %}
+                            <a href="/admin/boat"
+                               class="block w-100 py-2 hover:text-primary-600 border-t">Boote</a>
+                        {% endif %}
+                        <a href="/auth/logout"
+                           class="block w-100 py-2 hover:text-primary-600 border-t">Ausloggen
+                            <svg class="inline h-4"
+                                 width="24"
+                                 height="24"
+                                 viewbox="0 0 24 24"
+                                 fill="none"
+                                 stroke="currentColor"
+                                 stroke-width="2"
+                                 stroke-linecap="round"
+                                 stroke-linejoin="round"
+                                 class="feather feather-log-out">
+                                <path d="M9 21H5a2 2 0 0 1-2-2V5a2 2 0 0 1 2-2h4"></path>
+                                <polyline points="16 17 21 12 16 7"></polyline>
+                                <line x1="21" y1="12" x2="9" y2="12"></line>
+                            </svg>
+                        </a>
                     </div>
+                </div>
             </div>
         </div>
     </header>