move room db queries to model

3 files changed, 14 insertions, 53 deletions

diff --git a/includes/pages/admin_import.php b/includes/pages/admin_import.php
index ccc62b9b..9883c0d5 100644
--- a/includes/pages/admin_import.php
+++ b/includes/pages/admin_import.php
@@ -253,11 +253,10 @@ function admin_import()
             list($rooms_new, $rooms_deleted) = prepare_rooms($import_file);
             foreach ($rooms_new as $room) {
                 $result = Room_create($room, true, null, null);
-
                 $rooms_import[trim($room)] = $result;
             }
             foreach ($rooms_deleted as $room) {
-                DB::delete('DELETE FROM `Room` WHERE `Name`=? LIMIT 1', [$room]);
+                Room_delete_by_name($room);
             }
 
             list($events_new, $events_updated, $events_deleted) = prepare_events(
@@ -378,7 +377,7 @@ function prepare_events($file, $shifttype_id, $add_minutes_start, $add_minutes_e
         ];
     }
 
-    $shifts = DB::select('SELECT * FROM `Shifts` WHERE `PSID` IS NOT NULL ORDER BY `start`');
+    $shifts = Shifts_from_frab();
     $shifts_db = [];
     foreach ($shifts as $shift) {
         $shifts_db[$shift['PSID']] = $shift;
diff --git a/includes/pages/admin_rooms.php b/includes/pages/admin_rooms.php
index c5b7b610..55b52cd9 100644
--- a/includes/pages/admin_rooms.php
+++ b/includes/pages/admin_rooms.php
@@ -1,7 +1,4 @@
 <?php
-
-use Engelsystem\Database\DB;
-
 /**
  * @return string
  */
@@ -15,7 +12,7 @@ function admin_rooms_title()
  */
 function admin_rooms()
 {
-    $rooms_source = DB::select('SELECT * FROM `Room` ORDER BY `Name`');
+    $rooms_source = Rooms();
     $rooms = [];
     $request = request();
 
@@ -40,7 +37,7 @@ function admin_rooms()
         $description = null;
         $room_id = 0;
 
-        $angeltypes_source = DB::select('SELECT `id`, `name` FROM `AngelTypes` ORDER BY `name`');
+        $angeltypes_source = AngelTypes();
         $angeltypes = [];
         $angeltypes_count = [];
         foreach ($angeltypes_source as $angeltype) {
@@ -60,10 +57,7 @@ function admin_rooms()
             $map_url = $room['map_url'];
             $description = $room['description'];
 
-            $needed_angeltypes = DB::select(
-                'SELECT `angel_type_id`, `count` FROM `NeededAngelTypes` WHERE `room_id`=?',
-                [$room_id]
-            );
+            $needed_angeltypes = NeededAngelTypes_by_room($room_id);
             foreach ($needed_angeltypes as $needed_angeltype) {
                 $angeltypes_count[$needed_angeltype['angel_type_id']] = $needed_angeltype['count'];
             }
@@ -74,16 +68,12 @@ function admin_rooms()
                 $valid = true;
 
                 if ($request->has('name') && strlen(strip_request_item('name')) > 0) {
-                    $name = strip_request_item('name');
-                    if (
-                        isset($room)
-                        && count(DB::select(
-                            'SELECT RID FROM `Room` WHERE `Name`=? AND NOT `RID`=?',
-                            [$name, $room_id]
-                        )) > 0
-                    ) {
+                    $result = Room_validate_name(strip_request_item('name'), $room_id);
+                    if(!$result->isValid()) {
                         $valid = false;
                         $msg .= error(_('This name is already in use.'), true);
+                    } else {
+                        $name = $result->getValue();
                     }
                 } else {
                     $valid = false;
@@ -116,38 +106,10 @@ function admin_rooms()
                 }
 
                 if ($valid) {
-                    if (!empty($room_id)) {
-                        DB::update('
-                            UPDATE `Room`
-                            SET
-                                `Name`=?,
-                                `from_frab`=?,
-                                `map_url`=?,
-                                `description`=?
-                            WHERE `RID`=?
-                            LIMIT 1
-                        ', [
-                            $name,
-                            (int) $from_frab,
-                            $map_url,
-                            $description,
-                            $room_id,
-                        ]);
-                        engelsystem_log(
-                            'Room updated: ' . $name
-                            . ', frab import: ' . ($from_frab ? 'Yes' : '')
-                            . ', map_url: ' . $map_url
-                            . ', description: ' . $description
-                        );
-                    } else {
+                    if (empty($room_id)) {
                         $room_id = Room_create($name, $from_frab, $map_url, $description);
-
-                        engelsystem_log(
-                            'Room created: ' . $name
-                            . ', frab import: ' . ($from_frab ? 'Yes' : '')
-                            . ', map_url: ' . $map_url
-                            . ', description: ' . $description
-                        );
+                    } else {
+                        Room_update($room_id, $name, $from_frab, $map_url, $description);
                     }
 
                     NeededAngelTypes_delete_by_room($room_id);
diff --git a/includes/pages/admin_shifts.php b/includes/pages/admin_shifts.php
index 721d0f4f..223ed074 100644
--- a/includes/pages/admin_shifts.php
+++ b/includes/pages/admin_shifts.php
@@ -29,8 +29,8 @@ function admin_shifts()
     $title = '';
     $shifttype_id = null;
 
-    // Locations laden (auch unsichtbare - fuer Erzengel ist das ok)
-    $rooms = DB::select('SELECT `RID`, `Name` FROM `Room` ORDER BY `Name`');
+    // Locations laden
+    $rooms = Rooms();
     $room_array = [];
     foreach ($rooms as $room) {
         $room_array[$room['RID']] = $room['Name'];