From 0054fe08868216cd3e380f62efdfdfaedbb6acf9 Mon Sep 17 00:00:00 2001
From: Dennis Reimann <mail@dennisreimann.de>
Date: Tue, 27 Jul 2021 19:52:57 +0200
Subject: [PATCH] Refactor copy to clipboard

Separated out of #2701, prerequisite for the LNbank plugin integration.
---
 BTCPayServer/Views/Manage/APIKeys.cshtml               |  1 -
 .../ShowLightningNodeInfo.cshtml                       |  7 +------
 BTCPayServer/Views/PullPayment/ViewPullPayment.cshtml  |  2 +-
 BTCPayServer/Views/Wallets/WalletReceive.cshtml        | 10 ----------
 BTCPayServer/bundleconfig.json                         |  1 +
 BTCPayServer/wwwroot/js/copy-to-clipboard.js           |  8 +++++++-
 6 files changed, 10 insertions(+), 19 deletions(-)
diff --git a/BTCPayServer/Views/Manage/APIKeys.cshtml b/BTCPayServer/Views/Manage/APIKeys.cshtml
index 5ef74bf5e..b295ac37d 100644
--- a/BTCPayServer/Views/Manage/APIKeys.cshtml
+++ b/BTCPayServer/Views/Manage/APIKeys.cshtml
@@ -86,7 +86,6 @@
 @section PageFootContent {
 
 <bundle name="wwwroot/bundles/camera-bundle.min.js"></bundle>
-<script src="~/js/copy-to-clipboard.js" asp-append-version="true"></script>
 <script>
     document.addEventListener("DOMContentLoaded", function () {
         $("[data-reveal-btn]").on("click", function (){
diff --git a/BTCPayServer/Views/PublicLightningNodeInfo/ShowLightningNodeInfo.cshtml b/BTCPayServer/Views/PublicLightningNodeInfo/ShowLightningNodeInfo.cshtml
index 059838536..7a4b2ad2d 100644
--- a/BTCPayServer/Views/PublicLightningNodeInfo/ShowLightningNodeInfo.cshtml
+++ b/BTCPayServer/Views/PublicLightningNodeInfo/ShowLightningNodeInfo.cshtml
@@ -21,12 +21,6 @@
     <link href="@Context.Request.GetRelativePathOrAbsolute(theme.ThemeCssUri)" rel="stylesheet" asp-append-version="true"/>
     <link href="~/vendor/font-awesome/css/font-awesome.min.css" rel="stylesheet" asp-append-version="true" />
     <link href="~/main/qrcode.css" rel="stylesheet" asp-append-version="true" />
-    <script src="~/js/copy-to-clipboard.js"></script>
-    <script>
-    window.onload = function (){        
-        document.querySelectorAll('[data-clipboard]').forEach(value => value.addEventListener('click', window.copyToClipboard));
-    }
-    </script>
 </head>
 <body>
     <div id="app" class="container">
@@ -63,5 +57,6 @@
             </div>
         </div>
     </div>
+    <script src="~/js/copy-to-clipboard.js" asp-append-version="true"></script>
 </body>
 </html>
diff --git a/BTCPayServer/Views/PullPayment/ViewPullPayment.cshtml b/BTCPayServer/Views/PullPayment/ViewPullPayment.cshtml
index 98391f775..0502e934f 100644
--- a/BTCPayServer/Views/PullPayment/ViewPullPayment.cshtml
+++ b/BTCPayServer/Views/PullPayment/ViewPullPayment.cshtml
@@ -39,7 +39,6 @@
         <link href="@Model.CustomCSSLink" rel="stylesheet" />
     }
     @Safe.Raw(Model.EmbeddedCSS)
-    <script src="~/js/copy-to-clipboard.js" asp-append-version="true"></script>
     <noscript>
         <style>
             .hide-when-js, [v-cloak] { display: block !important; }
@@ -191,6 +190,7 @@
             </div>
         </footer>
     </div>
+    <script src="~/js/copy-to-clipboard.js" asp-append-version="true"></script>
     <script>
         document.getElementById("copyLink").addEventListener("click", window.copyUrlToClipboard);
     </script>
diff --git a/BTCPayServer/Views/Wallets/WalletReceive.cshtml b/BTCPayServer/Views/Wallets/WalletReceive.cshtml
index cfdead84e..1022fff57 100644
--- a/BTCPayServer/Views/Wallets/WalletReceive.cshtml
+++ b/BTCPayServer/Views/Wallets/WalletReceive.cshtml
@@ -10,16 +10,6 @@
     <link href="~/main/qrcode.css" rel="stylesheet" asp-append-version="true"/>
 }
 
-@section PageFootContent
-{
-    <script src="~/js/copy-to-clipboard.js"></script>
-    <script>
-        window.onload = function (){        
-            document.querySelectorAll('[data-clipboard]').forEach(value => value.addEventListener('click', window.copyToClipboard));
-        }
-    </script>
-}
-
 <div class="row no-gutters">
     <div class="col-lg-8 mx-auto my-auto">
         <form method="post" asp-action="WalletReceive" class="card text-center">
diff --git a/BTCPayServer/bundleconfig.json b/BTCPayServer/bundleconfig.json
index d7f387f2a..f197ab6c8 100644
--- a/BTCPayServer/bundleconfig.json
+++ b/BTCPayServer/bundleconfig.json
@@ -23,6 +23,7 @@
         "inputFiles": [
             "wwwroot/vendor/moment/moment.min.js",
             "wwwroot/vendor/flatpickr/flatpickr.js",
+            "wwwroot/js/copy-to-clipboard.js",
             "wwwroot/main/utils.js",
             "wwwroot/main/site.js"
         ]
diff --git a/BTCPayServer/wwwroot/js/copy-to-clipboard.js b/BTCPayServer/wwwroot/js/copy-to-clipboard.js
index 1915471e1..da3c9308b 100644
--- a/BTCPayServer/wwwroot/js/copy-to-clipboard.js
+++ b/BTCPayServer/wwwroot/js/copy-to-clipboard.js
@@ -18,5 +18,11 @@ window.copyToClipboard = function (e, text) {
 }
 
 window.copyUrlToClipboard = function (e) {
-    window.copyToClipboard(e,  window.location);
+    window.copyToClipboard(e, window.location)
 }
+
+document.addEventListener("DOMContentLoaded", () => {
+    document.querySelectorAll("[data-clipboard]").forEach(item => {
+        item.addEventListener("click", window.copyToClipboard)
+    })
+})
