From e33eb2e9a9eba33b97758f03c53bb5f8849585e6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andreas=20=C3=96sterreicher?= <oesi@technikum-wien.at>
Date: Tue, 9 Mar 2021 13:23:56 +0100
Subject: [PATCH] =?UTF-8?q?Zeitraumabfragen=20f=C3=BCr=20Projektphasen=20k?=
 =?UTF-8?q?orrigiert?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 cis/private/tools/zeitaufzeichnung.php        |  24 +-
 .../tools/zeitaufzeichnung_projektphasen.php  |  12 +-
 include/projektphase.class.php                | 232 ++++++++----------
 3 files changed, 129 insertions(+), 139 deletions(-)

diff --git a/cis/private/tools/zeitaufzeichnung.php b/cis/private/tools/zeitaufzeichnung.php
index 2a534dbea..83ae23fc3 100644
--- a/cis/private/tools/zeitaufzeichnung.php
+++ b/cis/private/tools/zeitaufzeichnung.php
@@ -543,7 +543,13 @@ echo '
 							var projphasenhtml = "";
 							for (var i = 0; i < json.length; i++)
 							{
-								projphasenhtml += "<option value = \'" + json[i].projektphase_id + "\'>" + json[i].bezeichnung + "<\/option>";
+								projphasenhtml += "<option value = \'" + json[i].projektphase_id + "\'>";
+								projphasenhtml += json[i].bezeichnung;
+								if(json[i].start != \'\' && json[i].ende !=\'\')
+								{
+									projphasenhtml += " ( "+json[i].start+" - "+json[i].ende+" )";
+								}
+								projphasenhtml += "<\/option>";
 							}
 
 							$("#projektphase").append(projphasenhtml);
@@ -1525,8 +1531,8 @@ if($projekt->getProjekteMitarbeiter($user, true))
 					printTableHeadings($fieldheadings, $za_simple);
 
 
-		    $tag=null;
-		   $woche=date('W');
+			$tag=null;
+			$woche=date('W');
 
 			$tagessumme='00:00';
 			$pausesumme='00:00';
@@ -1546,8 +1552,6 @@ if($projekt->getProjekteMitarbeiter($user, true))
 			{
 				$datumtag = $datum_obj->formatDatum($row->datum, 'Y-m-d');
 
-				//echo '<tr><th colspan="13">foo<th></tr>';
-
 				// Nach jedem Tag eine Summenzeile einfuegen
 				if(is_null($tag))
 					$tag = $datumtag;
@@ -1979,7 +1983,10 @@ function getDataForProjectOverviewCSV($user)
 	$projects = $projects_of_user->getProjekteListForMitarbeiter($user);
 
 	$projektphase = new projektphase();
-	$projektphasen = $projektphase->getProjectphaseForMitarbeiter($user);
+	if($projektphase->getProjectphaseForMitarbeiter($user))
+		$projektphasen = $projektphase->result;
+	else
+		$projetkphasen = array();
 
 	$csvData = array();
 
@@ -1993,9 +2000,8 @@ function getDataForProjectOverviewCSV($user)
 		$ende = $project->ende;
 
 		$csvData[] = array($titel, $projekt_kurzbz, $projekt_phase, $projekt_phase_id, $beginn, $ende);
-
-
 	}
+
 	foreach ($projektphasen as $prjp)
 	{
 		if (true)
@@ -2007,7 +2013,7 @@ function getDataForProjectOverviewCSV($user)
 			$beginn = $prjp->start;
 			$ende = $prjp->ende;
 
-			array_push($csvData, array($projekt_kurzbz, $projekt_phase, $projekt_phase_id, $beginn, $ende)  );
+			array_push($csvData, array($titel, $projekt_kurzbz, $projekt_phase, $projekt_phase_id, $beginn, $ende)  );
 		}
 	}
 
diff --git a/cis/private/tools/zeitaufzeichnung_projektphasen.php b/cis/private/tools/zeitaufzeichnung_projektphasen.php
index c6fe99363..4e4671367 100644
--- a/cis/private/tools/zeitaufzeichnung_projektphasen.php
+++ b/cis/private/tools/zeitaufzeichnung_projektphasen.php
@@ -4,16 +4,24 @@ require_once('../../../config/cis.config.inc.php');
 require_once('../../../include/functions.inc.php');
 require_once('../../../include/basis_db.class.php');
 require_once('../../../include/projektphase.class.php');
