From 962f7d5f5c05ad630f7e8ed6b7cfa6d5ac082c29 Mon Sep 17 00:00:00 2001
From: Jens Langhammer <jens.langhammer@beryju.org>
Date: Sun, 22 Nov 2020 21:36:07 +0100
Subject: [PATCH] providers/saml: update modal to pb-modal-button

---
 .../providers/saml/processors/metadata.py     |  2 +-
 .../providers/saml/admin_metadata_modal.html  | 38 +++++++++----------
 2 files changed, 19 insertions(+), 21 deletions(-)

diff --git a/passbook/providers/saml/processors/metadata.py b/passbook/providers/saml/processors/metadata.py
index e24384c7b..8f8245239 100644
--- a/passbook/providers/saml/processors/metadata.py
+++ b/passbook/providers/saml/processors/metadata.py
@@ -105,4 +105,4 @@ class MetadataProcessor:
         for binding in self.get_bindings():
             idp_sso_descriptor.append(binding)
 
-        return tostring(entity_descriptor).decode()
+        return tostring(entity_descriptor, pretty_print=True).decode()
diff --git a/passbook/providers/saml/templates/providers/saml/admin_metadata_modal.html b/passbook/providers/saml/templates/providers/saml/admin_metadata_modal.html
index 606711433..e535dd14e 100644
--- a/passbook/providers/saml/templates/providers/saml/admin_metadata_modal.html
+++ b/passbook/providers/saml/templates/providers/saml/admin_metadata_modal.html
@@ -1,24 +1,22 @@
 {% load i18n %}
 {% load static %}
-<button class="pf-c-button pf-m-tertiary" data-target="modal" data-modal="saml-{{ provider.pk }}">{% trans 'View Metadata' %}</button>
-
-<div class="pf-c-backdrop" id="saml-{{ provider.pk }}" hidden>
-    <div class="pf-l-bullseye">
-        <div class="pf-c-modal-box pf-m-lg" role="dialog">
-            <button data-modal-close class="pf-c-button pf-m-plain" type="button" aria-label="Close dialog">
-                <i class="fas fa-times" aria-hidden="true"></i>
-            </button>
-            <div class="pf-c-modal-box__header">
-                <h1 class="pf-c-title pf-m-2xl" id="modal-title">{% trans 'Metadata' %}</h1>
-            </div>
-            <div class="pf-c-modal-box__body" id="modal-description">
-                <form method="post">
-                    <textarea class="codemirror" readonly data-cm-mode="xml">{{ metadata }}</textarea>
-                </form>
-            </div>
-            <footer class="pf-c-modal-box__footer pf-m-align-left">
-                <button data-modal-close class="pf-c-button pf-m-primary" type="button">{% trans 'Close' %}</button>
-            </footer>
+<pb-modal-button>
+    <button slot="trigger" class="pf-c-button pf-m-tertiary">
+        {% trans 'View Metadata' %}
+    </button>
+    <div slot="modal">
+        <div class="pf-c-modal-box__header">
+            <h1 class="pf-c-title pf-m-2xl" id="modal-title">{% trans 'Metadata' %}</h1>
         </div>
+        <div class="pf-c-modal-box__body" id="modal-description">
+            <form method="post">
+                <pb-codemirror mode="xml"><textarea class="pf-c-form-control"
+                        readonly>{{ metadata }}</textarea></pb-codemirror>
+            </form>
+        </div>
+        <footer class="pf-c-modal-box__footer pf-m-align-left">
+            <a class="pf-c-button pf-m-primary">{% trans 'Close' %}</a>
+        </footer>
     </div>
-</div>
+</pb-modal-button>
+