From 08536d63df9e8f1d14e2f7eeae8307d2bf736e61 Mon Sep 17 00:00:00 2001
From: OliiverHacker <hacker@technikum-wien.at>
Date: Fri, 25 Sep 2020 12:19:18 +0200
Subject: [PATCH] display Full Name of Supervisors for vacation reqeust

---
 cis/private/profile/urlaubstool.php           |  9 +++++-
 cis/private/profile/zeitsperre_resturlaub.php | 12 ++++++--
 include/person.class.php                      | 29 +++++++++++++++++++
 3 files changed, 47 insertions(+), 3 deletions(-)

diff --git a/cis/private/profile/urlaubstool.php b/cis/private/profile/urlaubstool.php
index 9d33b25ae..08e9eb766 100644
--- a/cis/private/profile/urlaubstool.php
+++ b/cis/private/profile/urlaubstool.php
@@ -257,19 +257,26 @@ if(isset($_GET['speichern']) && isset($_GET['wtag']))
 		if(!$error)
 		{
 			//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;
 					}
 				}
 
@@ -295,7 +302,7 @@ if(isset($_GET['speichern']) && isset($_GET['wtag']))
 				$mail = new mail($to, 'vilesci@'.DOMAIN,$p->t('urlaubstool/freigabeansuchenUrlaub'), $message);
 				if($mail->send())
 				{
-					$vgmail="<span style='color:green;'>".$p->t('urlaubstool/freigabemailWurdeVersandt',array($to))."</span>";
+					$vgmail="<span style='color:green;'>".$p->t('urlaubstool/freigabemailWurdeVersandt',array($fullName))."</span>";
 				}
 				else
 				{
diff --git a/cis/private/profile/zeitsperre_resturlaub.php b/cis/private/profile/zeitsperre_resturlaub.php
index b7654310e..2f7840108 100644
--- a/cis/private/profile/zeitsperre_resturlaub.php
+++ b/cis/private/profile/zeitsperre_resturlaub.php
@@ -413,15 +413,23 @@ if(isset($_GET['type']) && ($_GET['type']=='edit_sperre' || $_GET['type']=='new_
 				if($zeitsperre->new && $zeitsperre->zeitsperretyp_kurzbz=='Urlaub')
 				{
 					//Beim Anlegen von neuen Urlauben wird ein Mail an den Vorgesetzten versendet um diesen Freizugeben
-					$vorgesetzter = $ma->getVorgesetzte($uid);
+					$prsn = new person();
+
+                    $vorgesetzter = $ma->getVorgesetzte($uid);
 					if($vorgesetzter)
 					{
 						$to='';
+						$fullName ='';
 						foreach($ma->vorgesetzte as $vg)
 						{
 							if (!empty($to))
+                            {
 								$to.=',';
+								$fullName =',';
+                            }
 							$to.=trim($vg.'@'.DOMAIN);
+							$name = $prsn->getFullNameFromBenutzer($vg);
+							$fullName = $name;
 						}
 
 						$benutzer = new benutzer();
@@ -440,7 +448,7 @@ if(isset($_GET['type']) && ($_GET['type']=='edit_sperre' || $_GET['type']=='new_
 						$mail = new mail($to, $from, 'Freigabeansuchen', $message);
 						if($mail->send())
 						{
-							echo "<br><b>".$p->t('urlaubstool/freigabemailWurdeVersandt',array($to))."</b>";
+							echo "<br><b>".$p->t('urlaubstool/freigabemailWurdeVersandt',array($fullName))."</b>";
 						}
 						else
 						{
diff --git a/include/person.class.php b/include/person.class.php
index edfa0ec7f..b220bb55b 100644
--- a/include/person.class.php
+++ b/include/person.class.php
@@ -1018,4 +1018,33 @@ class person extends basis_db
 			return false;
 		}
 	}
+
+	public function getFullNameFromBenutzer($uid)
+	{
+		$qry = "SELECT
+					*
+				FROM
+					public.tbl_person
+					JOIN public.tbl_benutzer USING(person_id)
+				WHERE
+					uid=".$this->db_add_param($uid, FHC_STRING);
+
+		if ($this->db_query($qry))
+		{
+			if ($row = $this->db_fetch_object())
+			{
+				return (string)$row->vorname.' '.$row->nachname;
+			}
+			else
+			{
+				$this->errormsg = 'Keine Personendaten zu dieser UID gefunden';
+				return false;
+			}
+		}
+		else
+		{
+			$this->errormsg = "Fehler beim Laden der Personendaten";
+			return false;
+		}
+	}
 }