+require_once('../../../include/datum.class.php');
 
 if (!$db = new basis_db())
 	die('Es konnte keine Verbindung zum Server aufgebaut werden.');
 $user = get_uid();
+
+$datum_obj = new datum();
+
 if(isset($_GET['projekt_kurzbz'])) // TODO maybe check that phasen only shown if projekt is projekt of logged in user
 {
 	$projekt_kurzbz = $_GET['projekt_kurzbz'];
 	$projektphase = new projektphase();
 
-	$projektphasen_user = $projektphase->getProjectphaseForMitarbeiterByKurzBz($user, $projekt_kurzbz);
+	if($projektphase->getProjectphaseForMitarbeiterByKurzBz($user, $projekt_kurzbz))
+		$projektphasen_user = $projektphase->result;
+	else
+		$projektphasen_user = array();
+
 	$pp_user_ids = array();
 	foreach ($projektphasen_user as $pp_user)
 	{
@@ -29,6 +37,8 @@ if(isset($_GET['projekt_kurzbz'])) // TODO maybe check that phasen only shown if
 			{
 				$item['projektphase_id'] = $row->projektphase_id;
 				$item['bezeichnung'] = $row->bezeichnung;
+				$item['start'] = $datum_obj->formatDatum($row->start, 'd.m.Y');
+				$item['ende'] = $datum_obj->formatDatum($row->ende, 'd.m.Y');
 				$result_obj[] = $item;
 			}
 		}
diff --git a/include/projektphase.class.php b/include/projektphase.class.php
index a0e2b5c6b..1b0818ca0 100644
--- a/include/projektphase.class.php
+++ b/include/projektphase.class.php
@@ -587,34 +587,38 @@ public function getFortschritt($projektphase_id)
 		{
 			$projektphase = $this->getProjectphaseById($projektphase_id);
 			if(strtotime($projektphase->start))
-      		{
-        		$projektphase_start = date('Y-m-d', strtotime($projektphase->start));
-      		}
+			{
+				$projektphase_start = date('Y-m-d', strtotime($projektphase->start));
+			}
 			else
-      		{
-        		$projektphase_start = NULL;
-      		}
+			{
+				$projektphase_start = NULL;
+			}
 			if(strtotime($projektphase->ende))
-      		{
-        		$projektphase_ende = date('Y-m-d', strtotime($projektphase->ende));
-      		}
+			{
+				$projektphase_ende = date('Y-m-d', strtotime($projektphase->ende));
+			}
 			else
-      		{
-        		$projektphase_ende = NULL;
-      		}
+			{
+				$projektphase_ende = NULL;
+			}
 
 			$given_start = date('Y-m-d', strtotime($given_projectphase_start));
 			$given_ende = date('Y-m-d', strtotime($given_projektphase_ende));
 
-			if ((empty($projektphase_start) || $given_start >= $projektphase_start) && (empty($projektphase_ende) || $given_ende <= $projektphase_ende))
+			if ((empty($projektphase_start) || $given_start >= $projektphase_start)
+			&& (empty($projektphase_ende) || $given_ende <= $projektphase_ende))
+			{
 				return true;
+			}
 			else
+			{
 				return false;
-
+			}
 		}
 		catch (Exception $e)
 		{
-      error_log('Exception abgefangen: ',  $e->getMessage(), "\n");
+			error_log('Exception abgefangen: ',  $e->getMessage(), "\n");
 		}
 	}
 
