fix: wrong notification update with fragments

core/templates/core/base/notifications.jinja

@@ -1,5 +1,4 @@
 <div id="quick-notifications"
-     x-data='{ messages: $notifications.getAll() }'
      x-init='$notifications.addMany([
              {%- for message in messages -%}
                {%- if not message.extra_tags -%}
@@ -8,7 +7,7 @@
              {%- endfor -%}
              ])'
 >
-  <template x-for="(message, index) in messages">
+  <template x-for="(message, index) in $notifications.getAll()">
     <div class="alert" :class="`alert-${message.tag}`" x-transition>
       <span class="alert-main" x-text="message.text"></span>
       <span class="clickable" @click="messages = messages.filter((item, i) => i !== index)">

core/templates/core/macros.jinja

@@ -226,7 +226,7 @@
   <button type="button" onclick="checkbox_{{form_id}}(false);">{% trans %}Unselect All{% endtrans %}</button>
 {% endmacro %}
 
-{% macro update_notifications(messages, clear) %}
+{% macro update_notifications(messages, clear = True) %}
   {# Update notification area from new messages sent by django backend
      This is useful when performing fragment swaps to keep messages up to date
      Without this, the fragment would need to take control of the notification area and
@@ -236,15 +236,14 @@
       messages: messages from django.contrib
       clear   : optional boolean that controls if notifications should be cleared first. True is the default
   #}
-  {% set clear = clear|default(true) %}
   {% if messages %}
     <div x-init="() => {
-                 {% if clear %}
-                   $notifications.clear()
-                 {% endif %}
-                 {% for message in messages %}
-                   $notifications.{{ message.tags }}('{{ message }}')
-                 {% endfor %}
+                 {%- if clear -%}
+                   $notifications.clear();
+                 {%- endif -%}
+                 {%- for message in messages -%}
+                   $notifications.{{ message.tags }}('{{ message }}');
+                 {%- endfor -%}
                  }"></div>
   {% endif %}
 {% endmacro %}

core/views/mixins.py

@@ -54,7 +54,7 @@ class FragmentRenderer(Protocol):
     ) -> SafeString: ...
 
 
-class FragmentMixin(TemplateResponseMixin, ContextMixin):
+class FragmentMixin(TemplateResponseMixin, AllowFragment, ContextMixin):
     """Make a view buildable as a fragment that can be embedded in a template.
 
     Most fragments are used in two different ways :

eboutic/templates/eboutic/eboutic_billing_info.jinja

@@ -32,5 +32,7 @@
     </form>
   </div>
   <br>
-  {{ update_notifications(messages) }}
+  {% if is_fragment %}
+    {{ update_notifications(messages) }}
+  {% endif %}
 </div>