aboutsummaryrefslogtreecommitdiff
path: root/app
diff options
context:
space:
mode:
authorFinn Schneider <schneider@data-quest.de>2025-11-10 13:43:38 +0000
committerElmar Ludwig <elmar.ludwig@uni-osnabrueck.de>2025-11-10 14:43:38 +0100
commitd086a4c7c38c86135bc207ba206a30f6de97f261 (patch)
treee1427ee0e5eea9b0132196d008e7bf7c24c434df /app
parent5081d774b076867eada91bad5b60c88a8481da1c (diff)
Resolve "Wiki: Löschen in Tabellenansicht löscht die falsche Seite"
Closes #6030 Merge request studip/studip!4598
Diffstat (limited to 'app')
-rw-r--r--app/controllers/course/wiki.php4
-rw-r--r--app/views/course/wiki/allpages.php13
2 files changed, 10 insertions, 7 deletions
diff --git a/app/controllers/course/wiki.php b/app/controllers/course/wiki.php
index c9a4fa2..e479868 100644
--- a/app/controllers/course/wiki.php
+++ b/app/controllers/course/wiki.php
@@ -186,7 +186,9 @@ class Course_WikiController extends AuthenticatedController
'delete',
_('Seite löschen'),
Icon::create('trash'),
- ['data-confirm' => _('Wollen Sie wirklich die komplette Seite löschen?'), 'form' => 'delete_page']
+ ['data-confirm' => _('Wollen Sie wirklich die komplette Seite löschen?'),
+ 'formaction' => $this->deleteURL($page),
+ 'form' => 'delete_page']
);
}
}
diff --git a/app/views/course/wiki/allpages.php b/app/views/course/wiki/allpages.php
index bbabb67..0797e8c 100644
--- a/app/views/course/wiki/allpages.php
+++ b/app/views/course/wiki/allpages.php
@@ -30,11 +30,6 @@
</thead>
<tbody>
<? foreach ($pages as $page) : ?>
- <? if ($page->isEditable()) : ?>
- <form action="<?= $controller->delete($page) ?>" method="post" id="delete_page">
- <?= CSRFProtection::tokenTag() ?>
- </form>
- <? endif ?>
<tr>
<td>
<input
@@ -62,7 +57,9 @@
<? endif; ?>
</td>
<td class="actions">
- <?= $controller->getActionMenu($page, 'allpages') ?>
+ <? if ($page->isEditable()) : ?>
+ <?= $controller->getActionMenu($page, 'allpages') ?>
+ <? endif ?>
</td>
</tr>
<? endforeach ?>
@@ -80,3 +77,7 @@
</tfoot>
</table>
</form>
+
+<form action="" method="post" id="delete_page">
+ <?= CSRFProtection::tokenTag() ?>
+</form>