From cd26667707c3380a8a00b46c4f6501f1f62d8cda Mon Sep 17 00:00:00 2001 From: ma0068 Date: Thu, 12 Jun 2025 18:40:01 +0200 Subject: [PATCH] refactor backend validierung --- .../api/frontend/v1/stv/Abschlusspruefung.php | 93 ++++++++----------- .../api/frontend/v1/stv/Aufnahmetermine.php | 56 +++++------ .../api/frontend/v1/stv/GemeinsameStudien.php | 32 ++++--- 3 files changed, 87 insertions(+), 94 deletions(-) diff --git a/application/controllers/api/frontend/v1/stv/Abschlusspruefung.php b/application/controllers/api/frontend/v1/stv/Abschlusspruefung.php index daaf043b5..d7581c3d1 100644 --- a/application/controllers/api/frontend/v1/stv/Abschlusspruefung.php +++ b/application/controllers/api/frontend/v1/stv/Abschlusspruefung.php @@ -224,19 +224,12 @@ class Abschlusspruefung extends FHCAPI_Controller $formData = $this->input->post('formData'); - $_POST['pruefungstyp_kurzbz'] = $formData['pruefungstyp_kurzbz']; - $_POST['akadgrad_id']= $formData['akadgrad_id']; - $_POST['vorsitz'] = isset($formData['vorsitz']['mitarbeiter_uid']) ? $formData['vorsitz']['mitarbeiter_uid'] : $formData['vorsitz']; - $_POST['pruefer1'] = isset($formData['pruefer1']['person_id']) ? $formData['pruefer1']['person_id'] : $formData['pruefer1']; - $_POST['pruefer2'] = isset($formData['pruefer2']['person_id']) ? $formData['pruefer2']['person_id'] : $formData['pruefer2']; - $_POST['pruefer3'] = isset($formData['pruefer3']['person_id']) ? $formData['pruefer3']['person_id'] : $formData['pruefer3']; - $_POST['pruefungsantritt_kurzbz'] = $formData['pruefungsantritt_kurzbz']; - $_POST['abschlussbeurteilung_kurzbz'] = $formData['abschlussbeurteilung_kurzbz']; - $_POST['datum']= $formData['datum']; - $_POST['sponsion']= $formData['sponsion']; - $_POST['anmerkung'] = $formData['anmerkung']; - $_POST['protokoll']= $formData['protokoll']; - $_POST['note'] = $formData['note']; + $vorsitz = isset($formData['vorsitz']['mitarbeiter_uid']) ? $formData['vorsitz']['mitarbeiter_uid'] : $formData['vorsitz']; + $pruefer1 = isset($formData['pruefer1']['person_id']) ? $formData['pruefer1']['person_id'] : $formData['pruefer1']; + $pruefer2 = isset($formData['pruefer2']['person_id']) ? $formData['pruefer2']['person_id'] : $formData['pruefer2']; + $pruefer3 = isset($formData['pruefer3']['person_id']) ? $formData['pruefer3']['person_id'] : $formData['pruefer3']; + + $this->form_validation->set_data($formData); $this->form_validation->set_rules('pruefungstyp_kurzbz', 'Typ', 'required', [ 'required' => $this->p->t('ui', 'error_fieldRequired', ['field' => 'Typ']) @@ -261,19 +254,19 @@ class Abschlusspruefung extends FHCAPI_Controller $result = $this->AbschlusspruefungModel->insert([ 'student_uid' => $student_uid, - 'pruefungstyp_kurzbz' => $this->input->post('pruefungstyp_kurzbz'), - 'akadgrad_id' => $this->input->post('akadgrad_id'), - 'vorsitz' => $this->input->post('vorsitz'), - 'pruefungsantritt_kurzbz' => $this->input->post('pruefungsantritt_kurzbz'), - 'abschlussbeurteilung_kurzbz' => $this->input->post('abschlussbeurteilung_kurzbz'), - 'datum' => $this->input->post('datum'), //TODO(Manu) check if minute format like FAS - 'sponsion' => $this->input->post('sponsion'), - 'pruefer1' => $this->input->post('pruefer1'), - 'pruefer2' => $this->input->post('pruefer2'), - 'pruefer3' => $this->input->post('pruefer3'), - 'protokoll' => $this->input->post('protokoll'), - 'note' => $this->input->post('note'), - 'anmerkung' => $this->input->post('anmerkung'), + 'pruefungstyp_kurzbz' => $formData['pruefungstyp_kurzbz'], + 'akadgrad_id' => $formData['akadgrad_id'], + 'vorsitz' => $vorsitz, + 'pruefungsantritt_kurzbz' => $formData['pruefungsantritt_kurzbz'], + 'abschlussbeurteilung_kurzbz' => $formData['abschlussbeurteilung_kurzbz'], + 'datum' => $formData['datum'], //TODO(Manu) check if minute format like FAS + 'sponsion' => $formData['sponsion'], + 'pruefer1' => $pruefer1, + 'pruefer2' => $pruefer2, + 'pruefer3' => $pruefer3, + 'protokoll' => $formData['protokoll'], + 'note' => $formData['note'], + 'anmerkung' => $formData['anmerkung'], 'insertamum' => date('c'), 'insertvon' => getAuthUID() ]); @@ -295,20 +288,12 @@ class Abschlusspruefung extends FHCAPI_Controller } $formData = $this->input->post('formData'); - $_POST['student_uid'] = $formData['student_uid']; - $_POST['pruefungstyp_kurzbz'] = $formData['pruefungstyp_kurzbz']; - $_POST['akadgrad_id']= $formData['akadgrad_id']; - $_POST['vorsitz'] = isset($formData['vorsitz']['mitarbeiter_uid']) ? $formData['vorsitz']['mitarbeiter_uid'] : $formData['vorsitz']; - $_POST['pruefer1'] = isset($formData['pruefer1']['person_id']) ? $formData['pruefer1']['person_id'] : $formData['pruefer1']; - $_POST['pruefer2'] = isset($formData['pruefer2']['person_id']) ? $formData['pruefer2']['person_id'] : $formData['pruefer2']; - $_POST['pruefer3'] = isset($formData['pruefer3']['person_id']) ? $formData['pruefer3']['person_id'] : $formData['pruefer3']; - $_POST['pruefungsantritt_kurzbz'] = $formData['pruefungsantritt_kurzbz']; - $_POST['abschlussbeurteilung_kurzbz'] = $formData['abschlussbeurteilung_kurzbz']; - $_POST['datum']= $formData['datum']; - $_POST['sponsion']= $formData['sponsion']; - $_POST['anmerkung'] = $formData['anmerkung']; - $_POST['protokoll']= $formData['protokoll']; - $_POST['note'] = $formData['note']; + $vorsitz = isset($formData['vorsitz']['mitarbeiter_uid']) ? $formData['vorsitz']['mitarbeiter_uid'] : $formData['vorsitz']; + $pruefer1 = isset($formData['pruefer1']['person_id']) ? $formData['pruefer1']['person_id'] : $formData['pruefer1']; + $pruefer2 = isset($formData['pruefer2']['person_id']) ? $formData['pruefer2']['person_id'] : $formData['pruefer2']; + $pruefer3 = isset($formData['pruefer3']['person_id']) ? $formData['pruefer3']['person_id'] : $formData['pruefer3']; + + $this->form_validation->set_data($formData); $this->form_validation->set_rules('pruefungstyp_kurzbz', 'Typ', 'required', [ 'required' => $this->p->t('ui', 'error_fieldRequired', ['field' => 'Typ']) @@ -337,20 +322,20 @@ class Abschlusspruefung extends FHCAPI_Controller 'abschlusspruefung_id' => $abschlusspruefung_id ], [ - 'student_uid' => $this->input->post('student_uid'), - 'pruefungstyp_kurzbz' => $this->input->post('pruefungstyp_kurzbz'), - 'akadgrad_id' => $this->input->post('akadgrad_id'), - 'vorsitz' => $this->input->post('vorsitz'), - 'pruefungsantritt_kurzbz' => $this->input->post('pruefungsantritt_kurzbz'), - 'abschlussbeurteilung_kurzbz' => $this->input->post('abschlussbeurteilung_kurzbz'), - 'datum' => $this->input->post('datum'), - 'sponsion' => $this->input->post('sponsion'), - 'pruefer1' => $this->input->post('pruefer1'), - 'pruefer2' => $this->input->post('pruefer2'), - 'pruefer3' => $this->input->post('pruefer3'), - 'protokoll' => $this->input->post('protokoll'), - 'note' => $this->input->post('note'), - 'anmerkung' => $this->input->post('anmerkung'), + 'student_uid' => $formData['student_uid'], + 'pruefungstyp_kurzbz' => $formData['pruefungstyp_kurzbz'], + 'akadgrad_id' => $formData['akadgrad_id'], + 'vorsitz' => $vorsitz, + 'pruefungsantritt_kurzbz' => $formData['pruefungsantritt_kurzbz'], + 'abschlussbeurteilung_kurzbz' => $formData['abschlussbeurteilung_kurzbz'], + 'datum' => $formData['datum'], + 'sponsion' => $formData['sponsion'], + 'pruefer1' => $pruefer1, + 'pruefer2' => $pruefer2, + 'pruefer3' => $pruefer3, + 'protokoll' => $formData['protokoll'], + 'note' => $formData['note'], + 'anmerkung' => $formData['anmerkung'], 'insertamum' => date('c'), 'insertvon' => getAuthUID() ] diff --git a/application/controllers/api/frontend/v1/stv/Aufnahmetermine.php b/application/controllers/api/frontend/v1/stv/Aufnahmetermine.php index 538626789..b787ea225 100644 --- a/application/controllers/api/frontend/v1/stv/Aufnahmetermine.php +++ b/application/controllers/api/frontend/v1/stv/Aufnahmetermine.php @@ -61,12 +61,13 @@ class Aufnahmetermine extends FHCAPI_Controller } + $rt_id = (isset($formData['rt_id']) && !empty($formData['rt_id'])) ? $formData['rt_id'] : null; + $anmeldedatum = (isset($formData['anmeldedatum']) && !empty($formData['anmeldedatum'])) ? $formData['anmeldedatum'] : null; + $teilgenommen = (isset($formData['teilgenommen']) && !empty($formData['teilgenommen'])) ? $formData['teilgenommen'] : false; + $studienplan_id = (isset($formData['studienplan_id']) && !empty($formData['studienplan_id'])) ? $formData['studienplan_id'] : null; + $punkte = (isset($formData['punkte']) && !empty($formData['punkte'])) ? $formData['punkte'] : null; - $_POST['rt_id'] = (isset($formData['rt_id']) && !empty($formData['rt_id'])) ? $formData['rt_id'] : null; - $_POST['anmeldedatum'] = (isset($formData['anmeldedatum']) && !empty($formData['anmeldedatum'])) ? $formData['anmeldedatum'] : null; - $_POST['teilgenommen'] = (isset($formData['teilgenommen']) && !empty($formData['teilgenommen'])) ? $formData['teilgenommen'] : false; - $_POST['studienplan_id'] = (isset($formData['studienplan_id']) && !empty($formData['studienplan_id'])) ? $formData['studienplan_id'] : null; - $_POST['punkte'] = (isset($formData['punkte']) && !empty($formData['punkte'])) ? $formData['punkte'] : null; + $this->form_validation->set_data($formData); $this->form_validation->set_rules('punkte', 'Punkte', 'numeric', [ 'required' => $this->p->t('ui', 'error_fieldNotNumeric', ['field' => 'Punkte']) @@ -90,11 +91,11 @@ class Aufnahmetermine extends FHCAPI_Controller $result = $this->RtPersonModel->insert([ 'person_id' => $person_id, - 'rt_id' => $_POST['rt_id'], - 'anmeldedatum' => $_POST['anmeldedatum'], - 'teilgenommen' => $_POST['teilgenommen'], - 'studienplan_id' => $_POST['studienplan_id'], - 'punkte' => $_POST['punkte'], + 'rt_id' => $rt_id, + 'anmeldedatum' => $anmeldedatum, + 'teilgenommen' => $teilgenommen, + 'studienplan_id' => $studienplan_id, + 'punkte' => $punkte, 'insertamum' => date('c'), 'insertvon' => $authUID, ]); @@ -133,12 +134,13 @@ class Aufnahmetermine extends FHCAPI_Controller return $this->terminateWithError($this->p->t('ui', 'error_missingId', ['id'=> 'RT_Person ID']), self::ERROR_TYPE_GENERAL); } + $rt_id = (isset($formData['rt_id']) && !empty($formData['rt_id'])) ? $formData['rt_id'] : null; + $anmeldedatum = (isset($formData['anmeldedatum']) && !empty($formData['anmeldedatum'])) ? $formData['anmeldedatum'] : null; + $teilgenommen = (isset($formData['teilgenommen']) && !empty($formData['teilgenommen'])) ? $formData['teilgenommen'] : false; + $studienplan_id = (isset($formData['studienplan_id']) && !empty($formData['studienplan_id'])) ? $formData['studienplan_id'] : null; + $punkte = (isset($formData['punkte']) && !empty($formData['punkte'])) ? $formData['punkte'] : null; - $_POST['rt_id'] = (isset($formData['rt_id']) && !empty($formData['rt_id'])) ? $formData['rt_id'] : null; - $_POST['anmeldedatum'] = (isset($formData['anmeldedatum']) && !empty($formData['anmeldedatum'])) ? $formData['anmeldedatum'] : null; - $_POST['teilgenommen'] = (isset($formData['teilgenommen']) && !empty($formData['teilgenommen'])) ? $formData['teilgenommen'] : false; - $_POST['studienplan_id'] = (isset($formData['studienplan_id']) && !empty($formData['studienplan_id'])) ? $formData['studienplan_id'] : null; - $_POST['punkte'] = (isset($formData['punkte']) && !empty($formData['punkte'])) ? $formData['punkte'] : null; + $this->form_validation->set_data($formData); $this->form_validation->set_rules('punkte', 'Punkte', 'numeric', [ 'required' => $this->p->t('ui', 'error_fieldNotNumeric', ['field' => 'Punkte']) @@ -166,11 +168,11 @@ class Aufnahmetermine extends FHCAPI_Controller 'person_id' => $person_id, ], [ - 'rt_id' => $_POST['rt_id'], - 'anmeldedatum' => $_POST['anmeldedatum'], - 'teilgenommen' => $_POST['teilgenommen'], - 'studienplan_id' => $_POST['studienplan_id'], - 'punkte' => $_POST['punkte'], + 'rt_id' => $rt_id, + 'anmeldedatum' => $anmeldedatum, + 'teilgenommen' => $teilgenommen, + 'studienplan_id' => $studienplan_id, + 'punkte' => $punkte, 'insertamum' => date('c'), 'insertvon' => $authUID, ] @@ -280,19 +282,21 @@ class Aufnahmetermine extends FHCAPI_Controller { return $this->terminateWithError($this->p->t('ui', 'error_missingId', ['id'=> 'Prestudent ID']), self::ERROR_TYPE_GENERAL); } - $_POST['rt_gesamtpunkte'] = + $rt_gesamtpunkte = (isset($formData['rt_gesamtpunkte']) && !empty($formData['rt_gesamtpunkte'])) ? $formData['rt_gesamtpunkte'] : null; - $_POST['reihungstestangetreten'] = + $reihungstestangetreten = (isset($formData['reihungstestangetreten']) && !empty($formData['reihungstestangetreten'])) ? $formData['reihungstestangetreten'] : null; - $_POST['aufnahmegruppe_kurzbz'] = + $aufnahmegruppe_kurzbz = (isset($formData['aufnahmegruppe_kurzbz']) && !empty($formData['aufnahmegruppe_kurzbz'])) ? $formData['aufnahmegruppe_kurzbz'] : null; + $this->form_validation->set_data($formData); + $this->form_validation->set_rules('rt_gesamtpunkte', 'Rt_gesamtpunkte', 'numeric', [ 'required' => $this->p->t('ui', 'error_fieldNotNumeric', ['field' => 'Rt_gesamtpunkte']) ]); @@ -309,9 +313,9 @@ class Aufnahmetermine extends FHCAPI_Controller 'prestudent_id' => $prestudent_id, ], [ - 'reihungstestangetreten' => $_POST['reihungstestangetreten'], - 'rt_gesamtpunkte' => $_POST['rt_gesamtpunkte'], - 'aufnahmegruppe_kurzbz' => $_POST['aufnahmegruppe_kurzbz'], + 'reihungstestangetreten' => $reihungstestangetreten, + 'rt_gesamtpunkte' => $rt_gesamtpunkte, + 'aufnahmegruppe_kurzbz' => $aufnahmegruppe_kurzbz, 'updateamum' => date('c'), 'updatevon' => $authUID, ] diff --git a/application/controllers/api/frontend/v1/stv/GemeinsameStudien.php b/application/controllers/api/frontend/v1/stv/GemeinsameStudien.php index dbf9efce1..97dad48fd 100644 --- a/application/controllers/api/frontend/v1/stv/GemeinsameStudien.php +++ b/application/controllers/api/frontend/v1/stv/GemeinsameStudien.php @@ -137,15 +137,17 @@ class GemeinsameStudien extends FHCAPI_Controller } $formData = $this->input->post('formData'); - $_POST['ausbildungssemester'] = (isset($formData['ausbildungssemester']) && !empty($formData['ausbildungssemester'])) + $ausbildungssemester = (isset($formData['ausbildungssemester']) && !empty($formData['ausbildungssemester'])) ? $formData['ausbildungssemester'] : null; - $_POST['mobilitaetstyp_kurzbz'] = (isset($formData['mobilitaetstyp_kurzbz']) && !empty($formData['mobilitaetstyp_kurzbz'])) + $mobilitaetstyp_kurzbz = (isset($formData['mobilitaetstyp_kurzbz']) && !empty($formData['mobilitaetstyp_kurzbz'])) ? $formData['mobilitaetstyp_kurzbz'] : null; - $_POST['studiensemester_kurzbz'] = (isset($formData['studiensemester_kurzbz']) && !empty($formData['studiensemester_kurzbz'])) + $studiensemester_kurzbz = (isset($formData['studiensemester_kurzbz']) && !empty($formData['studiensemester_kurzbz'])) ? $formData['studiensemester_kurzbz'] : null; + $this->form_validation->set_data($formData); + $this->form_validation->set_rules('mobilitaetstyp_kurzbz', 'Typ', 'required', [ 'required' => $this->p->t('ui', 'error_fieldRequired', ['field' => 'Typ']) ]); @@ -177,13 +179,13 @@ class GemeinsameStudien extends FHCAPI_Controller $result = $this->MobilitaetModel->insert([ 'prestudent_id' => $prestudent_id, - 'mobilitaetstyp_kurzbz' => $_POST['mobilitaetstyp_kurzbz'], + 'mobilitaetstyp_kurzbz' =>$mobilitaetstyp_kurzbz, 'status_kurzbz' => $status_kurzbz, - 'studiensemester_kurzbz' => $_POST['studiensemester_kurzbz'], + 'studiensemester_kurzbz' =>$studiensemester_kurzbz, 'mobilitaetsprogramm_code' => $mobilitaetsprogramm_code, 'gsprogramm_id' => $gsprogramm_id, 'firma_id' => $firma_id, - 'ausbildungssemester' => $_POST['ausbildungssemester'], + 'ausbildungssemester' =>$ausbildungssemester, 'insertvon' => $authUID, 'insertamum' => date('c'), ]); @@ -205,19 +207,21 @@ class GemeinsameStudien extends FHCAPI_Controller $formData = $this->input->post('formData'); - $_POST['mobilitaet_id'] = (isset($formData['mobilitaet_id']) && !empty($formData['mobilitaet_id'])) + $mobilitaet_id = (isset($formData['mobilitaet_id']) && !empty($formData['mobilitaet_id'])) ? $formData['mobilitaet_id'] : $this->terminateWithError($this->p->t('ui', 'error_missingId', ['id'=> 'Mobilitaet ID']), self::ERROR_TYPE_GENERAL); - $_POST['ausbildungssemester'] = (isset($formData['ausbildungssemester']) && !empty($formData['ausbildungssemester'])) + $ausbildungssemester = (isset($formData['ausbildungssemester']) && !empty($formData['ausbildungssemester'])) ? $formData['ausbildungssemester'] : null; - $_POST['mobilitaetstyp_kurzbz'] = (isset($formData['mobilitaetstyp_kurzbz']) && !empty($formData['mobilitaetstyp_kurzbz'])) + $mobilitaetstyp_kurzbz = (isset($formData['mobilitaetstyp_kurzbz']) && !empty($formData['mobilitaetstyp_kurzbz'])) ? $formData['mobilitaetstyp_kurzbz'] : null; - $_POST['studiensemester_kurzbz'] = (isset($formData['studiensemester_kurzbz']) && !empty($formData['studiensemester_kurzbz'])) + $studiensemester_kurzbz = (isset($formData['studiensemester_kurzbz']) && !empty($formData['studiensemester_kurzbz'])) ? $formData['studiensemester_kurzbz'] : null; + $this->form_validation->set_data($formData); + $this->form_validation->set_rules('mobilitaetstyp_kurzbz', 'Typ', 'required', [ 'required' => $this->p->t('ui', 'error_fieldRequired', ['field' => 'Typ']) ]); @@ -247,17 +251,17 @@ class GemeinsameStudien extends FHCAPI_Controller $result = $this->MobilitaetModel->update( [ - 'mobilitaet_id' => $_POST['mobilitaet_id'], + 'mobilitaet_id' => $mobilitaet_id, ], [ 'prestudent_id' => $prestudent_id, - 'mobilitaetstyp_kurzbz' => $_POST['mobilitaetstyp_kurzbz'], + 'mobilitaetstyp_kurzbz' => $mobilitaetstyp_kurzbz, 'status_kurzbz' => $status_kurzbz, - 'studiensemester_kurzbz' => $_POST['studiensemester_kurzbz'], + 'studiensemester_kurzbz' => $studiensemester_kurzbz, 'mobilitaetsprogramm_code' => $mobilitaetsprogramm_code, 'gsprogramm_id' => $gsprogramm_id, 'firma_id' => $firma_id, - 'ausbildungssemester' => $_POST['ausbildungssemester'], + 'ausbildungssemester' => $ausbildungssemester, 'updatevon' => $authUID, 'updateamum' => date('c'), ]