From 772a54e895548de49427d21492cbab675ff11633 Mon Sep 17 00:00:00 2001
From: root <root@mail.dropamsee.de>
Date: Tue, 27 May 2025 17:28:25 +0200
Subject: [PATCH] smaller bugfixes in dev

---
 html/Admin.php              |  2 +-
 html/AdminAlleSchichten.php |  4 ++--
 html/AdminDienste.php       |  2 +-
 html/AdminUserdaten.php     | 13 +++++++------
 html/AlleSchichten.php      |  9 +++++++--
 html/_login.php             |  2 +-
 6 files changed, 19 insertions(+), 13 deletions(-)

diff --git a/html/Admin.php b/html/Admin.php
index 03e2663..120c718 100644
--- a/html/Admin.php
+++ b/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>";
 }
 
diff --git a/html/AdminAlleSchichten.php b/html/AdminAlleSchichten.php
index d5b6acc..86b8825 100644
--- a/html/AdminAlleSchichten.php
+++ b/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
diff --git a/html/AdminDienste.php b/html/AdminDienste.php
index c76fc98..68cb1e7 100644
--- a/html/AdminDienste.php
+++ b/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>";
 }
 
diff --git a/html/AdminUserdaten.php b/html/AdminUserdaten.php
index 872ecb4..5059360 100644
--- a/html/AdminUserdaten.php
+++ b/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>';
diff --git a/html/AlleSchichten.php b/html/AlleSchichten.php
index 6486a03..bbce021 100644
--- a/html/AlleSchichten.php
+++ b/html/AlleSchichten.php
@@ -14,7 +14,7 @@ require '_login.php';
   <link rel="stylesheet" href="css/style_desktop.css" media="screen and (min-width:781px)"/>
   <link rel="stylesheet" href="css/style_mobile.css" media="screen and (max-width:780px)"/>
   <meta name="viewport" content="width=480" />
-  <script src="<?php echo JQUERY ?>" type="text/javascript"></script> 
+  <script src="<?php echo JQUERY ?>" type="text/javascript"></script>
   <script src="js/helferdb.js" type="text/javascript"></script>
   <script> collapse_table_rows();
  </script>
@@ -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
diff --git a/html/_login.php b/html/_login.php
index c449ecc..8535205 100644
--- a/html/_login.php
+++ b/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"];