From dc8710d89e4591f6fe3067c44c472321c245223a Mon Sep 17 00:00:00 2001
From: Sami Mokaddem <sami.mokaddem@circl.lu>
Date: Thu, 27 Jan 2022 10:21:55 +0100
Subject: [PATCH] fix: [users:view] Correctly reload authkey child panel when
 performing operations

---
 templates/AuthKeys/index.php                                | 6 ++++--
 .../element/genericElements/ListTopBar/element_simple.php   | 6 ++++--
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/templates/AuthKeys/index.php b/templates/AuthKeys/index.php
index b5fbc35..4f42c1b 100644
--- a/templates/AuthKeys/index.php
+++ b/templates/AuthKeys/index.php
@@ -10,7 +10,8 @@ echo $this->element('genericElements/IndexTable/index_table', [
                         'data' => [
                             'type' => 'simple',
                             'text' => __('Add authentication key'),
-                            'popover_url' => '/authKeys/add'
+                            'popover_url' => '/authKeys/add',
+                            'reload_url' => $this->request->getRequestTarget()
                         ]
                     ]
                 ],
@@ -65,7 +66,8 @@ echo $this->element('genericElements/IndexTable/index_table', [
             [
                 'open_modal' => '/authKeys/delete/[onclick_params_data_path]',
                 'modal_params_data_path' => 'id',
-                'icon' => 'trash'
+                'icon' => 'trash',
+                'reload_url' => $this->request->getRequestTarget()
             ]
         ]
     ]
diff --git a/templates/element/genericElements/ListTopBar/element_simple.php b/templates/element/genericElements/ListTopBar/element_simple.php
index a14976f..5573254 100644
--- a/templates/element/genericElements/ListTopBar/element_simple.php
+++ b/templates/element/genericElements/ListTopBar/element_simple.php
@@ -1,8 +1,10 @@
 <?php
+    $seed = 'f_' . mt_rand();
     if (!isset($data['requirement']) || $data['requirement']) {
         if (!empty($data['popover_url'])) {
             $onClick = sprintf(
-                'onClick="openModalForButton(this, \'%s\', \'%s\')"',
+                'onClick="openModalForButton%s(this, \'%s\', \'%s\')"',
+                $seed,
                 h($data['popover_url']),
                 h(!empty($data['reload_url']) ? $data['reload_url'] : '')
             );
@@ -70,7 +72,7 @@
 ?>
 
 <script>
-    function openModalForButton(clicked, url, reloadUrl='') {
+    function openModalForButton<?= $seed ?>(clicked, url, reloadUrl='') {
         const fallbackReloadUrl = '<?= $this->Url->build(['action' => 'index']); ?>'
         reloadUrl = reloadUrl != '' ? reloadUrl : fallbackReloadUrl
         UI.overlayUntilResolve(clicked, UI.submissionModalForIndex(url, reloadUrl, '<?= $tableRandomValue ?>'))