From 9fb05ad55e8b16f69b26e8791a963152cbc3c0c8 Mon Sep 17 00:00:00 2001
From: ma0068 <ma0068@technikum-wien.at>
Date: Thu, 12 Oct 2023 10:41:41 +0200
Subject: [PATCH] =?UTF-8?q?try-catch-block=20f=C3=BCr=20dbinsert=20Betreuu?=
 =?UTF-8?q?ngen?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../controllers/jobs/ProjektarbeitJob.php     | 64 +++++++++++--------
 1 file changed, 37 insertions(+), 27 deletions(-)

diff --git a/application/controllers/jobs/ProjektarbeitJob.php b/application/controllers/jobs/ProjektarbeitJob.php
index 0497bc6cd..21542d9f9 100644
--- a/application/controllers/jobs/ProjektarbeitJob.php
+++ b/application/controllers/jobs/ProjektarbeitJob.php
@@ -124,7 +124,7 @@ class ProjektarbeitJob extends JOB_Controller
 				'final' => $projektarbeit->final,
 			]);
 
-			$this->db->order_by("projektarbeit_id", "desc"); //TODO(manu) Cronjob soll weiterlaufen, continue?
+			$this->db->order_by("projektarbeit_id", "desc");
 			$result = $this->ProjektarbeitModel->loadWhere([
 				'student_uid' => $projektarbeit->student_uid,
 				'insertvon' => 'Projektjob',
@@ -158,7 +158,10 @@ class ProjektarbeitJob extends JOB_Controller
 				'projektarbeit_id' => $projekt->projektarbeit_id
 			]);
 			if (isError($result))
+			{
 				$this->logError(getError($result));
+				continue;
+			}
 
 			elseif (!hasData($result))
 			{
@@ -168,34 +171,42 @@ class ProjektarbeitJob extends JOB_Controller
 			{
 				$betreuung = getData($result);
 
-				foreach ($betreuung as $bet) {
+				//error handler, catching also warnings
+				set_error_handler(function ($err_severity, $err_msg, $err_file, $err_line, array $err_context)
+				{
+					throw new ErrorException( $err_msg, 0, $err_severity, $err_file, $err_line );
+				}, E_WARNING);
+
+				foreach ($betreuung as $bet)
+				{
 					$now = new Datetime();
-					$result = $this->ProjektbetreuerModel->insert([
-						'person_id' => $bet->person_id,
-						'projektarbeit_id' => $projekt_id_copy,
-						'note' => null,
-						'faktor' => $bet->faktor,
-						'name' => $bet->name,
-						'punkte' => $bet->punkte,
-						'stundensatz' => $bet->stundensatz,
-						'updateamum' => null,
-						'updatevon' => null,
-						'insertamum' => $now->format('c'),
-						'insertvon' => 'Projektjob',
-						'ext_id' => $bet->ext_id,
-						'betreuerart_kurzbz' => $bet->betreuerart_kurzbz,
-						'stunden' => null,
-						'vertrag_id' => null,
-						'zugangstoken' => null, //TODO(manu) sonst insertfehler DB: 1 datensatz 34195
-						'zugangstoken_gueltigbis' => null //TODO analog zu token
-					]);
-					if (isError($result))
-					{
-						$this->logError(getError($result));
+					try{
+						$result = $this->ProjektbetreuerModel->insert([
+							'person_id' => $bet->person_id,
+							'projektarbeit_id' => $projekt_id_copy,
+							'note' => null,
+							'faktor' => $bet->faktor,
+							'name' => $bet->name,
+							'punkte' => $bet->punkte,
+							'stundensatz' => $bet->stundensatz,
+							'updateamum' => null,
+							'updatevon' => null,
+							'insertamum' => $now->format('c'),
+							'insertvon' => 'Projektjob',
+							'ext_id' => $bet->ext_id,
+							'betreuerart_kurzbz' => $bet->betreuerart_kurzbz,
+							'stunden' => null,
+							'vertrag_id' => null,
+							'zugangstoken' => null,
+							'zugangstoken_gueltigbis' => null
+						]);
+					}
+					catch (ErrorException $e){
+						$this->logError("Error Insert Betreuungen, check projekt_id: " . $projekt_id_copy);
+						continue;
 					}
-				//	else
-				//		echo $nl . "neue Betreuung für person_id " . $bet->person_id . ' und projektarbeit_id ' . $projekt_id_copy . ' angelegt';
 				}
+				restore_error_handler();
 			}
 		}
 
@@ -234,6 +245,5 @@ class ProjektarbeitJob extends JOB_Controller
 		}
 		$this->logInfo($countTotal . ' projektarbeiten not uploaded in time, ' . $countMails . ' sent mails.');
 		$this->logInfo('End Job Projektarbeit');
-		echo $nl . 'End Job Projektarbeit: ' . $countTotal . ' projektarbeiten not uploaded in time, ' . $countMails . ' sent mails. ' . $nl;
 	}
 }