split display and booking action into two separate actions, fixes #3771

Closes #3771

Merge request studip/studip!3180

1 files changed, 16 insertions, 13 deletions

diff --git a/app/controllers/consultation/admin.php b/app/controllers/consultation/admin.php
index a1f16c5..1a83124 100644
--- a/app/controllers/consultation/admin.php
+++ b/app/controllers/consultation/admin.php
@@ -394,24 +394,27 @@ class Consultation_AdminController extends ConsultationController
                 'sem_perm'   => $permissions,
             ]);
         }
+    }
 
-        if (Request::isPost()) {
-            CSRFProtection::verifyUnsafeRequest();
+    public function store_booking_action($block_id, $slot_id, $page = 0): void
+    {
+        CSRFProtection::verifyUnsafeRequest();
 
-            if ($this->slot->isOccupied()) {
-                PageLayout::postError(_('Dieser Termin ist bereits belegt.'));
-            } else {
-                $booking = new ConsultationBooking();
-                $booking->slot_id = $this->slot->id;
-                $booking->user_id = Request::option('user_id');
-                $booking->reason  = trim(Request::get('reason'));
-                $booking->store();
+        $slot = $this->loadSlot($block_id, $slot_id);
 
-                PageLayout::postSuccess(_('Der Termin wurde reserviert.'));
-            }
+        if ($slot->isOccupied()) {
+            PageLayout::postError(_('Dieser Termin ist bereits belegt.'));
+        } else {
+            $booking = new ConsultationBooking();
+            $booking->slot_id = $slot->id;
+            $booking->user_id = Request::option('user_id');
+            $booking->reason  = trim(Request::get('reason'));
+            $booking->store();
 
-            $this->redirect("consultation/admin/index/{$page}#slot-{$this->slot->id}");
+            PageLayout::postSuccess(_('Der Termin wurde reserviert.'));
         }
+
+        $this->redirect("consultation/admin/index/{$page}#slot-{$slot->id}");
     }
 
     public function edit_action($block_id, $page = 0)