From 14fa1b24ad91826031ca70a0b8773aded43a7de5 Mon Sep 17 00:00:00 2001
From: OliiverHacker <hacker@technikum-wien.at>
Date: Wed, 14 Oct 2020 13:23:19 +0200
Subject: [PATCH] Display Full Name instead of Email Adress of Supervisor

---
 cis/private/profile/urlaubstool.php           | 41 +++++++++++--------
 cis/private/profile/zeitsperre_resturlaub.php | 19 ++++++---
 2 files changed, 37 insertions(+), 23 deletions(-)

diff --git a/cis/private/profile/urlaubstool.php b/cis/private/profile/urlaubstool.php
index 9fc8ec22d..11620a99a 100644
--- a/cis/private/profile/urlaubstool.php
+++ b/cis/private/profile/urlaubstool.php
@@ -177,22 +177,29 @@ if((isset($_GET['delete'])  && isset($_GET['informSupervisor'])) || (isset($_POS
     if(!$zeitsperre->delete($_GET['delete']))
         echo $zeitsperre->errormsg;
 
-    //Mail an Vorgesetzten
-    $vorgesetzter = $ma->getVorgesetzte($uid);
-    if($vorgesetzter)
-    {
-        $to='';
-        foreach($ma->vorgesetzte as $vg)
+        //Mail an Vorgesetzten
+        $prsn = new person();
+
+        $vorgesetzter = $ma->getVorgesetzte($uid);
+        if($vorgesetzter)
         {
-            if($to!='')
+            $to='';
+            $fullName ='';
+            foreach($ma->vorgesetzte as $vg)
             {
-                $to.=', '.$vg.'@'.DOMAIN;
+                if($to!='')
+                {
+                    $to.=', '.$vg.'@'.DOMAIN;
+                    $name = $prsn->getFullNameFromBenutzer($vg);
+                    $fullName.=', '.$name;
+                }
+                else
+                {
+                    $to.=$vg.'@'.DOMAIN;
+                    $name = $prsn->getFullNameFromBenutzer($vg);
+                    $fullName.=$name;
+                }
             }
-            else
-            {
-                $to.=$vg.'@'.DOMAIN;
-            }
-        }
 
         $benutzer = new benutzer();
         $benutzer->load($uid);
@@ -204,11 +211,11 @@ if((isset($_GET['delete'])  && isset($_GET['informSupervisor'])) || (isset($_POS
         $mail = new mail($to, 'vilesci@'.DOMAIN,$p->t('urlaubstool/freigegebenerUrlaubGeloescht'), $message);
         if($mail->send())
         {
-            $vgmail="<span style='color:green;'>".$p->t('urlaubstool/VorgesetzteInformiert',array($to))."</span>";
+            $vgmail="<span style='color:green;'>".$p->t('urlaubstool/VorgesetzteInformiert',array($fullName))."</span>";
         }
         else
         {
-            $vgmail="<br><span class='error'>".$p->t('urlaubstool/fehlerBeimSendenAufgetreten',array($to))."!</span>";
+            $vgmail="<br><span class='error'>".$p->t('urlaubstool/fehlerBeimSendenAufgetreten',array($fullName))."!</span>";
         }
     }
     else
@@ -325,13 +332,13 @@ if(isset($_GET['speichern']) && isset($_GET['wtag']))
 					{
 						$to.=', '.$vg.'@'.DOMAIN;
 						$name = $prsn->getFullNameFromBenutzer($vg);
-						$fullName = ', '.$name;
+						$fullName.=', '.$name;
 					}
 					else
 					{
 						$to.=$vg.'@'.DOMAIN;
 						$name = $prsn->getFullNameFromBenutzer($vg);
-						$fullName = $name;
+						$fullName.=$name;
 					}
 				}
 
diff --git a/cis/private/profile/zeitsperre_resturlaub.php b/cis/private/profile/zeitsperre_resturlaub.php
index 087b2ce80..efe6ed98d 100644
--- a/cis/private/profile/zeitsperre_resturlaub.php
+++ b/cis/private/profile/zeitsperre_resturlaub.php
@@ -419,17 +419,17 @@ if(isset($_GET['type']) && ($_GET['type']=='edit_sperre' || $_GET['type']=='new_
 					if($vorgesetzter)
 					{
 						$to='';
-						$fullName ='';
+						$fullName='';
 						foreach($ma->vorgesetzte as $vg)
 						{
 							if (!empty($to))
                             {
 								$to.=',';
-								$fullName =',';
+								$fullName.=',';
                             }
 							$to.=trim($vg.'@'.DOMAIN);
 							$name = $prsn->getFullNameFromBenutzer($vg);
-							$fullName = $name;
+							$fullName.=$name;
 						}
 
 						$benutzer = new benutzer();
@@ -452,7 +452,7 @@ if(isset($_GET['type']) && ($_GET['type']=='edit_sperre' || $_GET['type']=='new_
 						}
 						else
 						{
-							echo "<br><span class='error'>".$p->t('urlaubstool/fehlerBeimSendenAufgetreten',array($to))."</span>";
+							echo "<br><span class='error'>".$p->t('urlaubstool/fehlerBeimSendenAufgetreten',array($fullName))."</span>";
 						}
 					}
 					else
@@ -482,19 +482,26 @@ if((isset($_GET['type']) && $_GET['type']=='delete_sperre' && isset($_GET['infor
         echo $zeitsperre->errormsg;
 
     //Mail an Vorgesetzten
+    $prsn = new person();
+
     $vorgesetzter = $ma->getVorgesetzte($uid);
     if($vorgesetzter)
     {
         $to='';
+        $fullName ='';
         foreach($ma->vorgesetzte as $vg)
         {
             if($to!='')
             {
                 $to.=', '.$vg.'@'.DOMAIN;
+                $name = $prsn->getFullNameFromBenutzer($vg);
+                $fullName.=', '.$name;
             }
             else
             {
                 $to.=$vg.'@'.DOMAIN;
+                $name = $prsn->getFullNameFromBenutzer($vg);
+                $fullName.=$name;
             }
         }
 
@@ -510,11 +517,11 @@ if((isset($_GET['type']) && $_GET['type']=='delete_sperre' && isset($_GET['infor
         $mail = new mail($to, 'vilesci@'.DOMAIN,$p->t('urlaubstool/freigegebenerUrlaubGeloescht'), $message);
         if($mail->send())
         {
-            echo "<br><b>".$p->t('urlaubstool/VorgesetzteInformiert',array($to))."</b>";
+            echo "<br><b>".$p->t('urlaubstool/VorgesetzteInformiert',array($fullName))."</b>";
         }
         else
         {
-            echo "<br><span class='error'>".$p->t('urlaubstool/fehlerBeimSendenAufgetreten',array($to))."!</span>";
+            echo "<br><span class='error'>".$p->t('urlaubstool/fehlerBeimSendenAufgetreten',array($fullName))."!</span>";
         }
     }
     else