Added better formatted extra section for certain log types

src/Entity/LogSystem/SecurityEventLogEntry.php

@@ -113,7 +113,7 @@ class SecurityEventLogEntry extends AbstractLogEntry
     {
         $key = $this->extra['e'];
 
-        return static::SECURITY_TYPE_MAPPING[$key] ?? 'unkown';
+        return static::SECURITY_TYPE_MAPPING[$key] ?? 'unknown';
     }
 
     /**

templates/log_system/details/_extra_database_updated.html.twig

@@ -0,0 +1,23 @@
+{# @var entry \App\Entity\LogSystem\DatabaseUpdatedLogEntry #}
+
+{% if entry.successful %}
+    <h5><span class="badge bg-success badge-success">
+            <i class="fa-solid fa-check"></i>
+            {% trans %}log.database_updated.success{% endtrans %}
+        </span></h5>
+{% else %}
+    <h5><span class="badge bg-danger badge-danger">
+            <i class="fa-solid fa-xmark"></i>
+            {% trans %}log.database_updated.failed{% endtrans %}</span>
+    </h5>
+{% endif %}
+
+<span class="badge bg-secondary badge-secondary badge-pill" title="{% trans %}log.database_updated.old_version{% endtrans %}">
+    <i class="fa-solid fa-database"></i>
+    {{ entry.oldVersion }}
+</span>
+<i class="fa-solid fa-arrow-right-long"></i>
+<span class="badge bg-primary badge-primary badge-pill" title="{% trans %}log.database_updated.new_version{% endtrans %}">
+    <i class="fa-solid fa-database"></i>
+    {{ entry.newVersion }}
+</span>

templates/log_system/details/_extra_security_event.html.twig

@@ -0,0 +1,9 @@
+{# @var entry \App\Entity\LogSystem\UserLoginLogEntry #}
+
+IP:  
+<span class="badge bg-primary badge-primary">
+    <i class="fa-solid fa-network-wired"></i>
+    {{ entry.iPAddress }}
+</span>
+
+<p class="text-muted">{% trans %}log.user_login.ip_anonymize_hint{% endtrans %}</p>

templates/log_system/details/_extra_user_login.html.twig

@@ -0,0 +1,9 @@
+{# @var entry \App\Entity\LogSystem\UserLoginLogEntry #}
+
+{% trans %}log.user_login.login_from_ip{% endtrans %}:  
+<span class="badge bg-primary badge-primary">
+    <i class="fa-solid fa-network-wired"></i>
+    {{ entry.iPAddress }}
+</span>
+
+<p class="text-muted">{% trans %}log.user_login.ip_anonymize_hint{% endtrans %}</p>

templates/log_system/details/_extra_user_not_allowed.html.twig

@@ -0,0 +1,4 @@
+{# @var entry \App\Entity\LogSystem\UserNotAllowedLogEntry #}
+
+{% trans %}log.user_not_allowed.unauthorized_access_attempt_to{% endtrans %}: &nbsp;<code>{{ entry.path }}</code>
+<p class="text-muted">{% trans %}log.user_not_allowed.hint{% endtrans %}</p>

templates/log_system/details/log_details.html.twig

@@ -58,6 +58,19 @@
     </table>
 
     <div class="card-body">
+        {# This assignment is to improve autocomplete on the subpages, as PHPstorm ignores typehints for log_entry #}
+        {% set entry = log_entry %}
+        {% if log_entry is instanceof('App\\Entity\\LogSystem\\DatabaseUpdatedLogEntry') %}
+            {% include "log_system/details/_extra_database_updated.html.twig" %}
+        {% elseif log_entry is instanceof('App\\Entity\\LogSystem\\UserLoginLogEntry')
+            or log_entry is instanceof('App\\Entity\\LogSystem\\UserLogoutLogEntry')  %}
+            {% include "log_system/details/_extra_user_login.html.twig" %}
+        {% elseif log_entry is instanceof('App\\Entity\\LogSystem\\UserNotAllowedLogEntry') %}
+            {% include "log_system/details/_extra_user_not_allowed.html.twig" %}
+        {% elseif log_entry is instanceof('App\\Entity\\LogSystem\\SecurityEventLogEntry') %}
+            {% include "log_system/details/_extra_security_event.html.twig" %}
+        {% else %}
             {{ extra_html | raw }}
+        {% endif %}
     </div>
 {% endblock %}

translations/messages.en.xlf

@@ -11295,5 +11295,29 @@ Element 3</target>
         <target>Log details</target>
       </segment>
     </unit>
+    <unit id="aeYMkHS" name="log.user_login.login_from_ip">
+      <segment>
+        <source>log.user_login.login_from_ip</source>
+        <target>Login from IP address</target>
+      </segment>
+    </unit>
+    <unit id="9jOklgS" name="log.user_login.ip_anonymize_hint">
+      <segment>
+        <source>log.user_login.ip_anonymize_hint</source>
+        <target>If the last digits of the IP address are missing, then the GPDR mode is enabled, in which IP addresses are anynomized.</target>
+      </segment>
+    </unit>
+    <unit id="kaMyDVi" name="log.user_not_allowed.unauthorized_access_attempt_to">
+      <segment>
+        <source>log.user_not_allowed.unauthorized_access_attempt_to</source>
+        <target>Unauthorized access attempt to page</target>
+      </segment>
+    </unit>
+    <unit id="EibB1Wh" name="log.user_not_allowed.hint">
+      <segment>
+        <source>log.user_not_allowed.hint</source>
+        <target>The request was blocked. No action should be required.</target>
+      </segment>
+    </unit>
   </file>
 </xliff>

translations/security.de.xlf

@@ -1,14 +1,14 @@
 <?xml version="1.0" encoding="utf-8"?>
 <xliff xmlns="urn:oasis:names:tc:xliff:document:2.0" version="2.0" srcLang="en" trgLang="de">
-  <file id="security.en">
+  <file id="security.de">
     <unit id="aazoCks" name="user.login_error.user_disabled">
-      <segment state="translated">
+      <segment>
         <source>user.login_error.user_disabled</source>
         <target>Ihr Account ist deaktiviert! Kontaktiere einen Administrator, wenn Sie denken, dass dies ein Fehler ist.</target>
       </segment>
     </unit>
     <unit id="Dpb9AmY" name="saml.error.cannot_login_local_user_per_saml">
-      <segment state="translated">
+      <segment>
         <source>saml.error.cannot_login_local_user_per_saml</source>
         <target>Sie können sich per SSO nicht als lokaler Nutzer einloggen! Nutzen Sie stattdessen ihr lokales Passwort.</target>
       </segment>

translations/validators.de.xlf

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <xliff xmlns="urn:oasis:names:tc:xliff:document:2.0" version="2.0" srcLang="en" trgLang="de">
-  <file id="validators.en">
+  <file id="validators.de">
     <unit id="xevSdCK" name="part.master_attachment.must_be_picture">
       <notes>
         <note category="file-source" priority="1">Part-DB1\src\Entity\Attachments\AttachmentContainingDBElement.php:0</note>
@@ -37,7 +37,7 @@
         <note priority="1">Part-DB1\src\Entity\UserSystem\Group.php:0</note>
         <note priority="1">Part-DB1\src\Entity\UserSystem\User.php:0</note>
       </notes>
-      <segment state="translated">
+      <segment>
         <source>part.master_attachment.must_be_picture</source>
         <target>Der Vorschauanhang muss ein gültiges Bild sein!</target>
       </segment>
@@ -82,7 +82,7 @@
         <note priority="1">src\Entity\StructuralDBElement.php:0</note>
         <note priority="1">src\Entity\Supplier.php:0</note>
       </notes>
-      <segment state="translated">
+      <segment>
         <source>structural.entity.unique_name</source>
         <target>Es kann auf jeder Ebene nur ein Objekt mit dem gleichem Namen geben!</target>
       </segment>
@@ -102,7 +102,7 @@
         <note category="file-source" priority="1">Part-DB1\src\Entity\Parameters\StorelocationParameter.php:0</note>
         <note category="file-source" priority="1">Part-DB1\src\Entity\Parameters\SupplierParameter.php:0</note>
       </notes>
-      <segment state="translated">
+      <segment>
         <source>parameters.validator.min_lesser_typical</source>
         <target>Wert muss kleiner oder gleich als der typische Wert sein ({{ compared_value }}).</target>
       </segment>
@@ -122,7 +122,7 @@
         <note category="file-source" priority="1">Part-DB1\src\Entity\Parameters\StorelocationParameter.php:0</note>
         <note category="file-source" priority="1">Part-DB1\src\Entity\Parameters\SupplierParameter.php:0</note>
       </notes>
-      <segment state="translated">
+      <segment>
         <source>parameters.validator.min_lesser_max</source>
         <target>Wert muss kleiner als der Maximalwert sein ({{ compared_value }}).</target>
       </segment>
@@ -142,7 +142,7 @@
         <note category="file-source" priority="1">Part-DB1\src\Entity\Parameters\StorelocationParameter.php:0</note>
         <note category="file-source" priority="1">Part-DB1\src\Entity\Parameters\SupplierParameter.php:0</note>
       </notes>
-      <segment state="translated">
+      <segment>
         <source>parameters.validator.max_greater_typical</source>
         <target>Wert muss größer oder gleich dem typischen Wert sein ({{ compared_value }}).</target>
       </segment>
@@ -152,7 +152,7 @@
         <note category="file-source" priority="1">Part-DB1\src\Entity\UserSystem\User.php:0</note>
         <note priority="1">Part-DB1\src\Entity\UserSystem\User.php:0</note>
       </notes>
-      <segment state="translated">
+      <segment>
         <source>validator.user.username_already_used</source>
         <target>Es existiert bereits ein Benutzer mit diesem Namen.</target>
       </segment>
@@ -162,7 +162,7 @@
         <note category="file-source" priority="1">Part-DB1\src\Entity\UserSystem\User.php:0</note>
         <note priority="1">Part-DB1\src\Entity\UserSystem\User.php:0</note>
       </notes>
-      <segment state="translated">
+      <segment>
         <source>user.invalid_username</source>
         <target>Der Benutzername darf nur Buchstaben, Zahlen, Unterstriche, Punkte, Plus- oder Minuszeichen enthalten.</target>
       </segment>
@@ -171,7 +171,7 @@
       <notes>
         <note category="state" priority="1">obsolete</note>
       </notes>
-      <segment state="translated">
+      <segment>
         <source>validator.noneofitschild.self</source>
         <target>Ein Element kann nicht sein eigenenes übergeordnetes Element sein!</target>
       </segment>
@@ -180,140 +180,130 @@
       <notes>
         <note category="state" priority="1">obsolete</note>
       </notes>
-      <segment state="final">
+      <segment>
         <source>validator.noneofitschild.children</source>
         <target>Ein Kindelement kann nicht das übergeordnete Element sein!</target>
       </segment>
     </unit>
     <unit id="ayNr6QK" name="validator.select_valid_category">
-      <segment state="translated">
+      <segment>
         <source>validator.select_valid_category</source>
         <target>Bitte wählen Sie eine gültige Kategorie.</target>
       </segment>
     </unit>
     <unit id="6vIlN5q" name="validator.part_lot.only_existing">
-      <segment state="translated">
+      <segment>
         <source>validator.part_lot.only_existing</source>
         <target>Der Lagerort wurde als "nur bestehende Teile" markiert, daher können keine neuen Teile hinzugefügt werden.</target>
       </segment>
     </unit>
     <unit id="3xoKOIS" name="validator.part_lot.location_full.no_increase">
-      <segment state="translated">
+      <segment>
         <source>validator.part_lot.location_full.no_increase</source>
         <target>Lagerort ist voll. Bestand kann nicht erhöht werden (neuer Wert muss kleiner sein als {{old_amount}}).</target>
       </segment>
     </unit>
     <unit id="R6Ov4Yt" name="validator.part_lot.location_full">
-      <segment state="final">
+      <segment>
         <source>validator.part_lot.location_full</source>
         <target>Der Lagerort ist voll, daher können keine neue Teile hinzugefügt werden.</target>
       </segment>
     </unit>
     <unit id="BNQk2e7" name="validator.part_lot.single_part">
-      <segment state="final">
+      <segment>
         <source>validator.part_lot.single_part</source>
         <target>Der Lagerort wurde als "Nur ein Bauteil" markiert, daher kann kein neues Bauteil hinzugefügt werden.</target>
       </segment>
     </unit>
     <unit id="4gPskOG" name="validator.attachment.must_not_be_null">
-      <segment state="translated">
+      <segment>
         <source>validator.attachment.must_not_be_null</source>
         <target>Sie müssen ein Dateitypen auswählen!</target>
       </segment>
     </unit>
     <unit id="cDDVrWT" name="validator.orderdetail.supplier_must_not_be_null">
-      <segment state="translated">
+      <segment>
         <source>validator.orderdetail.supplier_must_not_be_null</source>
         <target>Sie müssen einen Lieferanten auswählen!</target>
       </segment>
     </unit>
     <unit id="k5DDdB4" name="validator.measurement_unit.use_si_prefix_needs_unit">
-      <segment state="translated">
+      <segment>
         <source>validator.measurement_unit.use_si_prefix_needs_unit</source>
         <target>Um SI-Prefixe zu aktivieren, müssen Sie einen Einheitensymbol setzen!</target>
       </segment>
     </unit>
     <unit id="DuzIOCr" name="part.ipn.must_be_unique">
-      <segment state="translated">
+      <segment>
         <source>part.ipn.must_be_unique</source>
         <target>Die Internal Part Number (IPN) muss einzigartig sein. Der Wert {{value}} wird bereits benutzt!</target>
       </segment>
     </unit>
     <unit id="Z4Kuuo2" name="validator.project.bom_entry.name_or_part_needed">
-      <segment state="translated">
+      <segment>
         <source>validator.project.bom_entry.name_or_part_needed</source>
         <target>Sie müssen ein Bauteil auswählen, oder einen Namen für ein nicht-Bauteil BOM-Eintrag setzen!</target>
       </segment>
     </unit>
     <unit id="WF_v4ih" name="project.bom_entry.name_already_in_bom">
-      <segment state="translated">
+      <segment>
         <source>project.bom_entry.name_already_in_bom</source>
         <target>Es gibt bereits einen BOM Eintrag mit diesem Namen!</target>
       </segment>
     </unit>
     <unit id="5v4p85H" name="project.bom_entry.part_already_in_bom">
-      <segment state="translated">
+      <segment>
         <source>project.bom_entry.part_already_in_bom</source>
         <target>Dieses Bauteil existiert bereits in der BOM!</target>
       </segment>
     </unit>
     <unit id="3lM32Tw" name="project.bom_entry.mountnames_quantity_mismatch">
-      <segment state="translated">
+      <segment>
         <source>project.bom_entry.mountnames_quantity_mismatch</source>
         <target>Die Anzahl der Bestückungsnamen muss mit der Menge der zu bestückenden Bauteile übereinstimmen!</target>
       </segment>
     </unit>
     <unit id="x47D5WT" name="project.bom_entry.can_not_add_own_builds_part">
-      <segment state="translated">
+      <segment>
         <source>project.bom_entry.can_not_add_own_builds_part</source>
         <target>Die BOM eines Projektes kann nicht das eigene Produktionsbauteil enthalten!</target>
       </segment>
     </unit>
     <unit id="2x2XDI_" name="project.bom_has_to_include_all_subelement_parts">
-      <segment state="translated">
+      <segment>
         <source>project.bom_has_to_include_all_subelement_parts</source>
         <target>Die Projekt-BOM muss alle Produktionsbauteile der Unterprojekte enthalten. Bauteil %part_name% des Projektes %project_name% fehlt!</target>
       </segment>
     </unit>
     <unit id="U9b1EzD" name="project.bom_entry.price_not_allowed_on_parts">
-      <segment state="translated">
+      <segment>
         <source>project.bom_entry.price_not_allowed_on_parts</source>
         <target>Sie können keinen Preis für Bauteil-BOM-Einträge definieren. Definieren Sie die Preise stattdessen auf dem Bauteil.</target>
       </segment>
     </unit>
     <unit id="ID056SR" name="validator.project_build.lot_bigger_than_needed">
-      <segment state="translated">
+      <segment>
         <source>validator.project_build.lot_bigger_than_needed</source>
         <target>Sie haben mehr zur Entnahme ausgewählt als notwendig. Entfernen Sie die überflüssige Anzahl.</target>
       </segment>
     </unit>
     <unit id="6hV5UqD" name="validator.project_build.lot_smaller_than_needed">
-      <segment state="translated">
+      <segment>
         <source>validator.project_build.lot_smaller_than_needed</source>
         <target>Sie haben weniger zur Entnahme ausgewählt, als zum Bau notwendig ist! Fügen Sie mehr hinzu.</target>
       </segment>
     </unit>
     <unit id="G9ZKt.4" name="part.name.must_match_category_regex">
-      <segment state="translated">
+      <segment>
         <source>part.name.must_match_category_regex</source>
         <target>Der Bauteilename entspricht nicht dem regulären Ausdruck, der von der Kategorie vorgegeben wurde: %regex%</target>
       </segment>
     </unit>
     <unit id="m8kMFhf" name="validator.attachment.name_not_blank">
-      <segment state="translated">
+      <segment>
         <source>validator.attachment.name_not_blank</source>
         <target>Wählen Sie einen Wert, oder laden Sie eine Datei hoch, um dessen Dateiname automatisch als Namen für diesen Anhang zu nutzen.</target>
       </segment>
-    </unit>
-        <unit id="0IF0VIF" name="validator.isSelectable">
-          <notes>
-            <note priority="1">obsolete</note>
-            <note category="state" priority="1">obsolete</note>
-          </notes>
-          <segment>
-            <source>validator.isSelectable</source>
-            <target>Das Element muss auswählbar sein!</target>
-          </segment>
     </unit>
     <unit id="0IF0VIF" name="validator.isSelectable">
       <notes>