smaller bugfixes in dev

html/Admin.php

@@ -100,7 +100,7 @@ while ($zeile = mysqli_fetch_array($db_erg, MYSQLI_ASSOC)) {
         $selectedSet = true;
     }
 }
-if( !$selectedSet ) {
+if( ! isset($selectedSet) and ! $selectedSet) {
   echo "<option value='none' selected='selected'>Bitte auswählen</optionen>";
 }
 

html/AdminAlleSchichten.php

@@ -206,8 +206,8 @@ if (isset($_POST['minusschicht'])) {
     echo '</th></tr></table>';
 /// Schichten Auswahl
 ////////////////////////////////////////////////////////
-$addschicht = $_SESSION["addschicht"];
-$dienstsort = $_SESSION["dienstsort"];
+$addschicht = $_SESSION["addschicht"] ?? null;
+$dienstsort = $_SESSION["dienstsort"] ?? null;
 
 
 //addschicht und dienst-sort sollten wohl nach Diensten bzw Tagen sortieren

html/AdminDienste.php

@@ -185,7 +185,7 @@ while ($zeile = mysqli_fetch_array($db_erg, MYSQLI_ASSOC)) {
         $HelferLevel = $zeile['HelferLevel'];
     }
 }
-if( ! $selectedset ) { 
+if( ! isset($selectedset) or ! $selectedset ) {
 echo "<option value='none' selected='selected'>Bitte auswählen</option>";
 }
 

html/AdminUserdaten.php

@@ -24,7 +24,7 @@ function HelferAuswahlButton($db_link, $AliasHelferID)
         $selectedSet = true;
         }
     }
-    if( !$selectedSet ) {
+    if( ! isset($selectedSet) or ! $selectedSet ) {
       echo "<option value='none' selected='selected'>Bitte auswählen</optionen>";
     }
 
@@ -83,15 +83,16 @@ if (isset($_POST['change'])) {
     $HelferHandy = $_POST['helfer-handy'];
     $HelferLevel = $_POST['helfer-level'];
     $HelferNewPasswort  = $_POST['helfer-newpasswort'];
-    if ($_POST['IsAdmin']) {
-        $HelferIsAdmin = 1;
-        //echo "is Admin<br>";
+
+    if (isset($_POST['IsAdmin']) and $_POST['IsAdmin']) {
+        $HelferBecomesAdmin = 1;
     } else {
-        $HelferIsAdmin = 0;
+        $HelferBecomesAdmin = 0;
     }
+
     if (empty($messages)) {
         // Helferdaten Ändern
-        HelferdatenAendern($db_link, $HelferName, $HelferEmail, $HelferHandy, $HelferNewPasswort, $AliasHelferID, $HelferLevel, $HelferIsAdmin, $HelferID);
+        HelferdatenAendern($db_link, $HelferName, $HelferEmail, $HelferHandy, $HelferNewPasswort, $AliasHelferID, $HelferLevel, $HelferBecomesAdmin, $HelferID);
     } else {
         // Fehlermeldungen ausgeben:
         echo '<div class="error"><ul>';

html/AlleSchichten.php

@@ -42,7 +42,11 @@ if (isset($_POST['CloseInfo'])) {
     //    $InfoMeineSchichtID = $_POST['InfoMeineSchichtID'];
 
         $zeile = DetailSchicht($db_link, $SchichtID);
-
+    if(!isset($zeile['Was'])){
+    error_log("Zeile not set in Schichtinfo");
+    error_log("called with: SchichtID $SchichtID $Was, $Wo, $Dauer, $Leiter, $LeiterHandy etc");
+    // Das ist vermutlich kein Fehler mehr, wenn wir den selben Account mehrfach auf die selbe Schicht lassen für Familien etc
+    }
     $Was = $zeile['Was'];
     $Wo = $zeile['Wo'];
     $Dauer = $zeile['Dauer'];
@@ -51,6 +55,7 @@ if (isset($_POST['CloseInfo'])) {
     $LeiterEmail =  $zeile['Email'];
     $Info = $zeile['Info'];
     $db_link->close();
+    return;
 }
 
 // wird nur mit anderer Datei DetailsSchichten.php verwendet, nicht hier

html/_login.php

@@ -101,7 +101,7 @@ if (!isset($_SESSION["HelferID"])) {
 $HelferID = $_SESSION["HelferID"];
 $HelferName = $_SESSION["HelferName"];
 $HelferEmail = $_SESSION["HelferEmail"];
-$AdminID = $_SESSION["AdminID"];
+$AdminID = isset($_SESSION["AdminID"]) ? $_SESSION["AdminID"]  : -1;
 //TODO vereinheitlichen. index.php verwendet HelferIsAdmin
 $HelferIsAdmin = $AdminStatus = $_SESSION["AdminStatus"];
 $HelferLevel = $_SESSION["HelferLevel"];