From 472803dbeca2dd4d12c03864945753041b17f0c2 Mon Sep 17 00:00:00 2001
From: manu <ma0068@technikum-wien.at>
Date: Mon, 7 Jun 2021 16:55:39 +0200
Subject: [PATCH] 13412 Refactoring Functions Dokumente ZGV Master

---
 include/dokument.class.php   | 32 +++++---------------------------
 include/prestudent.class.php | 14 +++++---------
 2 files changed, 10 insertions(+), 36 deletions(-)

diff --git a/include/dokument.class.php b/include/dokument.class.php
index da03983ab..4d70891c0 100644
--- a/include/dokument.class.php
+++ b/include/dokument.class.php
@@ -888,7 +888,7 @@ class dokument extends basis_db
 	 * Akzeptiert ein bestimmtes Dokument
 	 * @param char $dokument_kurzbz Bezeichner Dokument.
 	 * @param int $person_id Personenkennzeichen.
-	 * @return boolean true wenn akzeptiert, false wenn noch nicht akzeptiert
+	 * @return boolean true wenn akzeptiert bzw geprüft ohne Akzeptieren, false wenn Fehler
 	 */
 	public function akzeptiereDokument($dokument_kurzbz, $person_id)
 	{
@@ -910,10 +910,7 @@ class dokument extends basis_db
 					where dok.prestudent_id = ps.prestudent_id
 					and dokument_kurzbz = ".$this->db_add_param($dokument_kurzbz).")";
 
-
 		//echo var_dump($qry);
-		//for all prestudents
-
 
 		//gibt ein Array von zu akzeptierenden Dokumenten zurück
 		if ($db->db_query($qry))
@@ -929,10 +926,8 @@ class dokument extends basis_db
 				}
 				//print_r($arrayDoksZuAkzeptieren);
 
-				//und jetzt für alle prestudent_ids das Dokument akzeptieren
-
+				//für alle prestudent_ids das Dokument akzeptieren
 				$qry = "INSERT INTO public.tbl_dokumentprestudent(dokument_kurzbz, prestudent_id) VALUES";
-				//echo $qry;
 
 				foreach ($arrayDoksZuAkzeptieren as $prestudent_id)
 				{
@@ -945,11 +940,8 @@ class dokument extends basis_db
 				}
 				$qry .=  ";";
 
-				echo $qry;
-
 				if ($this->db_query($qry))
 				{
-					echo " Jawoll: query ausgeführt";
 					return true;
 				}
 				else
@@ -958,10 +950,6 @@ class dokument extends basis_db
 					return false;
 				}
 			}
-			else
-			{
-				echo " Keine zu akzeptierenden Doks vorhanden";
-			}
 			return true;
 		}
 
@@ -973,7 +961,7 @@ class dokument extends basis_db
 	 * entakzeptiert ein bestimmtes Dokument
 	 * @param char $dokument_kurzbz Kurzbezeichnung des zu entakzeptierenden Dokuments.
 	 * @param int $person_id Personenkennzeichen.
-	 * @return boolean true wenn entakzeptiert, false wenn noch nicht entakzeptiert
+	 * @return boolean true wenn entakzeptiert bzw geprüft ohne Entakzeptieren, false wenn Fehler
 	 */
 	public function entakzeptiereDokument($dokument_kurzbz, $person_id)
 	{
@@ -981,7 +969,6 @@ class dokument extends basis_db
 		$arrayDoksZuEntakzeptieren = array();
 
 		//get Prestudent_ids
-
 		$qry = "SELECT
 					prestudent_id
 				from
@@ -1003,15 +990,12 @@ class dokument extends basis_db
 			{
 				while ($row = $db->db_fetch_object())
 				{
-					//echo var_dump($row->prestudent_id);
 					$arrayDoksZuEntakzeptieren[] = $row->prestudent_id;
 				}
-				print_r($arrayDoksZuEntakzeptieren);
-
-				//und jetzt für alle prestudent_ids das Dokument Entakzeptieren
+				//print_r($arrayDoksZuEntakzeptieren);
 
+				//für alle prestudent_ids das Dokument Entakzeptieren
 				$qry = "DELETE FROM public.tbl_dokumentprestudent WHERE prestudent_id in (";
-				//echo $qry;
 
 				foreach ($arrayDoksZuEntakzeptieren as $prestudent_id)
 				{
@@ -1024,8 +1008,6 @@ class dokument extends basis_db
 				}
 				$qry .=  ") AND dokument_kurzbz = ".$this->db_add_param($dokument_kurzbz).";";
 
-				echo $qry;
-
 				if ($this->db_query($qry))
 				{
 					return true;
@@ -1036,10 +1018,6 @@ class dokument extends basis_db
 					return false;
 				}
 			}
-			else
-			{
-				echo " Keine Entzuakzeptierenden Doks vorhanden";
-			}
 			return true;
 		}
 		else
diff --git a/include/prestudent.class.php b/include/prestudent.class.php
index 06115dab2..747fd5da4 100644
--- a/include/prestudent.class.php
+++ b/include/prestudent.class.php
@@ -2215,9 +2215,9 @@ class prestudent extends person
 				WHERE laststatus NOT IN ('Abbrecher', 'Abgewiesener', 'Absolvent')
 				AND priorisierung <= ".$this->db_add_param($priorisierungAbsolut, FHC_INTEGER);
 
-		if($result = $this->db_query($qry))
+		if ($result = $this->db_query($qry))
 		{
-			if($row = $this->db_fetch_object($result))
+			if ($row = $this->db_fetch_object($result))
 			{
 				return $row->prio_relativ;
 			}
@@ -2328,7 +2328,7 @@ class prestudent extends person
 					//echo var_dump($row->prestudent_id);
 					$arrayleereManations[] = $row->prestudent_id;
 				}
-				print_r($arrayleereManations);
+				//print_r($arrayleereManations);
 
 				$qry = "UPDATE
 					public.tbl_prestudent
@@ -2352,7 +2352,7 @@ class prestudent extends person
 
 				if ($this->db_query($qry))
 				{
-					echo " ZGV-Master Nation A eingetragen!";
+					//echo " ZGV-Master Nation A eingetragen!";
 					return true;
 				}
 				else
@@ -2362,7 +2362,7 @@ class prestudent extends person
 				}
 			}
 			else
-				echo " Keine leere ZGVManation gefunden";
+				//echo " Keine leere ZGVManation gefunden";
 				return true;
 		}
 	}
@@ -2447,9 +2447,6 @@ class prestudent extends person
 				//prestudentIds auf Interessentenstatus prüfen
 				foreach ($prestudentsOfMaster as $prestudent_id)
 				{
-					//echo $prestudent_id;
-					//echo "last status = " . $this->getLastPrestudentStatus($prestudent_id);
-					//echo "<br>";
 					if ($this->getLastPrestudentStatus($prestudent_id) == "Interessent")
 					{
 						return true;
@@ -2460,5 +2457,4 @@ class prestudent extends person
 		else
 			return false;
 	}
-
 }