@@ -681,120 +685,26 @@ public function getFortschritt($projektphase_id)
 	public function getProjectphaseForMitarbeiter($mitarbeiter_uid)
 	{
 		$projecphasetList = array();
-		/*$qry = "SELECT DISTINCT
-					tbl_projekt.*
-				FROM
-					fue.tbl_ressource
-					JOIN fue.tbl_projekt_ressource USING(ressource_id)
-					JOIN fue.tbl_projekt USING(projekt_kurzbz)
-				WHERE (beginn<=now() or beginn is null)
-				AND (ende + interval '1 month 1 day' >=now() OR ende is null)
-				AND
-				(
-					mitarbeiter_uid=" . $this->db_add_param($mitarbeiter_uid) . " OR
-					student_uid=" . $this->db_add_param($mitarbeiter_uid) . "
-				)";*/
 
 		$qry = "
-
-                             SELECT DISTINCT
-                                        tbl_projektphase.*
-                                FROM
-                                        fue.tbl_projektphase
-                                        JOIN fue.tbl_projekt USING (projekt_kurzbz)
-                                        JOIN fue.tbl_projekt_ressource USING (projektphase_id)
-                                        JOIN fue.tbl_ressource ON (tbl_ressource.ressource_id=tbl_projekt_ressource.ressource_id)
-                                WHERE
-                                (
-									(
-										(tbl_projekt.beginn<=now() or tbl_projekt.beginn is null)
-										AND (tbl_projekt.ende + interval '1 month 1 day' >=now() OR tbl_projekt.ende is null)
-									) OR (
-										(tbl_projektphase.start<=now() or tbl_projektphase.start is null)
-										AND (tbl_projektphase.ende + interval '1 month 1 day' >=now() OR tbl_projektphase.ende is null)
-									)
-								)
-                                AND mitarbeiter_uid=" . $this->db_add_param($mitarbeiter_uid);
-
-		if($result = $this->db_query($qry))
-		{
-			while($row = $this->db_fetch_object($result))
-			{
-					$obj = new projektphase();
-
-					$obj->projekt_kurzbz = $row->projekt_kurzbz;
-					$obj->projektphase_id = $row->projektphase_id;
-					$obj->projektphase_fk = $row->projektphase_fk;
-					$obj->bezeichnung = $row->bezeichnung;
-					$obj->typ = $row->typ;
-					$obj->beschreibung = $row->beschreibung;
-					$obj->start = $row->start;
-					$obj->ende = $row->ende;
-					$obj->personentage = $row->personentage;
-					$obj->farbe = $row->farbe;
-					$obj->budget = $row->budget;
-					$obj->ressource_id = $row->ressource_id;
-					$obj->insertamum = $row->insertamum;
-					$obj->insertvon = $row->insertvon;
-					$obj->updateamum = $row->updateamum;
-					$obj->updatevon = $row->updatevon;
-
-					$this->result[] = $obj;
-
-					array_push($projecphasetList, $obj);
-			}
-			return $projecphasetList;
-		}
-		else
-		{
-			$this->errormsg = 'Fehler beim Laden der Daten';
-			return false;
-		}
-	}
-
-	/**
-	 * Laedt die Projektphase mit der ID des mitarbeiters für das jeweilige Projekt
-	 * @param  $mitarbeiter_uid der zu ladenden Projektphase des users
-	 * @param  $prjkzbz des zu landenen Projekts
-	 * @return array wenn ok, false im Fehlerfall
-	 */
-	public function getProjectphaseForMitarbeiterByKurzBz($mitarbeiter_uid, $prjkzbz)
-	{
-		$projecphasetList = array();
-		/*$qry = "SELECT DISTINCT
-					tbl_projekt.*
-				FROM
-					fue.tbl_ressource
-					JOIN fue.tbl_projekt_ressource USING(ressource_id)
-					JOIN fue.tbl_projekt USING(projekt_kurzbz)
-				WHERE (beginn<=now() or beginn is null)
-				AND (ende + interval '1 month 1 day' >=now() OR ende is null)
-				AND
-				(
-					mitarbeiter_uid=" . $this->db_add_param($mitarbeiter_uid) . " OR
-					student_uid=" . $this->db_add_param($mitarbeiter_uid) . "
-				)";*/
-
-		$qry = "
-
-                             SELECT DISTINCT
-                                        tbl_projektphase.*
-                                FROM
-                                        fue.tbl_projektphase
-                                        JOIN fue.tbl_projekt USING (projekt_kurzbz)
-                                        JOIN fue.tbl_projekt_ressource USING (projektphase_id)
-                                        JOIN fue.tbl_ressource ON (tbl_ressource.ressource_id=tbl_projekt_ressource.ressource_id)
-                                WHERE
-                                (
-									(
-										(tbl_projekt.beginn<=now() or tbl_projekt.beginn is null)
-										AND (tbl_projekt.ende + interval '1 month 1 day' >=now() OR tbl_projekt.ende is null)
-									) OR (
-										(tbl_projektphase.start<=now() or tbl_projektphase.start is null)
-										AND (tbl_projektphase.ende + interval '1 month 1 day' >=now() OR tbl_projektphase.ende is null)
-									)
-								)
-                                AND mitarbeiter_uid=" . $this->db_add_param($mitarbeiter_uid);
+		SELECT
+			DISTINCT tbl_projektphase.*
+		FROM
+			fue.tbl_projektphase
+			JOIN fue.tbl_projekt USING (projekt_kurzbz)
+			JOIN fue.tbl_projekt_ressource USING (projektphase_id)
+			JOIN fue.tbl_ressource ON (tbl_ressource.ressource_id=tbl_projekt_ressource.ressource_id)
+		WHERE
+		(
+			(
+				(tbl_projekt.beginn<=now() or tbl_projekt.beginn is null)
+				AND (tbl_projekt.ende + interval '1 month 1 day' >=now() OR tbl_projekt.ende is null)
+			) AND (
+				(tbl_projektphase.start<=now() or tbl_projektphase.start is null)
+				AND (tbl_projektphase.ende + interval '1 month 1 day' >=now() OR tbl_projektphase.ende is null)
+			)
+		)
+		AND mitarbeiter_uid=" . $this->db_add_param($mitarbeiter_uid);
 
 		if($result = $this->db_query($qry))
 		{
@@ -821,8 +731,7 @@ public function getFortschritt($projektphase_id)
 
 				$this->result[] = $obj;
 
-				if($prjkzbz === $row->projekt_kurzbz )
-					array_push($projecphasetList, $obj);
+				array_push($projecphasetList, $obj);
 			}
 			return $projecphasetList;
 		}
