MDL-72691 reportbuilder: Encode popup data inside report actions.

Encode all popup data for report actions inside the action link
itself, because previously reloading a report via AJAX meant
actions no longer opened in popups, because the required JS was
absent from the page.

reportbuilder/amd/src/local/selectors.js

@@ -33,6 +33,9 @@ const SELECTORS = {
         systemReport: '[data-region="core_reportbuilder/system-report"]',
         filtersForm: '[data-region="filters-form"]',
     },
+    actions: {
+        reportActionPopup: '[data-action="report-action-popup"]',
+    }
 };
 
 /**

reportbuilder/amd/src/report.js

@@ -46,4 +46,15 @@ export const init = reportId => {
         await setPageNumber(tableRoot, pageNumber, false)
             .then(refreshTableContent);
     });
+
+    // Listen for trigger popup events.
+    document.addEventListener('click', event => {
+        const reportActionPopup = event.target.closest(reportSelectors.actions.reportActionPopup);
+        if (reportActionPopup === null) {
+            return;
+        }
+        event.preventDefault();
+        const popupAction = JSON.parse(reportActionPopup.dataset.popupAction);
+        window.openpopup(event, popupAction.jsfunctionargs);
+    });
 };

reportbuilder/classes/local/report/action.php

@@ -109,9 +109,12 @@ public function get_action_link(stdClass $row): ?string {
             self::replace_placeholders($this->url->params(), $row)
         );
 
-        $popupaction = $this->popup ? new popup_action('click', $url) : null;
+        if ($this->popup) {
+            $this->attributes['data-action'] = 'report-action-popup';
+            $this->attributes['data-popup-action'] = json_encode(new popup_action('click', $url));
+        }
 
-        return $OUTPUT->action_icon($url, $this->icon, $popupaction, self::replace_placeholders($this->attributes, $row));
+        return $OUTPUT->action_icon($url, $this->icon, null, self::replace_placeholders($this->attributes, $row));
     }
 
     /**