diff --git a/app/controllers/course/wiki.php b/app/controllers/course/wiki.php index 6bfa0ce6f0c5bb968ec62fc51be4f7d5692f5ce1..cf87b081a80dfe6d0b43b6eaf8674de969d2f354 100644 --- a/app/controllers/course/wiki.php +++ b/app/controllers/course/wiki.php @@ -308,29 +308,24 @@ class Course_WikiController extends AuthenticatedController $this->redirect($this->allpagesURL()); } - public function deleteversion_action(WikiPage $page, $version_id = null) + public function deleteversion_action(WikiPage $page) { if (!Request::isPost() || !$page->isEditable() || !CSRFProtection::verifyRequest()) { throw new AccessDeniedException(); } - if ($version_id === null) { - $version = $page->versions[0]; - if ($version) { - $page['name'] = $version['name']; - $page['content'] = $version['content']; - $page['user_id'] = $version['user_id']; - $page['chdate'] = $version['mkdate']; - $page->store(); - $version->delete(); - } else { - $page->delete(); - } + + $version = $page->versions[0]; + if ($version) { + $page['name'] = $version['name']; + $page['content'] = $version['content']; + $page['user_id'] = $version['user_id']; + $page['chdate'] = $version['mkdate']; + $page->store(); + $version->delete(); } else { - $version = WikiVersion::find($version_id); - if ($version['page_id'] === $page->id) { - $version->delete(); - } + $page->delete(); } + PageLayout::postSuccess(_('Version wurde gelöscht.')); if (Request::get('redirect_to') === 'page') { $this->redirect($this->page($page)); diff --git a/app/views/course/wiki/history.php b/app/views/course/wiki/history.php index cbd324d981710c2c3b97d861fd5571049eaadc20..81fde594c85fa335d9faba747400dcc1333f250a 100644 --- a/app/views/course/wiki/history.php +++ b/app/views/course/wiki/history.php @@ -45,16 +45,6 @@ <a href="<?= $controller->versiondiff($page) ?>" data-dialog> <?= Icon::create('log')->asImg(['class' => 'text-bottom']) ?> </a> - <? if ($page->isEditable()) : ?> - <form action="<?= $controller->deleteversion($page) ?>" - method="post" - class="inline" - title="<?= _('Version löschen') ?>" - data-confirm="<?= _('Wirklich diese Version löschen?') ?>"> - <?= CSRFProtection::tokenTag() ?> - <?= Icon::create('trash')->asInput() ?> - </form> - <? endif ?> </td> </tr> <? foreach ($page->versions as $i => $version) : ?> @@ -79,16 +69,6 @@ <a href="<?= $controller->versiondiff($page, $version->id) ?>" data-dialog> <?= Icon::create('log')->asImg(['class' => 'text-bottom']) ?> </a> - <? if ($page->isEditable()) : ?> - <form action="<?= $controller->deleteversion($page, $version->id) ?>" - method="post" - class="inline" - title="<?= _('Version löschen') ?>" - data-confirm="<?= _('Wirklich diese Version löschen?') ?>"> - <?= CSRFProtection::tokenTag() ?> - <?= Icon::create('trash')->asInput() ?> - </form> - <? endif ?> </td> </tr> <? endforeach ?>