From 4e3e42549107806c2b6522eb1808a70eb7fa86c0 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andreas=20=C3=96sterreicher?= <oesi@technikum-wien.at>
Date: Wed, 9 Nov 2022 11:58:31 +0100
Subject: [PATCH] Mailversand bei Urlaubstool korrigiert

---
 cis/private/profile/urlaubstool.php   | 86 ++++++++++++++-------------
 config/cis.config-default.inc.php     |  6 +-
 config/vilesci.config-default.inc.php |  5 +-
 3 files changed, 52 insertions(+), 45 deletions(-)

diff --git a/cis/private/profile/urlaubstool.php b/cis/private/profile/urlaubstool.php
index afd62559a..42bbe2ec7 100644
--- a/cis/private/profile/urlaubstool.php
+++ b/cis/private/profile/urlaubstool.php
@@ -170,19 +170,22 @@ if (isset($_GET['rechts_x']) || isset($_POST['rechts_x']))
 
 //Bereits freigegebenen Eintrag löschen
 //Eintragung löschen
-if((isset($_GET['delete'])  && isset($_GET['informSupervisor'])) || (isset($_POST['delete']) && isset($_POST['informSupervisor'])))
+if(isset($_GET['delete']))
 {
     $zeitsperre = new zeitsperre();
     $zeitsperre->load($_GET['delete']);
 
     $vondatum = $zeitsperre->getVonDatum();
     $bisdatum = $zeitsperre->getBisDatum();
-		$vondatum = $datum_obj->formatDatum($vondatum ,'d.m.Y');
-		$bisdatum = $datum_obj->formatDatum($bisdatum,'d.m.Y');
+	$vondatum = $datum_obj->formatDatum($vondatum ,'d.m.Y');
+	$bisdatum = $datum_obj->formatDatum($bisdatum,'d.m.Y');
 
     if(!$zeitsperre->delete($_GET['delete']))
+	{
         echo $zeitsperre->errormsg;
-
+	}
+	else
+	{
         //Mail an Vorgesetzten
         $prsn = new person();
 
@@ -207,48 +210,36 @@ if((isset($_GET['delete'])  && isset($_GET['informSupervisor'])) || (isset($_POS
                 }
             }
 
-        $benutzer = new benutzer();
-        $benutzer->load($uid);
+	        $benutzer = new benutzer();
+	        $benutzer->load($uid);
 
-				//new sanchomail
-				$nameMitarbeiter =  $benutzer->vorname. " ". $benutzer->nachname;
-				$beschreibung = $zeitsperre->bezeichnung;
-				$subject = "Urlaub wurde gelöscht";
-				$mailvorlage = 'Sancho_Mail_Urlaub_Loeschen';
+			//new sanchomail
+			$nameMitarbeiter =  $benutzer->vorname. " ". $benutzer->nachname;
+			$beschreibung = $zeitsperre->bezeichnung;
+			$subject = "Urlaub wurde gelöscht";
+			$mailvorlage = 'Sancho_Mail_Urlaub_Loeschen';
 
-				$from='vilesci@'.DOMAIN;
+			$from='vilesci@'.DOMAIN;
 
-				//Sanchomail mit Vorlage Sancho Mail Urlaub
-				$template_data = array(
-					'vorgesetzter' => $fullName,
-					'nameMitarbeiter' => $nameMitarbeiter,
-					'beschreibung' =>$beschreibung,
-					'vonDatum' => $vondatum,
-					'bisDatum' => $bisdatum
-				);
+			//Sanchomail mit Vorlage Sancho Mail Urlaub
+			$template_data = array(
+				'vorgesetzter' => $fullName,
+				'nameMitarbeiter' => $nameMitarbeiter,
+				'beschreibung' =>$beschreibung,
+				'vonDatum' => $vondatum,
+				'bisDatum' => $bisdatum
+			);
 
-				if (sendSanchoMail($mailvorlage, $template_data, $to, $subject))
-        {
-            $vgmail="<span style='color:green;'>".$p->t('urlaubstool/VorgesetzteInformiert',array($fullName))."</span>";
-        }
-        else
-        {
-            $vgmail="<br><span class='error'>".$p->t('urlaubstool/fehlerBeimSendenAufgetreten',array($fullName))."!</span>";
-        }
+			if (sendSanchoMail($mailvorlage, $template_data, $to, $subject))
+	        {
+	            $vgmail="<span style='color:green;'>".$p->t('urlaubstool/VorgesetzteInformiert',array($fullName))."</span>";
+	        }
+	        else
+	        {
+	            $vgmail="<br><span class='error'>".$p->t('urlaubstool/fehlerBeimSendenAufgetreten',array($fullName))."!</span>";
+	        }
+		}
     }
-    else
-    {
-        $vgmail="<br><span class='error'>".$p->t('urlaubstool/konnteKeinFreigabemailVersendetWerden')."</span>";
-    }
-}
-
-
-//Eintragung löschen
-if((isset($_GET['delete']) || isset($_POST['delete'])))
-{
-	$zeitsperre = new zeitsperre();
-	if(!$zeitsperre->delete($_GET['delete']))
-		echo $zeitsperre->errormsg;
 }
 
 //Eintragung speichern
