Fix bug in Copy Email Addresses button when there is no anchor

meeting-room-booking-system · Aug 7, 2023 · 3c911b8 · 3c911b8
1 parent 6359fe8
commit 3c911b8
Showing 1 changed file with 13 additions and 7 deletions.
diff --git a/web/js/report.js.php b/web/js/report.js.php
@@ -318,15 +318,21 @@ function (result) {
         var scheme = 'mailto:';
         var message;
         $.each(dt.column('#col_create_by').data(), function(index, value) {
-          var href = $(value).attr('href');
-          if ((href !== undefined) && href.startsWith(scheme))
-          {
-            var address = href.substring(scheme.length);
-            if ((address !== '') && !result.includes(address))
-            {
-              result.push(address);
+          try {
+            var href = $(value).attr('href');
+            if ((href !== undefined) && href.startsWith(scheme)) {
+              var address = href.substring(scheme.length);
+              if ((address !== '') && !result.includes(address)) {
+                result.push(address);
+              }
             }
           }
+          catch (error) {
+            <?php
+            // No need to do anything. This will catch the cases when $(value) fails because
+            // value is not a valid anchor element, and so we are not interested in it anyway.
+            ?>
+          }
         });
         result.sort();
         navigator.clipboard.writeText(result.join(', '))