From 18e82225cebbd3b7226fb577fb946d99e4727639 Mon Sep 17 00:00:00 2001 From: Harald Bamberger Date: Wed, 26 Jul 2023 17:28:37 +0200 Subject: [PATCH] handle vertragsbestandteil_kurzbz different for insert and update --- .../vertragsbestandteil/AbstractBestandteil.php | 11 ++++++++++- .../vertragsbestandteil/Vertragsbestandteil.php | 4 ++-- 2 files changed, 12 insertions(+), 3 deletions(-) diff --git a/application/libraries/vertragsbestandteil/AbstractBestandteil.php b/application/libraries/vertragsbestandteil/AbstractBestandteil.php index 72119968b..bdaf46bd4 100644 --- a/application/libraries/vertragsbestandteil/AbstractBestandteil.php +++ b/application/libraries/vertragsbestandteil/AbstractBestandteil.php @@ -22,7 +22,16 @@ abstract class AbstractBestandteil } protected function markDirty($columnname, $old_value, $new_value) { - if( !$this->fromdb && ($old_value != $new_value) ) { + if( $this->fromdb ) { + // data comes from db dont check for changes + if( isset($this->modifiedcolumns[$columnname]) ) { + unset($this->modifiedcolumns[$columnname]); + } + return; + } + + if($old_value != $new_value) { + $this->modifiedcolumns[$columnname] = $columnname; } } diff --git a/application/libraries/vertragsbestandteil/Vertragsbestandteil.php b/application/libraries/vertragsbestandteil/Vertragsbestandteil.php index cb3b066f8..beda78773 100644 --- a/application/libraries/vertragsbestandteil/Vertragsbestandteil.php +++ b/application/libraries/vertragsbestandteil/Vertragsbestandteil.php @@ -40,6 +40,7 @@ abstract class Vertragsbestandteil extends AbstractBestandteil implements \JsonS isset($data->dienstverhaeltnis_id) && $this->setDienstverhaeltnis_id($data->dienstverhaeltnis_id); isset($data->von) && $this->setVon($data->von); isset($data->bis) && $this->setBis($data->bis); + isset($data->vertragsbestandteiltyp_kurzbz) && $this->setVertragsbestandteiltyp_kurzbz($data->vertragsbestandteiltyp_kurzbz); isset($data->insertamum) && $this->setInsertamum($data->insertamum); isset($data->insertvon) && $this->setInsertvon($data->insertvon); isset($data->updateamum) && $this->setUpdateamum($data->updateamum); @@ -149,8 +150,7 @@ abstract class Vertragsbestandteil extends AbstractBestandteil implements \JsonS public function setVertragsbestandteiltyp_kurzbz($vertragsbestandteiltyp_kurzbz) { - // type should never change and is set in constructor so do not mark dirty - //$this->markDirty('vertragsbestandteiltyp_kurzbz', $this->vertragsbestandteiltyp_kurzbz, $vertragsbestandteiltyp_kurzbz); + $this->markDirty('vertragsbestandteiltyp_kurzbz', $this->vertragsbestandteiltyp_kurzbz, $vertragsbestandteiltyp_kurzbz); $this->vertragsbestandteiltyp_kurzbz = $vertragsbestandteiltyp_kurzbz; return $this; }