improve shift rendering

3 files changed, 9 insertions, 7 deletions

diff --git a/includes/model/NeededAngelTypes_model.php b/includes/model/NeededAngelTypes_model.php
index 77a23c3d..47c9626f 100644
--- a/includes/model/NeededAngelTypes_model.php
+++ b/includes/model/NeededAngelTypes_model.php
@@ -65,7 +65,7 @@ function NeededAngelTypes_by_shift($shiftId) {
         ORDER BY `room_id` DESC
         ");
   if ($needed_angeltypes_source === false) {
-    return false;
+    engelsystem_error("Unable to load needed angeltypes.");
   }
   
   // Use settings from room
@@ -80,18 +80,16 @@ function NeededAngelTypes_by_shift($shiftId) {
         ORDER BY `room_id` DESC
         ");
     if ($needed_angeltypes_source === false) {
-      return false;
+      engelsystem_error("Unable to load needed angeltypes.");
     }
   }
   
   $needed_angeltypes = [];
   foreach ($needed_angeltypes_source as $angeltype) {
     $shift_entries = ShiftEntries_by_shift_and_angeltype($shiftId, $angeltype['angel_type_id']);
-    if ($shift_entries === false) {
-      return false;
-    }
     
     $angeltype['taken'] = count($shift_entries);
+    $angeltype['shift_entries'] = $shift_entries;
     $needed_angeltypes[] = $angeltype;
   }
   
diff --git a/includes/model/ShiftEntry_model.php b/includes/model/ShiftEntry_model.php
index 425b92e8..63127bc7 100644
--- a/includes/model/ShiftEntry_model.php
+++ b/includes/model/ShiftEntry_model.php
@@ -110,12 +110,16 @@ function ShiftEntries_finished_by_user($user) {
  * @param int $angeltype_id          
  */
 function ShiftEntries_by_shift_and_angeltype($shift_id, $angeltype_id) {
-  return sql_select("
+  $result = sql_select("
       SELECT *
       FROM `ShiftEntry`
       WHERE `SID`=" . sql_escape($shift_id) . "
       AND `TID`=" . sql_escape($angeltype_id) . "
       ");
+  if ($result === false) {
+    engelsystem_error("Unable to load shift entries.");
+  }
+  return $result;
 }
 
 /**
diff --git a/includes/model/Shifts_model.php b/includes/model/Shifts_model.php
index 5a38abdd..f232360e 100644
--- a/includes/model/Shifts_model.php
+++ b/includes/model/Shifts_model.php
@@ -2,7 +2,7 @@
 use Engelsystem\ShiftsFilter;
 
 function Shifts_by_room($room) {
-  $result = sql_select("SELECT * FROM `Shifts` WHERE `RID`=" . sql_escape($room['RID']));
+  $result = sql_select("SELECT * FROM `Shifts` WHERE `RID`=" . sql_escape($room['RID']) . " ORDER BY `start`");
   if ($result === false) {
     engelsystem_error("Unable to load shifts.");
   }