From 6dd779b42108c62523c5429888c7bd41fea5d865 Mon Sep 17 00:00:00 2001
From: cgfhtw <christopher.gerbrich@technikum-wien.at>
Date: Tue, 6 Aug 2024 16:47:12 +0200
Subject: [PATCH] Status Abgewiesener

---
 .../api/frontend/v1/stv/Status.php            | 23 ++++-----
 application/libraries/PrestudentLib.php       | 51 +++++--------------
 2 files changed, 22 insertions(+), 52 deletions(-)

diff --git a/application/controllers/api/frontend/v1/stv/Status.php b/application/controllers/api/frontend/v1/stv/Status.php
index a002926f9..c25ffbe6c 100644
--- a/application/controllers/api/frontend/v1/stv/Status.php
+++ b/application/controllers/api/frontend/v1/stv/Status.php
@@ -480,40 +480,37 @@ class Status extends FHCAPI_Controller
 				$result = $this->prestudentlib->setDiplomand(
 					$prestudent_id,
 					$studiensemester_kurzbz,
-					$ausbildungssemester
+					$ausbildungssemester // TODO(chris): statusgrund_id
 				);
 				break;
 			case Prestudentstatus_model::STATUS_ABSOLVENT:
 				$result = $this->prestudentlib->setAbsolvent(
 					$prestudent_id,
 					$studiensemester_kurzbz,
-					$ausbildungssemester
+					$ausbildungssemester // TODO(chris): statusgrund_id
 				);
 				break;
 			case Prestudentstatus_model::STATUS_BEWERBER:
 				$result = $this->prestudentlib->setBewerber(
 					$prestudent_id,
 					$studiensemester_kurzbz,
-					$ausbildungssemester
+					$ausbildungssemester // TODO(chris): statusgrund_id
 				);
 				break;
 			case Prestudentstatus_model::STATUS_AUFGENOMMENER:
 				$result = $this->prestudentlib->setAufgenommener(
 					$prestudent_id,
 					$studiensemester_kurzbz,
-					$ausbildungssemester
+					$ausbildungssemester // TODO(chris): statusgrund_id
 				);
 				break;
 			case Prestudentstatus_model::STATUS_ABGEWIESENER:
-				$this->load->library('PrestudentLib');
-				$result = $this->prestudentlib->setAbgewiesener($prestudent_id, $studiensemester_kurzbz, $ausbildungssemester, $statusgrund_id);
-
-				if (isError($result))
-				{
-					return $this->terminateWithError(getError($result), self::ERROR_TYPE_GENERAL);
-				}
-				else
-					$this->terminateWithSuccess($prestudent_id);
+				$result = $this->prestudentlib->setAbgewiesener(
+					$prestudent_id,
+					$studiensemester_kurzbz,
+					$ausbildungssemester,
+					$statusgrund_id
+				);
 				break;
 			case Prestudentstatus_model::STATUS_WARTENDER:
 				$this->load->library('PrestudentLib');
diff --git a/application/libraries/PrestudentLib.php b/application/libraries/PrestudentLib.php
index 6394713e5..33a2ef328 100644
--- a/application/libraries/PrestudentLib.php
+++ b/application/libraries/PrestudentLib.php
@@ -518,7 +518,7 @@ class PrestudentLib
 
 	public function setDiplomand($prestudent_id, $studiensemester_kurzbz, $ausbildungssemester)
 	{
-		return $this->setBasic(Prestudentstatus_model::STATUS_DIPLOMAND, $prestudent_id, $studiensemester_kurzbz, $ausbildungssemester);
+		return $this->setBasic(getAuthUID(), date('c'), Prestudentstatus_model::STATUS_DIPLOMAND, $prestudent_id, $studiensemester_kurzbz, $ausbildungssemester);
 	}
 
 	public function setAbsolvent($prestudent_id, $studiensemester_kurzbz, $ausbildungssemester)
@@ -527,7 +527,7 @@ class PrestudentLib
 		$now = date('c');
 
 
-		$result = $this->setBasic(Prestudentstatus_model::STATUS_ABSOLVENT, $prestudent_id, $studiensemester_kurzbz, $ausbildungssemester);
+		$result = $this->setBasic($authUID, $now, Prestudentstatus_model::STATUS_ABSOLVENT, $prestudent_id, $studiensemester_kurzbz, $ausbildungssemester);
 
 		if (isError($result))
 			return $result;
