From cc60c7073baaaaee9923532318aadfdd0808de4f Mon Sep 17 00:00:00 2001
From: David Siegfried <david.siegfried@uni-vechta.de>
Date: Thu, 4 Apr 2024 13:24:27 +0000
Subject: [PATCH] dont store empty room-request-properties, closes #3947

Closes #3947

Merge request studip/studip!2804
---
 app/controllers/course/room_requests.php | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/app/controllers/course/room_requests.php b/app/controllers/course/room_requests.php
index ab802b73b69..14c0179b08f 100644
--- a/app/controllers/course/room_requests.php
+++ b/app/controllers/course/room_requests.php
@@ -630,7 +630,9 @@ class Course_RoomRequestsController extends AuthenticatedController
 
             //Store the properties:
             foreach ($_SESSION[$request_id]['selected_properties'] as $name => $state) {
-                $this->request->setProperty($name, $state);
+                if (!empty($state)) {
+                    $this->request->setProperty($name, $state);
+                }
             }
 
             // once stored, we can delete the session data for this request
-- 
GitLab