From 34a083d0ee144d4ba83bc66ab882b31fb1b85ee4 Mon Sep 17 00:00:00 2001
From: Harald Bamberger <harald.bamberger@technikum-wien.at>
Date: Mon, 15 Jan 2024 18:16:54 +0100
Subject: [PATCH] update Logik beim Beenden von DVs verwenden

---
 .../GehaltsbestandteilLib.php                 | 13 ++-------
 .../VertragsbestandteilLib.php                | 28 ++++---------------
 2 files changed, 7 insertions(+), 34 deletions(-)

diff --git a/application/libraries/vertragsbestandteil/GehaltsbestandteilLib.php b/application/libraries/vertragsbestandteil/GehaltsbestandteilLib.php
index 1e1fb1ec3..b75bdd722 100644
--- a/application/libraries/vertragsbestandteil/GehaltsbestandteilLib.php
+++ b/application/libraries/vertragsbestandteil/GehaltsbestandteilLib.php
@@ -133,17 +133,8 @@ class GehaltsbestandteilLib
 			return;
 		}
 		
-		$ret = $this->GehaltsbestandteilModel->update($gehaltsbestandteil->getGehaltsbestandteil_id(), 
-			(object) array(
-				'bis' => $enddate, 
-				'updatevon' => getAuthUID(),
-				'updateamum' => strftime('%Y-%m-%d %H:%M')
-			));
-		
-		if (isError($ret))
-		{
-			throw new Exception('error ending gehaltsbestandteil');
-		}
+		$gehaltsbestandteil->setBis($enddate);
+		$this->updateGehaltsbestandteil($gehaltsbestandteil);
 	}
 		
 	protected function setUIDtoPGSQL() {
diff --git a/application/libraries/vertragsbestandteil/VertragsbestandteilLib.php b/application/libraries/vertragsbestandteil/VertragsbestandteilLib.php
index a6c99ab43..8fb3900d5 100644
--- a/application/libraries/vertragsbestandteil/VertragsbestandteilLib.php
+++ b/application/libraries/vertragsbestandteil/VertragsbestandteilLib.php
@@ -408,18 +408,8 @@ class VertragsbestandteilLib
 					$this->endVertragsbestandteil($vb, $enddate);
 				}			    				
 				
-				$ret = $this->DienstverhaeltnisModel->update($dv->getDienstverhaeltnis_id(),
-					(object) array(
-						'bis' => $enddate, 
-						'updatevon' => getAuthUID(),
-						'updateamum' => strftime('%Y-%m-%d %H:%M')
-					));
-				if(isError($ret) )
-				{
-					log_message('debug', "end DV failed");
-					throw new Exception('error ending dienstverhaeltnis '
-						. $dv->getDienstverhaeltnis_id());
-				}
+				$dv->setBis($enddate);
+				$this->updateDienstverhaeltnis($dv);
 
 				if( $this->CI->db->trans_status() === false )
 				{
@@ -431,6 +421,7 @@ class VertragsbestandteilLib
 		}
 		catch (Exception $ex)
 		{
+			log_message('debug', "end DV failed " . $dv->getDienstverhaeltnis_id());
 			log_message('debug', "Transaction rolled back. " . $ex->getMessage());
 			$this->CI->db->trans_rollback();
 			return $ex->getMessage();
@@ -445,17 +436,8 @@ class VertragsbestandteilLib
 			return;
 		}
 		
-		$ret = $this->VertragsbestandteilModel->update($vertragsbestandteil->getVertragsbestandteil_id(), 
-			(object) array(
-				'bis' => $enddate, 
-				'updatevon' => getAuthUID(),
-				'updateamum' => strftime('%Y-%m-%d %H:%M')
-			));
-		
-		if (isError($ret))
-		{
-			throw new Exception('error ending vertragsbestandteil');
-		}
+		$vertragsbestandteil->setBis($enddate);
+		$this->updateVertragsbestandteil($vertragsbestandteil);
 	}
 	
 	protected function setUIDtoPGSQL() {