@@ -594,6 +585,17 @@ $datum_obj = new datum();
 		<script type="text/javascript" src="../../../vendor/components/jqueryui/jquery-ui.min.js"></script>
 		<script type="text/javascript" src="../../../include/js/jquery.ui.datepicker.translation.js"></script>
 <?php
+// ADDONS laden
+$addon_obj = new addon();
+$addon_obj->loadAddons();
+foreach($addon_obj->result as $addon)
+{
+       if(file_exists('../../../addons/'.$addon->kurzbz.'/cis/init.js.php'))
+       {
+               echo '
+               <script type="application/x-javascript" src="../../../addons/'.$addon->kurzbz.'/cis/init.js.php" ></script>';
+       }
+}
 
 // Wenn Seite fertig geladen ist Addons aufrufen
 echo '
@@ -900,7 +902,7 @@ for ($i=0;$i<6;$i++)
 					if($hgfarbe[$j+7*$i]=='#CDDDEE')
 					{
 						$k=$j+7*$i;
-						echo "<a href='$PHP_SELF?wmonat=$wmonat&wjahr=$wjahr&delete=$datensatz[$k]&informSupervisor=True' onclick='return conf_del()'>";
+						echo "<a href='$PHP_SELF?wmonat=$wmonat&wjahr=$wjahr&delete=$datensatz[$k]' onclick='return conf_del()'>";
                         echo '<img src="../../../skin/images/delete_x.png" alt="loeschen" title="'.$p->t('urlaubstool/eintragungLoeschen').'"></a></td>';
 					}
 				}
diff --git a/config/cis.config-default.inc.php b/config/cis.config-default.inc.php
index da3baf507..027af9504 100644
--- a/config/cis.config-default.inc.php
+++ b/config/cis.config-default.inc.php
@@ -261,5 +261,9 @@ define('CIS_ZEITWUNSCH_GD', false);
 define('CIS_SHOW_COVID_STATUS', false);
 
 //Vertrag Allin
-define ('DEFAULT_ALLIN_DIENSTVERTRAG',[110,111]);
+define ('DEFAULT_ALLIN_DIENSTVERTRAG',[111]);
+
+//Echter Dienstvertrag
+define ('DEFAULT_ECHTER_DIENSTVERTRAG',[103,111]);
+
 ?>
diff --git a/config/vilesci.config-default.inc.php b/config/vilesci.config-default.inc.php
index 6ed926b3f..17f668b90 100644
--- a/config/vilesci.config-default.inc.php
+++ b/config/vilesci.config-default.inc.php
@@ -266,8 +266,9 @@ define('FAS_DOPPELTE_BUCHUNGSTYPEN_CHECK', serialize(
 // Spezialnoten die am Zeunigs und Diplomasupplement ignoriert werden
 define('ZEUGNISNOTE_NICHT_ANZEIGEN',serialize(array('iar', 'nz')));
 
+//Default Lehrmodus
+define ('DEFAULT_LEHRMODUS','regulaer');
+
 //Echter Dienstvertrag
 define ('DEFAULT_ECHTER_DIENSTVERTRAG',[103,110]);
-
-
 ?>