From 0ba5cd849c9f68b73e19747e704d316925bde45f Mon Sep 17 00:00:00 2001
From: Julie Lenaerts <julielenaerts@gmail.com>
Date: Wed, 3 Dec 2025 11:38:39 +0100
Subject: [PATCH] Remove double display of person id in the banner when there
 is a deathdate

---
 .changes/unreleased/Fixed-20251203-113832.yaml   |  6 ++++++
 .../Resources/views/Entity/person.html.twig      | 16 ++++++++--------
 2 files changed, 14 insertions(+), 8 deletions(-)
 create mode 100644 .changes/unreleased/Fixed-20251203-113832.yaml

diff --git a/.changes/unreleased/Fixed-20251203-113832.yaml b/.changes/unreleased/Fixed-20251203-113832.yaml
new file mode 100644
index 000000000..45adf3dc4
--- /dev/null
+++ b/.changes/unreleased/Fixed-20251203-113832.yaml
@@ -0,0 +1,6 @@
+kind: Fixed
+body: Remove double display of person id in the banner when there is a deathdate
+time: 2025-12-03T11:38:32.653635449+01:00
+custom:
+    Issue: "441"
+    SchemaChange: No schema change
diff --git a/src/Bundle/ChillPersonBundle/Resources/views/Entity/person.html.twig b/src/Bundle/ChillPersonBundle/Resources/views/Entity/person.html.twig
index d8a930b40..5572bd7b1 100644
--- a/src/Bundle/ChillPersonBundle/Resources/views/Entity/person.html.twig
+++ b/src/Bundle/ChillPersonBundle/Resources/views/Entity/person.html.twig
@@ -81,25 +81,25 @@
         </div>
         {%- if options['addInfo'] -%}
             <p class="moreinfo">
-                {% if person.gender is not null %}{{ person.gender.icon|chill_entity_render_box }}{% endif %}
+                {% if person.gender is not null %}{{ person.gender.icon|chill_entity_render_box }} {% endif %}
             {%- if person.deathdate is not null -%}
                 {%- if person.birthdate is not null -%}
                     {# must be on one line to avoid spaces with dash #}
                     <time datetime="{{ person.birthdate|date('Y-m-d') }}" title="{{ 'birthdate'|trans|e('html_attr') }}">{{ person.birthdate|format_date("medium") }}</time>&#x2013;
                 {%- else -%}
-                    {{ 'Date of death'|trans }}:
+                    <span>{{ 'Date of death'|trans }}: </span>
                 {%- endif -%}
                 {#- must be on one line to avoid spaces with dash -#}
                 <time datetime="{{ person.deathdate|date('Y-m-d') }}" title="{{ 'deathdate'|trans }}">{{ person.deathdate|format_date("medium") }}</time>
                 {%- if options['addAge'] -%}
                     <span class="age">&nbsp;{{ 'years_old'|trans({ 'age': person.age }) }}</span>
                 {%- endif -%}
-                {%- if options['addId'] -%}
-                    {%- set personId = person|chill_person_id_render_text %}
-                    <span class="id-number" title="{{ 'Person'|trans ~ ' ' ~ personId }}">
-                    ({{ personId }})
-                </span>
-                {%- endif -%}
+{#                {%- if options['addId'] -%}#}
+{#                    {%- set personId = person|chill_person_id_render_text %}#}
+{#                    <span class="id-number" title="{{ 'Person'|trans ~ ' ' ~ personId }}">#}
+{#                    ({{ personId }})#}
+{#                </span>#}
+{#                {%- endif -%}#}
             {%- elseif person.birthdate is not null -%}
                 <time datetime="{{ person.birthdate|date('Y-m-d') }}" title="{{ 'Birthdate'|trans }}">
                 {{ 'Born the date'|trans({'gender': person.gender ? person.gender.genderTranslation.value : 'neutral',