From 2eef05e593fa5254d97575c23e346ec3b3c8c931 Mon Sep 17 00:00:00 2001 From: ma0068 Date: Wed, 11 Jun 2025 15:02:31 +0200 Subject: [PATCH] - refactore POST_Variablen with form_validation->set_data - add phrase: Vertragsstatus bearbeiten - filterActive: bugfix --- .../api/frontend/v1/vertraege/Vertraege.php | 75 ++++++++----------- public/js/apps/Vertragsverwaltung.js | 6 +- .../components/DetailHeader/DetailHeader.js | 2 + public/js/components/Vertraege/List/Status.js | 3 +- .../components/Vertraege/MitarbeiterHeader.js | 2 +- .../Vertraege/Vertragsverwaltung.js | 1 - 6 files changed, 37 insertions(+), 52 deletions(-) diff --git a/application/controllers/api/frontend/v1/vertraege/Vertraege.php b/application/controllers/api/frontend/v1/vertraege/Vertraege.php index 579883c8d..0d4b06a30 100644 --- a/application/controllers/api/frontend/v1/vertraege/Vertraege.php +++ b/application/controllers/api/frontend/v1/vertraege/Vertraege.php @@ -117,22 +117,15 @@ class Vertraege extends FHCAPI_Controller } $formData = $this->input->post('formData'); + $vertragstyp_kurzbz = $formData['vertragstyp_kurzbz'] ?? null; + $vertragsdatum = $formData['vertragsdatum'] ?? null; + $bezeichnung = $formData['bezeichnung'] ?? null; + $betrag = $formData['betrag'] ?? null; + $vertragsstunden = $formData['vertragsstunden'] ?? null; + $vertragsstunden_studiensemester_kurzbz = $formData['vertragsstunden_studiensemester_kurzbz'] ?? null; + $anmerkung = $formData['anmerkung'] ?? null; - //Manually rewrite to postparameter for form validation - $_POST['vertragstyp_kurzbz'] = $formData['vertragstyp_kurzbz']; - $_POST['vertragsdatum']= $formData['vertragsdatum']; - $_POST['bezeichnung']= $formData['bezeichnung']; - $_POST['betrag'] = $formData['betrag']; - $_POST['vertragsstunden'] = - isset($formData['vertragsstunden']) - ? $formData['vertragsstunden'] - : null; - $_POST['vertragsstunden_studiensemester_kurzbz'] = - isset($formData['vertragsstunden_studiensemester_kurzbz']) - ? $formData['vertragsstunden_studiensemester_kurzbz'] - : null; - $_POST['anmerkung'] = isset($formData['anmerkung']) ? $formData['anmerkung'] : null; - + $this->form_validation->set_data($formData); $this->form_validation->set_rules('bezeichnung', 'Bezeichnung', 'required', [ 'required' => $this->p->t('ui', 'error_fieldRequired', ['field' => 'Bezeichnung']) ]); @@ -163,13 +156,13 @@ class Vertraege extends FHCAPI_Controller $result = $this->VertragModel->insert([ 'person_id' => $person_id, - 'vertragsdatum' => $this->input->post('vertragsdatum'), - 'bezeichnung' => $this->input->post('bezeichnung'), - 'vertragstyp_kurzbz' => $this->input->post('vertragstyp_kurzbz'), - 'betrag' => $this->input->post('betrag'), - 'vertragsstunden' => $this->input->post('vertragsstunden'), - 'vertragsstunden_studiensemester_kurzbz' => $this->input->post('vertragsstunden_studiensemester_kurzbz'), - 'anmerkung' => $this->input->post('anmerkung'), + 'vertragsdatum' => $vertragsdatum, + 'bezeichnung' => $bezeichnung, + 'vertragstyp_kurzbz' => $vertragstyp_kurzbz, + 'betrag' => $betrag, + 'vertragsstunden' => $vertragsstunden, + 'vertragsstunden_studiensemester_kurzbz' => $vertragsstunden_studiensemester_kurzbz, + 'anmerkung' => $anmerkung, 'insertamum' => date('c'), 'insertvon' => getAuthUID() ]); @@ -236,7 +229,6 @@ class Vertraege extends FHCAPI_Controller } } } - $this->db->trans_complete(); $this->terminateWithSuccess(true); } @@ -250,21 +242,16 @@ class Vertraege extends FHCAPI_Controller $formData = $this->input->post('formData'); $lehrauftraege = $this->input->post('clickedRows'); - //Manually rewrite to postparameter for form validation - $_POST['vertragstyp_kurzbz'] = $formData['vertragstyp_kurzbz']; - $_POST['vertragsdatum']= $formData['vertragsdatum']; - $_POST['bezeichnung']= $formData['bezeichnung']; - $_POST['betrag'] = $formData['betrag']; - $_POST['vertragsstunden'] = - isset($formData['vertragsstunden']) - ? $formData['vertragsstunden'] - : null; - $_POST['vertragsstunden_studiensemester_kurzbz']= - isset($formData['vertragsstunden_studiensemester_kurzbz']) - ? $formData['vertragsstunden_studiensemester_kurzbz'] - : null; - $_POST['anmerkung']= isset($formData['anmerkung']) ? $formData['anmerkung'] : null; + $vertragstyp_kurzbz = $formData['vertragstyp_kurzbz'] ?? null; + $vertragsdatum = $formData['vertragsdatum'] ?? null; + $bezeichnung = $formData['bezeichnung'] ?? null; + $betrag = $formData['betrag'] ?? null; + $vertragsstunden = $formData['vertragsstunden'] ?? null; + $vertragsstunden_studiensemester_kurzbz = $formData['vertragsstunden_studiensemester_kurzbz'] ?? null; + $anmerkung = $formData['anmerkung'] ?? null; + + $this->form_validation->set_data($formData); $this->form_validation->set_rules('bezeichnung', 'Bezeichnung', 'required', [ 'required' => $this->p->t('ui', 'error_fieldRequired', ['field' => 'Bezeichnung']) ]); @@ -295,13 +282,13 @@ class Vertraege extends FHCAPI_Controller $vertrag_id, [ 'person_id' => $person_id, - 'vertragsdatum' => $this->input->post('vertragsdatum'), - 'bezeichnung' => $this->input->post('bezeichnung'), - 'vertragstyp_kurzbz' => $this->input->post('vertragstyp_kurzbz'), - 'betrag' => $this->input->post('betrag'), - 'vertragsstunden' => $this->input->post('vertragsstunden'), - 'vertragsstunden_studiensemester_kurzbz' => $this->input->post('vertragsstunden_studiensemester_kurzbz'), - 'anmerkung' => $this->input->post('anmerkung'), + 'vertragsdatum' => $vertragsdatum, + 'bezeichnung' => $bezeichnung, + 'vertragstyp_kurzbz' => $vertragstyp_kurzbz, + 'betrag' => $betrag, + 'vertragsstunden' => $vertragsstunden, + 'vertragsstunden_studiensemester_kurzbz' => $vertragsstunden_studiensemester_kurzbz, + 'anmerkung' => $anmerkung, 'updateamum' => date('c'), 'updatevon' => getAuthUID() ] diff --git a/public/js/apps/Vertragsverwaltung.js b/public/js/apps/Vertragsverwaltung.js index 1918345c0..3f8e47d3d 100644 --- a/public/js/apps/Vertragsverwaltung.js +++ b/public/js/apps/Vertragsverwaltung.js @@ -1,7 +1,5 @@ import Vertragsverwaltung from "../components/Vertraege/Vertragsverwaltung.js"; -import fhcapifactory from "./api/fhcapifactory.js"; - -import Phrasen from "../plugin/Phrasen.js"; +import Phrasen from "../plugins/Phrasen.js"; const ciPath = FHC_JS_DATA_STORAGE_OBJECT.app_root.replace(/(https:|)(^|\/\/)(.*?\/)/g, '') + FHC_JS_DATA_STORAGE_OBJECT.ci_router; @@ -12,12 +10,10 @@ const router = VueRouter.createRouter({ ] }); - const app = Vue.createApp(); app .use(router) - //.use(fhcapifactory) //nicht nötig .use(primevue.config.default, { zIndex: { overlay: 1100 diff --git a/public/js/components/DetailHeader/DetailHeader.js b/public/js/components/DetailHeader/DetailHeader.js index 8bed35beb..6f40c0972 100644 --- a/public/js/components/DetailHeader/DetailHeader.js +++ b/public/js/components/DetailHeader/DetailHeader.js @@ -203,6 +203,8 @@ export default { | DW {{headerDataMa?.telefonklappe}}

