#51 fix problems with changing needed angeltypes in rooms

includes/pages/admin_rooms.php

@@ -67,20 +67,22 @@ function admin_rooms() {
         else
           $ok = false;
 
-        foreach ($angeltypes as $angeltype_id => $angeltype)
-          if (isset ($_REQUEST['angeltype_count_' . $angeltype_id]) && preg_match("/^[0-9]{1,11}$/", $_REQUEST['angeltype_count_' . $angeltype_id]))
-          $angeltypes_count[$angeltype_id] = $_REQUEST['angeltype_count_' . $angeltype_id];
-        else {
-          $ok = false;
-          $msg .= error(sprintf("Please enter needed angels for type %s.", $angeltype), true);
+        foreach ($angeltypes as $angeltype_id => $angeltype) {
+          if (isset ($_REQUEST['angeltype_count_' . $angeltype_id]) && preg_match("/^[0-9]{1,4}$/", $_REQUEST['angeltype_count_' . $angeltype_id]))
+            $angeltypes_count[$angeltype_id] = $_REQUEST['angeltype_count_' . $angeltype_id];
+          else {
+            $ok = false;
+            $msg .= error(sprintf("Please enter needed angels for type %s.", $angeltype), true);
+          }
         }
 
         if ($ok) {
           if(isset($id))
             sql_query("UPDATE `Room` SET `Name`='" . sql_escape($name) . "', `FromPentabarf`='" . sql_escape($from_pentabarf) . "', `show`='" . sql_escape($public) . "', `Number`='" . sql_escape($number) . "' WHERE `RID`=" . sql_escape($id) . " LIMIT 1");
-          else
+          else {
             sql_query("INSERT INTO `Room` SET `Name`='" . sql_escape($name) . "', `FromPentabarf`='" . sql_escape($from_pentabarf) . "', `show`='" . sql_escape($public) . "', `Number`='" . sql_escape($number) . "'");
-          $id = sql_id();
+            $id = sql_id();
+          }
 
           sql_query("DELETE FROM `NeededAngelTypes` WHERE `room_id`=" . sql_escape($id));
           foreach ($angeltypes_count as $angeltype_id => $angeltype_count)

includes/pages/user_shifts.php

@@ -24,9 +24,9 @@ function user_shifts() {
       redirect(page_link_to('user_shifts'));
 
     /*
-    if (sql_num_query("SELECT * FROM `ShiftEntry` WHERE `SID`=" . sql_escape($shift_id) . " LIMIT 1") > 0) {
-      error("Du kannst nur Schichten bearbeiten, bei denen niemand eingetragen ist.");
-      redirect(page_link_to('user_shift'));
+     if (sql_num_query("SELECT * FROM `ShiftEntry` WHERE `SID`=" . sql_escape($shift_id) . " LIMIT 1") > 0) {
+    error("Du kannst nur Schichten bearbeiten, bei denen niemand eingetragen ist.");
+    redirect(page_link_to('user_shift'));
     }
     */
 
@@ -49,13 +49,17 @@ function user_shifts() {
 
     // Benötigte Engeltypen vom Raum
     $needed_angel_types_source = sql_select("SELECT `AngelTypes`.*, `NeededAngelTypes`.`count` FROM `AngelTypes` LEFT JOIN `NeededAngelTypes` ON (`NeededAngelTypes`.`angel_type_id` = `AngelTypes`.`id` AND `NeededAngelTypes`.`room_id`=" . sql_escape($shift['RID']) . ") ORDER BY `AngelTypes`.`name`");
-    foreach ($needed_angel_types_source as $type)
-      $needed_angel_types[$type['id']] = $type['count'] != "" ? $type['count'] : "0";
+    foreach ($needed_angel_types_source as $type) {
+      if($type['count'] != "")
+        $needed_angel_types[$type['id']] =$type['count'];
+    }
 
     // Benötigte Engeltypen von der Schicht
     $needed_angel_types_source = sql_select("SELECT `AngelTypes`.*, `NeededAngelTypes`.`count` FROM `AngelTypes` LEFT JOIN `NeededAngelTypes` ON (`NeededAngelTypes`.`angel_type_id` = `AngelTypes`.`id` AND `NeededAngelTypes`.`shift_id`=" . sql_escape($shift_id) . ") ORDER BY `AngelTypes`.`name`");
-    foreach ($needed_angel_types_source as $type)
-      $needed_angel_types[$type['id']] = $type['count'] != "" ? $type['count'] : "0";
+    foreach ($needed_angel_types_source as $type){
+      if($type['count'] != "")
+        $needed_angel_types[$type['id']] =$type['count'];
+    }
 
     $name = $shift['name'];
     $rid = $shift['RID'];