From e3ec4cd5aa96274be3b9292c5d30771f8c6e0ded Mon Sep 17 00:00:00 2001 From: Harald Bamberger Date: Mon, 1 May 2023 20:50:07 +0200 Subject: [PATCH] fix method names, handle insertamum updateamum insertvon updatevon in libs --- .../vertragsbestandteil/Dienstverhaeltnis.php | 25 ++++++++++++------- .../Gehaltsbestandteil.php | 5 +++- .../GehaltsbestandteilLib.php | 8 +++++- .../VertragsbestandteilLib.php | 19 ++++++++++---- .../VertragsbestandteilStunden.php | 2 +- 5 files changed, 42 insertions(+), 17 deletions(-) diff --git a/application/libraries/vertragsbestandteil/Dienstverhaeltnis.php b/application/libraries/vertragsbestandteil/Dienstverhaeltnis.php index 187325c7d..5cc264138 100644 --- a/application/libraries/vertragsbestandteil/Dienstverhaeltnis.php +++ b/application/libraries/vertragsbestandteil/Dienstverhaeltnis.php @@ -50,11 +50,16 @@ class Dienstverhaeltnis implements IValidation { public function toStdClass(): \stdClass { $tmp = array( - 'dienstverhaeltnis_id' => $this->getDienstverhaeltnisId(), - 'vertragsart_kurzbz' => $this->getVertragsartKurzbz(), - 'oe_kurzbz' => $this->getUnternehmen(), - 'gueltig_ab' => $this->getGueltigAb(), - 'gueltig_bis' => $this->getGueltigBis(), + 'dienstverhaeltnis_id' => $this->getDienstverhaeltnis_id(), + 'mitarbeiter_uid' => $this->getMitarbeiter_uid(), + 'vertragsart_kurzbz' => $this->getVertragsart_kurzbz(), + 'oe_kurzbz' => $this->getOe_kurzbz(), + 'von' => $this->getVon(), + 'bis' => $this->getBis(), + 'insertamum' => $this->getInsertamum(), + 'insertvon' => $this->getInsertvon(), + 'updateamum' => $this->getUpdateamum(), + 'updatevon' => $this->getUpdatevon() ); $tmp = array_filter($tmp, function($v) { @@ -68,10 +73,12 @@ class Dienstverhaeltnis implements IValidation { public function __toString() { $txt = <<getDienstverhaeltnisId()} - vertragsart_kurzbz: {$this->getVertragsartKurzbz()} - gueltig_ab: {$this->getGueltigAb()} - gueltig_bis: {$this->getGueltigBis()} + dienstverhaeltnis_id: {$this->getDienstverhaeltnis_id()} + mitarbeiter_uid: {$this->getMitarbeiter_uid()} + vertragsart_kurzbz: {$this->getVertragsart_kurzbz()} + oe_kurzbz: {$this->getOe_kurzbz()} + von: {$this->getVon()} + bis: {$this->getBis()} EOTXT; return $txt; diff --git a/application/libraries/vertragsbestandteil/Gehaltsbestandteil.php b/application/libraries/vertragsbestandteil/Gehaltsbestandteil.php index 88fce2e4e..3c0824f25 100644 --- a/application/libraries/vertragsbestandteil/Gehaltsbestandteil.php +++ b/application/libraries/vertragsbestandteil/Gehaltsbestandteil.php @@ -294,7 +294,10 @@ EOTXT; public function validate() { //do Validation here - + if( empty($this->grundbetrag) ) + { + $this->validationerrors[] = "Betrag fehlt."; + } // return status after Validation if( count($this->validationerrors) > 0 ) { $this->isvalid = false; diff --git a/application/libraries/vertragsbestandteil/GehaltsbestandteilLib.php b/application/libraries/vertragsbestandteil/GehaltsbestandteilLib.php index 3859eba43..e72de9cd9 100644 --- a/application/libraries/vertragsbestandteil/GehaltsbestandteilLib.php +++ b/application/libraries/vertragsbestandteil/GehaltsbestandteilLib.php @@ -15,8 +15,11 @@ class GehaltsbestandteilLib /** @var Gehaltsbestandteil_model */ protected $GehaltsbestandteilModel; + protected $loggedInUser; + public function __construct() { + $this->loggedInUser = getAuthUID(); $this->CI = get_instance(); $this->CI->load->model('vertragsbestandteil/Gehaltsbestandteil_model', 'GehaltsbestandteilModel'); @@ -63,6 +66,8 @@ class GehaltsbestandteilLib protected function insertGehaltsbestandteil(Gehaltsbestandteil $gehaltsbestandteil) { + $gehaltsbestandteil->setInsertvon($this->loggedInUser) + ->setInsertamum(strftime('%Y-%m-%d %H:%M:%S')); $ret = $this->GehaltsbestandteilModel->insert($gehaltsbestandteil->toStdClass(), $this->GehaltsbestandteilModel->getEncryptedColumns()); if( hasData($ret) ) @@ -77,7 +82,8 @@ class GehaltsbestandteilLib protected function updateGehaltsbestandteil(Gehaltsbestandteil $gehaltsbestandteil) { - $gehaltsbestandteil->setUpdateamum(strftime('%Y-%m-%d %H:%M:%S')); + $gehaltsbestandteil->setUpdatevon($this->loggedInUser) + ->setUpdateamum(strftime('%Y-%m-%d %H:%M:%S')); $ret = $this->GehaltsbestandteilModel->update($gehaltsbestandteil->getGehaltsbestandteil_id(), $gehaltsbestandteil->toStdClass(), $this->GehaltsbestandteilModel->getEncryptedColumns()); diff --git a/application/libraries/vertragsbestandteil/VertragsbestandteilLib.php b/application/libraries/vertragsbestandteil/VertragsbestandteilLib.php index 57f93bd44..8e1420495 100644 --- a/application/libraries/vertragsbestandteil/VertragsbestandteilLib.php +++ b/application/libraries/vertragsbestandteil/VertragsbestandteilLib.php @@ -30,8 +30,11 @@ class VertragsbestandteilLib */ protected $GehaltsbestandteilLib; + protected $loggedInUser; + public function __construct() { + $this->loggedInUser = getAuthUID(); $this->CI = get_instance(); $this->CI->load->model('vertragsbestandteil/Dienstverhaeltnis_model', 'DienstverhaeltnisModel'); @@ -82,13 +85,13 @@ class VertragsbestandteilLib public function storeDienstverhaeltnis(Dienstverhaeltnis $dv) { - if( inval($dv->getDienstverhaeltnis_id()) > 0 ) + if( intval($dv->getDienstverhaeltnis_id()) > 0 ) { - $this->insertDienstverhaeltnis($dv); + $this->updateDienstverhaeltnis($dv); } else { - $this->updateDienstverhaeltnis($dv); + $this->insertDienstverhaeltnis($dv); } } @@ -122,6 +125,8 @@ class VertragsbestandteilLib protected function insertDienstverhaeltnis(Dienstverhaeltnis $dv) { + $dv->setInsertvon($this->loggedInUser) + ->setInsertamum(strftime('%Y-%m-%d %H:%M:%S')); $ret = $this->DienstverhaeltnisModel->insert($dv->toStdClass()); if( hasData($ret) ) { @@ -135,6 +140,8 @@ class VertragsbestandteilLib protected function insertVertragsbestandteil(Vertragsbestandteil $vertragsbestandteil) { + $vertragsbestandteil->setInsertvon($this->loggedInUser) + ->setInsertamum(strftime('%Y-%m-%d %H:%M:%S')); $vertragsbestandteil->beforePersist(); $ret = $this->VertragsbestandteilModel->insert($vertragsbestandteil->baseToStdClass()); if( hasData($ret) ) @@ -170,7 +177,8 @@ class VertragsbestandteilLib protected function updateDienstverhaeltnis(Dienstverhaeltnis $dv) { - $dv->setUpdateamum(strftime('%Y-%m-%d %H:%M:%S')); + $dv->setUpdatevon($this->loggedInUser) + ->setUpdateamum(strftime('%Y-%m-%d %H:%M:%S')); $ret = $this->DienstverhaeltnisModel->update($dv->getDienstverhaeltnis_id(), $dv->toStdClass()); if(isError($ret) ) @@ -181,7 +189,8 @@ class VertragsbestandteilLib protected function updateVertragsbestandteil(Vertragsbestandteil $vertragsbestandteil) { - $vertragsbestandteil->setUpdateamum(strftime('%Y-%m-%d %H:%M:%S')); + $vertragsbestandteil->setUpdatevon($this->loggedInUser) + ->setUpdateamum(strftime('%Y-%m-%d %H:%M:%S')); $vertragsbestandteil->beforePersist(); $ret = $this->VertragsbestandteilModel->update($vertragsbestandteil->getVertragsbestandteil_id(), $vertragsbestandteil->baseToStdClass()); diff --git a/application/libraries/vertragsbestandteil/VertragsbestandteilStunden.php b/application/libraries/vertragsbestandteil/VertragsbestandteilStunden.php index e99d5e10e..fd4436f72 100644 --- a/application/libraries/vertragsbestandteil/VertragsbestandteilStunden.php +++ b/application/libraries/vertragsbestandteil/VertragsbestandteilStunden.php @@ -80,7 +80,7 @@ EOTXT; public function validate() { - if( (floatval($this->benutzerfunktion_id) < 1 ) ) { + if( (floatval($this->wochenstunden) < 1 ) ) { $this->validationerrors[] = 'Stunden zu niedrig.'; }