+ + ` } diff --git a/public/js/components/Vertraege/List/Status.js b/public/js/components/Vertraege/List/Status.js index 11410bfff..fd3b9afd7 100644 --- a/public/js/components/Vertraege/List/Status.js +++ b/public/js/components/Vertraege/List/Status.js @@ -282,7 +282,8 @@ export default {
diff --git a/public/js/components/Vertraege/MitarbeiterHeader.js b/public/js/components/Vertraege/MitarbeiterHeader.js index b49e069af..98f987a44 100644 --- a/public/js/components/Vertraege/MitarbeiterHeader.js +++ b/public/js/components/Vertraege/MitarbeiterHeader.js @@ -116,7 +116,7 @@ export default { this.$refs.table.tabulator.setFilter("aktiv", "=", true); } else { - this.$refs.table.tabulator.clearFilter("aktiv"); + this.$refs.table.tabulator.clearFilter(); } }, }, diff --git a/public/js/components/Vertraege/Vertragsverwaltung.js b/public/js/components/Vertraege/Vertragsverwaltung.js index 300b7a49e..5e27a6d77 100644 --- a/public/js/components/Vertraege/Vertragsverwaltung.js +++ b/public/js/components/Vertraege/Vertragsverwaltung.js @@ -27,7 +27,6 @@ export default { return { person_id: null, endpoint: ApiCoreVertraege - // } }, methods: {