From e0a2f0490be8a4e5ff491cc6bea4dcf4992d4096 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andreas=20=C3=96sterreicher?= <oesi@technikum-wien.at>
Date: Mon, 14 Mar 2022 15:58:13 +0100
Subject: [PATCH] =?UTF-8?q?Laden=20der=20Untergebenen=20optimiert=20bei=20?=
 =?UTF-8?q?vielen=20Eintr=C3=A4gen?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 cis/private/tools/zeitaufzeichnung.php | 11 +++++------
 include/mitarbeiter.class.php          | 23 +++++++++++++----------
 2 files changed, 18 insertions(+), 16 deletions(-)

diff --git a/cis/private/tools/zeitaufzeichnung.php b/cis/private/tools/zeitaufzeichnung.php
index e94bc0fa4..ba57b442f 100644
--- a/cis/private/tools/zeitaufzeichnung.php
+++ b/cis/private/tools/zeitaufzeichnung.php
@@ -1008,18 +1008,17 @@ if ($projekt->getProjekteMitarbeiter($user, true))
 		//Dropdown timesheets Mitarbeiter
 		if ($untergebenen_arr)
 		{
-			$ben = new benutzer();
+			$mitarbeiter = new mitarbeiter();
+			$mitarbeiter->getMitarbeiterArray($untergebenen_arr);
+
 			echo "
 			<hr><br>
 
 			<select name='mas' id='mas' onchange='location = this.options[this.selectedIndex].value;''>
 				<option>-- Timesheets Mitarbeiter*Innen --</option>";
-			foreach($untergebenen_arr as $k=>$v)
+			foreach($mitarbeiter->result as $row)
 			{
-				if ($ben->load($v))
-				{
-					echo "<option value='zeitaufzeichnung.php?uid=$v'>$ben->vorname $ben->nachname</option>";
-				}
+				echo "<option value='zeitaufzeichnung.php?uid=$row->uid'>$row->nachname $row->vorname</option>";
 			}
 			echo "<option value='zeitaufzeichnung.php'> --zurück zur Übersicht--</option>";
 			echo "</select>";
diff --git a/include/mitarbeiter.class.php b/include/mitarbeiter.class.php
index 658154170..cbf31bba1 100644
--- a/include/mitarbeiter.class.php
+++ b/include/mitarbeiter.class.php
@@ -1468,22 +1468,25 @@ class mitarbeiter extends benutzer
 		$hasUDF = false;
 		$udf = new UDF();
 
-		$qry = "SELECT DISTINCT ON(mitarbeiter_uid) *,
-									tbl_benutzer.aktiv as aktiv,
-									tbl_mitarbeiter.insertamum,
-									tbl_mitarbeiter.insertvon,
-									tbl_mitarbeiter.updateamum,
-									tbl_mitarbeiter.updatevon";
+		$qry = "SELECT
+					*,
+					tbl_benutzer.aktiv as aktiv,
+					tbl_mitarbeiter.insertamum,
+					tbl_mitarbeiter.insertvon,
+					tbl_mitarbeiter.updateamum,
+					tbl_mitarbeiter.updatevon";
 
 		if ($hasUDF = $udf->personHasUDF())
 		{
 			$qry .= ", public.tbl_person.udf_values AS p_udf_values";
 		}
 
-		$qry .= " FROM ((public.tbl_mitarbeiter JOIN public.tbl_benutzer ON(mitarbeiter_uid=uid))
-					JOIN public.tbl_person USING(person_id))
-					LEFT JOIN public.tbl_benutzerfunktion USING(uid)
-				WHERE uid in(".$this->db_implode4SQL($uid_arr).")";;
+		$qry .= " FROM
+					public.tbl_mitarbeiter
+					JOIN public.tbl_benutzer ON(mitarbeiter_uid=uid)
+					JOIN public.tbl_person USING(person_id)
+				WHERE uid in(".$this->db_implode4SQL($uid_arr).")";
+		$qry .= " ORDER BY nachname, vorname";
 
 		if($this->db_query($qry))
 		{