From 95fa0cfbf65741f136a296259188cc2e0b32415a Mon Sep 17 00:00:00 2001
From: Harald Bamberger <harald.bamberger@technikum-wien.at>
Date: Wed, 8 Nov 2023 11:57:57 +0100
Subject: [PATCH] change date handling benutzerfunktion

---
 .../VertragsbestandteilFunktion.php           | 28 +++++++++++++------
 1 file changed, 20 insertions(+), 8 deletions(-)

diff --git a/application/libraries/vertragsbestandteil/VertragsbestandteilFunktion.php b/application/libraries/vertragsbestandteil/VertragsbestandteilFunktion.php
index d935526a2..7b9d35906 100644
--- a/application/libraries/vertragsbestandteil/VertragsbestandteilFunktion.php
+++ b/application/libraries/vertragsbestandteil/VertragsbestandteilFunktion.php
@@ -42,18 +42,30 @@ class VertragsbestandteilFunktion extends Vertragsbestandteil
 	}
 	
 	protected function beforePersitExisting() {
-		$data = (object) array(
-			'datum_bis' => $this->getBis(),
-			'updateamum' => strftime('%Y-%m-%d %H:%M:%S'),
-			'updatevon' => getAuthUID()
-		);
+		$data = array();
 		
 		$curbfres = $this->CI->BenutzerfunktionModel->load($this->getBenutzerfunktion_id());
-		$curbf = (getData($curbfres))[0];
-		if( $curbf && ($this->getVon() < $curbf->datum_von) ) 
+		if(hasData($curbfres))
 		{
-			$data->datum_von = $this->getVon();
+			$curbf = (getData($curbfres))[0];
+			if($this->getVon() < $curbf->datum_von) 
+			{
+				$data['datum_von'] = $this->getVon();
+			}
+			if($this->getBis() === null 
+				|| ($curbf->datum_bis !== null && ($this->getBis() < $curbf->datum_bis))) 
+			{
+				$data['datum_bis'] = $this->getBis();
+			}
 		}
+
+		if( count($data) === 0 ) 
+		{
+			return;
+		}
+		
+		$data['updateamum'] = strftime('%Y-%m-%d %H:%M:%S');
+		$data['updatevon'] = getAuthUID();		
 		
 		$ret = $this->CI->BenutzerfunktionModel->update($this->getBenutzerfunktion_id(), $data);