From 6eddd69843d78bf442324dd10c501b79bbaf89d1 Mon Sep 17 00:00:00 2001
From: n-peugnet <n.peugnet@free.fr>
Date: Thu, 7 Nov 2019 17:58:08 +0100
Subject: feat: include sentry in frontend

---
 app/view/templates/layout.php | 7 +++++++
 1 file changed, 7 insertions(+)

(limited to 'app/view')

diff --git a/app/view/templates/layout.php b/app/view/templates/layout.php
index 85351d6..c249622 100644
--- a/app/view/templates/layout.php
+++ b/app/view/templates/layout.php
@@ -18,6 +18,13 @@
         echo '<link rel="stylesheet" href="' . Wcms\Model::csspath() . Wcms\Config::interfacecss() . '">';
     }
     ?>
+    <script>
+        const sentrydsn = '<?= Wcms\Config::sentrydsn() ?>';
+        const version = '<?= getversion() ?>';
+        const url = '<?= Wcms\Config::url() ?>';
+        const basepath = '<?= Wcms\Config::basepath() ?>';
+    </script>
+    <script src="<?= Wcms\Model::jspath() ?>sentry.bundle.js"></script>
 </head>
 
 
-- 
cgit v1.2.3


From d8032078c39edb42acb99a1de49283daba199d74 Mon Sep 17 00:00:00 2001
From: n-peugnet <n.peugnet@free.fr>
Date: Thu, 7 Nov 2019 19:11:04 +0100
Subject: feat: don't include Sentry in front if undefined

---
 app/view/templates/layout.php | 2 ++
 1 file changed, 2 insertions(+)

(limited to 'app/view')

diff --git a/app/view/templates/layout.php b/app/view/templates/layout.php
index c249622..ef28748 100644
--- a/app/view/templates/layout.php
+++ b/app/view/templates/layout.php
@@ -17,6 +17,7 @@
     if (!empty(Wcms\Config::interfacecss())) {
         echo '<link rel="stylesheet" href="' . Wcms\Model::csspath() . Wcms\Config::interfacecss() . '">';
     }
+    if (!empty(Wcms\Config::sentrydsn())) {
     ?>
     <script>
         const sentrydsn = '<?= Wcms\Config::sentrydsn() ?>';
@@ -25,6 +26,7 @@
         const basepath = '<?= Wcms\Config::basepath() ?>';
     </script>
     <script src="<?= Wcms\Model::jspath() ?>sentry.bundle.js"></script>
+    <?php } ?>
 </head>
 
 
-- 
cgit v1.2.3


From c1843c0a2ef49de3d2030398f5c14549cc1791ab Mon Sep 17 00:00:00 2001
From: n-peugnet <n.peugnet@free.fr>
Date: Fri, 15 Nov 2019 15:57:46 +0100
Subject: feat: load Sentry browser as an external script

- better way to check if we need to report errors in PHP
- fix make file $(js_bundles) dependencies (add webpack conf)
- update sentry-browser sdk and move it to dev requirements
---
 app/view/templates/layout.php | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

(limited to 'app/view')

diff --git a/app/view/templates/layout.php b/app/view/templates/layout.php
index ef28748..9259fda 100644
--- a/app/view/templates/layout.php
+++ b/app/view/templates/layout.php
@@ -17,7 +17,7 @@
     if (!empty(Wcms\Config::interfacecss())) {
         echo '<link rel="stylesheet" href="' . Wcms\Model::csspath() . Wcms\Config::interfacecss() . '">';
     }
-    if (!empty(Wcms\Config::sentrydsn())) {
+    if (isreportingerrors()) {
     ?>
     <script>
         const sentrydsn = '<?= Wcms\Config::sentrydsn() ?>';
@@ -25,6 +25,7 @@
         const url = '<?= Wcms\Config::url() ?>';
         const basepath = '<?= Wcms\Config::basepath() ?>';
     </script>
+    <script src="https://browser.sentry-cdn.com/5.9.0/bundle.min.js"></script>
     <script src="<?= Wcms\Model::jspath() ?>sentry.bundle.js"></script>
     <?php } ?>
 </head>
-- 
cgit v1.2.3


From a49dca5b92c141cafd89a9908309036179973410 Mon Sep 17 00:00:00 2001
From: n-peugnet <n.peugnet@free.fr>
Date: Fri, 15 Nov 2019 15:59:55 +0100
Subject: fix(home): Cannot set property 'innerHTML' of null

home.bundle.js was included even if the user was not connected
---
 app/view/templates/home.php | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'app/view')

diff --git a/app/view/templates/home.php b/app/view/templates/home.php
index 9758c95..01bc0e4 100644
--- a/app/view/templates/home.php
+++ b/app/view/templates/home.php
@@ -173,10 +173,10 @@
 
     <?php $this->insert('footer', ['footer' => $footer]) ?>
 
-    <?php } ?>
+    <script src="<?= Wcms\Model::jspath() ?>home.bundle.js"></script>
 
+    <?php } ?>
 
-    <script src="<?= Wcms\Model::jspath() ?>home.bundle.js"></script>
 </body>
 
 
-- 
cgit v1.2.3