From 39e0ed29628d95abe0701b200397ff300ddc0637 Mon Sep 17 00:00:00 2001
From: Jens Langhammer <jens.langhammer@beryju.org>
Date: Mon, 19 Dec 2022 10:54:51 +0100
Subject: [PATCH] web/admin: better show metadata download for saml provider

Signed-off-by: Jens Langhammer <jens.langhammer@beryju.org>
---
 .../providers/saml/SAMLProviderViewPage.ts    | 69 +++++++++++++------
 1 file changed, 48 insertions(+), 21 deletions(-)

diff --git a/web/src/admin/providers/saml/SAMLProviderViewPage.ts b/web/src/admin/providers/saml/SAMLProviderViewPage.ts
index b803f55fe..9bdc451ef 100644
--- a/web/src/admin/providers/saml/SAMLProviderViewPage.ts
+++ b/web/src/admin/providers/saml/SAMLProviderViewPage.ts
@@ -75,32 +75,59 @@ export class SAMLProviderViewPage extends AKElement {
     }
 
     async renderRelatedObjects(): Promise<TemplateResult> {
-        if (!this.provider?.signingKp) {
-            return Promise.resolve(html``);
+        const relatedObjects = [];
+        if (this.provider?.assignedApplicationName) {
+            relatedObjects.push(html`<div class="pf-c-description-list__group">
+                <dt class="pf-c-description-list__term">
+                    <span class="pf-c-description-list__text">${t`Metadata`}</span>
+                </dt>
+                <dd class="pf-c-description-list__description">
+                    <div class="pf-c-description-list__text">
+                        <a
+                            class="pf-c-button pf-m-primary"
+                            target="_blank"
+                            href=${ifDefined(this.provider?.urlDownloadMetadata)}
+                        >
+                            ${t`Download`}
+                        </a>
+                        <ak-action-button
+                            class="pf-m-secondary"
+                            .apiRequest=${() => {
+                                return navigator.clipboard.writeText(
+                                    this.provider?.urlDownloadMetadata || "",
+                                );
+                            }}
+                        >
+                            ${t`Copy download URL`}
+                        </ak-action-button>
+                    </div>
+                </dd>
+            </div>`);
+        }
+        if (this.provider?.signingKp) {
+            const kp = await new CryptoApi(DEFAULT_CONFIG).cryptoCertificatekeypairsRetrieve({
+                kpUuid: this.provider.signingKp,
+            });
+            relatedObjects.push(html`<div class="pf-c-description-list__group">
+                <dt class="pf-c-description-list__term">
+                    <span class="pf-c-description-list__text"
+                        >${t`Download signing certificate`}</span
+                    >
+                </dt>
+                <dd class="pf-c-description-list__description">
+                    <div class="pf-c-description-list__text">
+                        <a class="pf-c-button pf-m-primary" href=${kp.certificateDownloadUrl}
+                            >${t`Download`}</a
+                        >
+                    </div>
+                </dd>
+            </div>`);
         }
-        const kp = await new CryptoApi(DEFAULT_CONFIG).cryptoCertificatekeypairsRetrieve({
-            kpUuid: this.provider.signingKp,
-        });
         return html` <div class="pf-c-card pf-l-grid__item pf-m-12-col">
             <div class="pf-c-card__title">${t`Related objects`}</div>
             <div class="pf-c-card__body">
                 <dl class="pf-c-description-list pf-m-2-col">
-                    <div class="pf-c-description-list__group">
-                        <dt class="pf-c-description-list__term">
-                            <span class="pf-c-description-list__text"
-                                >${t`Download signing certificate`}</span
-                            >
-                        </dt>
-                        <dd class="pf-c-description-list__description">
-                            <div class="pf-c-description-list__text">
-                                <a
-                                    class="pf-c-button pf-m-primary"
-                                    href=${kp.certificateDownloadUrl}
-                                    >${t`Download`}</a
-                                >
-                            </div>
-                        </dd>
-                    </div>
+                    ${relatedObjects.length > 0 ? relatedObjects : html`-`}
                 </dl>
             </div>
         </div>`;