From 4ced199e88a2ef1c2412b7a18baf9ee605e9b611 Mon Sep 17 00:00:00 2001
From: OliiverHacker <hacker@technikum-wien.at>
Date: Tue, 13 Oct 2020 12:26:06 +0200
Subject: [PATCH] =?UTF-8?q?Freigegebenen=20Urlaub=20l=C3=B6schen=20auch=20?=
 =?UTF-8?q?in=20Zeitsperren?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 cis/private/profile/urlaubstool.php           |  5 +-
 cis/private/profile/zeitsperre_resturlaub.php | 64 ++++++++++++++++++-
 2 files changed, 62 insertions(+), 7 deletions(-)

diff --git a/cis/private/profile/urlaubstool.php b/cis/private/profile/urlaubstool.php
index 720935c46..efba62e99 100644
--- a/cis/private/profile/urlaubstool.php
+++ b/cis/private/profile/urlaubstool.php
@@ -198,11 +198,8 @@ if((isset($_GET['delete'])  && isset($_GET['informSupervisor'])) || (isset($_POS
         $benutzer->load($uid);
         $message = $p->t('urlaubstool/diesIstEineAutomatischeMail')."\n".
             $p->t('urlaubstool/xHatUrlaubGeloescht',array($benutzer->nachname,$benutzer->vorname)).":\n";
+        $message.= $p->t('urlaubstool/von')." ".date("d.m.Y", strtotime($vondatum))." ".$p->t('urlaubstool/bis')." ".date("d.m.Y", strtotime($bisdatum))."\n";
 
-        for($i=0;$i<count($akette);$i++)
-        {
-            $message.= $p->t('urlaubstool/von')." ".date("d.m.Y", strtotime($vondatum))." ".$p->t('urlaubstool/bis')." ".date("d.m.Y", strtotime($bisdatum))."\n";
-        }
 
         $mail = new mail($to, 'vilesci@'.DOMAIN,$p->t('urlaubstool/freigegebenerUrlaubGeloescht'), $message);
         if($mail->send())
diff --git a/cis/private/profile/zeitsperre_resturlaub.php b/cis/private/profile/zeitsperre_resturlaub.php
index b7654310e..348a4418e 100644
--- a/cis/private/profile/zeitsperre_resturlaub.php
+++ b/cis/private/profile/zeitsperre_resturlaub.php
@@ -461,8 +461,62 @@ if(isset($_GET['type']) && ($_GET['type']=='edit_sperre' || $_GET['type']=='new_
 		echo "<span class='error'>$error_msg</span>";
 }
 
+//loeschen eines bereits freigegebenen Urlaubs
+if((isset($_GET['type']) && $_GET['type']=='delete_sperre' && isset($_GET['informSupervisor'])))
+{
+    $zeitsperre = new zeitsperre();
+    $zeitsperre->load($_GET['id']);
+
+    $vondatum = $zeitsperre->getVonDatum();
+    $bisdatum = $zeitsperre->getBisDatum();
+
+    if(!$zeitsperre->delete($_GET['id']))
+        echo $zeitsperre->errormsg;
+
+    //Mail an Vorgesetzten
+    $vorgesetzter = $ma->getVorgesetzte($uid);
+    if($vorgesetzter)
+    {
+        $to='';
+        foreach($ma->vorgesetzte as $vg)
+        {
+            if($to!='')
+            {
+                $to.=', '.$vg.'@'.DOMAIN;
+            }
+            else
+            {
+                $to.=$vg.'@'.DOMAIN;
+            }
+        }
+
+        $benutzer = new benutzer();
+        $benutzer->load($uid);
+        $message = $p->t('urlaubstool/diesIstEineAutomatischeMail')."\n".
+            $p->t('urlaubstool/xHatUrlaubGeloescht',array($benutzer->nachname,$benutzer->vorname)).":\n";
+
+
+        $message.= $p->t('urlaubstool/von')." ".date("d.m.Y", strtotime($vondatum))." ".$p->t('urlaubstool/bis')." ".date("d.m.Y", strtotime($bisdatum))."\n";
+
+
+        $mail = new mail($to, 'vilesci@'.DOMAIN,$p->t('urlaubstool/freigegebenerUrlaubGeloescht'), $message);
+        if($mail->send())
+        {
+            echo "<br><b>".$p->t('urlaubstool/VorgesetzteInformiert',array($to))."</b>";
+        }
+        else
+        {
+            echo "<br><span class='error'>".$p->t('urlaubstool/fehlerBeimSendenAufgetreten',array($to))."!</span>";
+        }
+    }
+    else
+    {
+        $vgmail="<br><span class='error'>".$p->t('urlaubstool/konnteKeinFreigabemailVersendetWerden')."</span>";
+    }
+}
+
 //loeschen einer zeitsperre
-if(isset($_GET['type']) && $_GET['type']=='delete_sperre')
+if(isset($_GET['type']) && $_GET['type']=='delete_sperre' && !isset($_GET['informSupervisor'])   )
 {
 	$zeit = new zeitsperre();
 	$zeit->load($_GET['id']);
@@ -532,10 +586,14 @@ if(count($zeit->result)>0)
 			$content_table.="<td><a href='$PHP_SELF?type=edit&id=$row->zeitsperre_id' class='Item'>".$p->t('zeitsperre/edit')."</a></td>";
 		if ($row->vondatum < $gesperrt_bis AND in_array($row->zeitsperretyp_kurzbz,$typen_arr))
 			$content_table .= '<td>&nbsp;</td>';
-		else if($row->freigabeamum=='' || $row->zeitsperretyp_kurzbz!='Urlaub')
+		else if($row->vondatum>=date("Y-m-d",time()) && $row->zeitsperretyp_kurzbz=='Urlaub')
 		{
-			$content_table.="\n<td><a href='$PHP_SELF?type=delete_sperre&id=$row->zeitsperre_id' onclick='return conf_del()' class='Item'>".$p->t('zeitsperre/loeschen')."</a></td>";
+			$content_table.="\n<td><a href='$PHP_SELF?type=delete_sperre&id=$row->zeitsperre_id&informSupervisor=True' onclick='return conf_del()' class='Item'>".$p->t('zeitsperre/loeschen')."</a></td>";
 		}
+		elseif($row->zeitsperretyp_kurzbz!='Urlaub')
+        {
+            $content_table.="\n<td><a href='$PHP_SELF?type=delete_sperre&id=$row->zeitsperre_id' onclick='return conf_del()' class='Item'>".$p->t('zeitsperre/loeschen')."</a></td>";
+        }
 		else
 			$content_table .= '<td>&nbsp;</td>';
 		$content_table.="</tr>";