@@ -563,7 +563,7 @@ class PrestudentLib
 
 	public function setBewerber($prestudent_id, $studiensemester_kurzbz, $ausbildungssemester)
 	{
-		$result = $this->setBasic(Prestudentstatus_model::STATUS_BEWERBER, $prestudent_id, $studiensemester_kurzbz, $ausbildungssemester);
+		$result = $this->setBasic(getAuthUID(), date('c'), Prestudentstatus_model::STATUS_BEWERBER, $prestudent_id, $studiensemester_kurzbz, $ausbildungssemester);
 
 		if (isError($result))
 			return $result;
@@ -577,40 +577,12 @@ class PrestudentLib
 
 	public function setAufgenommener($prestudent_id, $studiensemester_kurzbz, $ausbildungssemester)
 	{
-		return $this->setBasic(Prestudentstatus_model::STATUS_AUFGENOMMENER, $prestudent_id, $studiensemester_kurzbz, $ausbildungssemester);
+		return $this->setBasic(getAuthUID(), $now, Prestudentstatus_model::STATUS_AUFGENOMMENER, $prestudent_id, $studiensemester_kurzbz, $ausbildungssemester);
 	}
 
 	public function setAbgewiesener($prestudent_id, $studiensemester_kurzbz, $ausbildungssemester, $statusgrund_id)
 	{
-		$resultLastStatus = $this->_ci->PrestudentstatusModel->getLastStatus($prestudent_id);
-		if (isError($resultLastStatus))
-			return $resultLastStatus;
-		$resultLastStatus = getData($resultLastStatus);
-
-		$prestudent_status = current($resultLastStatus);
-
-		//Status updaten
-		$result = $this->_ci->PrestudentstatusModel->insert([
-			'prestudent_id' => $prestudent_id,
-			'status_kurzbz' => Prestudentstatus_model::STATUS_ABGEWIESENER,
-			'studiensemester_kurzbz' => $studiensemester_kurzbz,
-			'ausbildungssemester' => $ausbildungssemester,
-			'datum' => date('c'),
-			'insertvon' => getAuthUID(),
-			'insertamum' => date('c'),
-			'orgform_kurzbz'=> $prestudent_status->orgform_kurzbz,
-			'studienplan_id'=> $prestudent_status->studienplan_id,
-			'bestaetigtvon' => getAuthUID(),
-			'bestaetigtam' => date('c'),
-			'statusgrund_id' => $statusgrund_id
-		]);
-
-		if (isError($result))
-		{
-			return $result;
-		}
-		else
-			return success();
+		return $this->setBasic(getAuthUID(), $now, Prestudentstatus_model::STATUS_ABGEWIESENER, $prestudent_id, $studiensemester_kurzbz, $ausbildungssemester, $statusgrund_id);
 	}
 
 	public function setWartender($prestudent_id, $studiensemester_kurzbz, $ausbildungssemester)
@@ -645,7 +617,7 @@ class PrestudentLib
 			return success();
 	}
 
-	protected function setBasic($status_kurzbz, $prestudent_id, $studiensemester_kurzbz, $ausbildungssemester)
+	protected function setBasic($authUID, $now, $status_kurzbz, $prestudent_id, $studiensemester_kurzbz, $ausbildungssemester, $statusgrund_id = null)
 	{
 		$result = $this->_ci->PrestudentstatusModel->getLastStatus($prestudent_id);
 
@@ -667,13 +639,14 @@ class PrestudentLib
 			'status_kurzbz' => $status_kurzbz,
 			'studiensemester_kurzbz' => $studiensemester_kurzbz,
 			'ausbildungssemester' => $ausbildungssemester,
-			'datum' => date('c'),
-			'insertvon' => getAuthUID(),
-			'insertamum' => date('c'),
+			'datum' => $now,
+			'insertvon' => $authUID,
+			'insertamum' => $now,
 			'orgform_kurzbz'=> $prestudent_status->orgform_kurzbz,
 			'studienplan_id'=> $prestudent_status->studienplan_id,
-			'bestaetigtvon' => getAuthUID(),
-			'bestaetigtam' => date('c')
+			'bestaetigtvon' => $authUID,
+			'bestaetigtam' => $now,
+			'statusgrund_id' => $statusgrund_id
 		]);
 
 		if (isError($result))