@@ -832,5 +741,70 @@ public function getFortschritt($projektphase_id)
 			return false;
 		}
 	}
+
+	/**
+	 * Laedt die Projektphase mit der ID des mitarbeiters für das jeweilige Projekt
+	 * @param  $mitarbeiter_uid der zu ladenden Projektphase des users
+	 * @param  $projekt_kurzbz des zu landenen Projekts
+	 * @return array wenn ok, false im Fehlerfall
+	 */
+	public function getProjectphaseForMitarbeiterByKurzBz($mitarbeiter_uid, $projekt_kurzbz)
+	{
+		$projecphasetList = array();
+
+		$qry = "
+		SELECT
+			DISTINCT tbl_projektphase.*
+		FROM
+			fue.tbl_projektphase
+			JOIN fue.tbl_projekt USING (projekt_kurzbz)
+			JOIN fue.tbl_projekt_ressource USING (projektphase_id)
+			JOIN fue.tbl_ressource ON (tbl_ressource.ressource_id=tbl_projekt_ressource.ressource_id)
+		WHERE
+		(
+			(
+				(tbl_projekt.beginn<=now() or tbl_projekt.beginn is null)
+				AND (tbl_projekt.ende + interval '1 month 1 day' >=now() OR tbl_projekt.ende is null)
+			) AND (
+				(tbl_projektphase.start<=now() or tbl_projektphase.start is null)
+				AND (tbl_projektphase.ende + interval '1 month 1 day' >=now() OR tbl_projektphase.ende is null)
+			)
+		)
+		AND mitarbeiter_uid = ".$this->db_add_param($mitarbeiter_uid)."
+		AND tbl_projekt.projekt_kurzbz = ".$this->db_add_param($projekt_kurzbz);
+
+		if($result = $this->db_query($qry))
+		{
+			while($row = $this->db_fetch_object($result))
+			{
+				$obj = new projektphase();
+
+				$obj->projekt_kurzbz = $row->projekt_kurzbz;
+				$obj->projektphase_id = $row->projektphase_id;
+				$obj->projektphase_fk = $row->projektphase_fk;
+				$obj->bezeichnung = $row->bezeichnung;
+				$obj->typ = $row->typ;
+				$obj->beschreibung = $row->beschreibung;
+				$obj->start = $row->start;
+				$obj->ende = $row->ende;
+				$obj->personentage = $row->personentage;
+				$obj->farbe = $row->farbe;
+				$obj->budget = $row->budget;
+				$obj->ressource_id = $row->ressource_id;
+				$obj->insertamum = $row->insertamum;
+				$obj->insertvon = $row->insertvon;
+				$obj->updateamum = $row->updateamum;
+				$obj->updatevon = $row->updatevon;
+
+				$this->result[] = $obj;
+			}
+			return true;
+		}
+		else
+		{
+			$this->errormsg = 'Fehler beim Laden der Daten';
+			return false;
+		}
+	}
 }
 ?>