diff --git a/application/controllers/api/frontend/v1/vertraege/Vertraege.php b/application/controllers/api/frontend/v1/vertraege/Vertraege.php index 1b1022887..246de82e0 100644 --- a/application/controllers/api/frontend/v1/vertraege/Vertraege.php +++ b/application/controllers/api/frontend/v1/vertraege/Vertraege.php @@ -29,7 +29,6 @@ class Vertraege extends FHCAPI_Controller 'getHeader' => ['vertrag/mitarbeiter:r'], 'getPersonAbteilung' => ['vertrag/mitarbeiter:r'], 'getLeitungOrg' => ['vertrag/mitarbeiter:r'], - 'getMitarbeiterUid' => ['vertrag/mitarbeiter:r'], ]); //Load Models and Libraries @@ -684,11 +683,11 @@ class Vertraege extends FHCAPI_Controller return $this->terminateWithSuccess(getData($result)); } - public function getPersonAbteilung($person_id) + public function getPersonAbteilung($mitarbeiter_uid) { $this->load->model('ressource/Mitarbeiter_model', 'Mitarbeitermodel'); - $result = $this->Mitarbeitermodel->getPersonAbteilung($person_id); + $result = $this->Mitarbeitermodel->getPersonAbteilung($mitarbeiter_uid); $data = $this->getDataOrTerminateWithError($result); @@ -716,15 +715,4 @@ class Vertraege extends FHCAPI_Controller $this->terminateWithSuccess(current($data)); } - - - public function getMitarbeiterUid($person_id) - { - $this->load->model('ressource/Mitarbeiter_model', 'Mitarbeitermodel'); - - $result = $this->Mitarbeitermodel->getMitarbeiterUid($person_id); - - //no error here: ma_uid | null as result - $this->terminateWithSuccess($result); - } } diff --git a/application/models/ressource/Mitarbeiter_model.php b/application/models/ressource/Mitarbeiter_model.php index 0909c1095..ef86ac5fd 100644 --- a/application/models/ressource/Mitarbeiter_model.php +++ b/application/models/ressource/Mitarbeiter_model.php @@ -151,12 +151,8 @@ class Mitarbeiter_model extends DB_Model * * @return Array benutzerfunktionsdata */ - public function getPersonAbteilung($person_id) + public function getPersonAbteilung($uid) { - $uid = $this->getMitarbeiterUid($person_id); - if (!$uid) { - return null; - } $qry = " SELECT bf.benutzerfunktion_id, bf.fachbereich_kurzbz, bf.uid, bf.funktion_kurzbz, bf.updateamum, @@ -213,7 +209,7 @@ class Mitarbeiter_model extends DB_Model { $qry = " SELECT - * + titelpre, vorname, nachname, titelpost, foto, foto_sperre, person_id, alias, telefonklappe FROM public.tbl_person JOIN public.tbl_benutzer b USING(person_id) @@ -382,26 +378,6 @@ class Mitarbeiter_model extends DB_Model return $this->execQuery($qry); } - public function getMitarbeiterUid($person_id) - { - $qry = " - SELECT tbl_benutzer.uid - FROM tbl_mitarbeiter - JOIN tbl_benutzer ON CAST(tbl_mitarbeiter.mitarbeiter_uid AS TEXT) = CAST(tbl_benutzer.uid AS TEXT) - JOIN tbl_person USING (person_id) - WHERE tbl_person.person_id = ? - "; - $result = $this->execQuery($qry, [$person_id]); - - $data = getData($result); - if (empty($data)) { - return null; // No UID found - } - - $uid = isset($data[0]->uid) ? $data[0]->uid : null; - return $uid; - } - /** * Gets Mitarbeiter for a certain Lehrveranstaltung. * diff --git a/public/js/api/factory/detailHeader.js b/public/js/api/factory/detailHeader.js index f9f368bc7..74563bd4b 100644 --- a/public/js/api/factory/detailHeader.js +++ b/public/js/api/factory/detailHeader.js @@ -22,10 +22,10 @@ export default { url: 'api/frontend/v1/vertraege/vertraege/getHeader/' + person_id, }; }, - getPersonAbteilung(person_id){ + getPersonAbteilung(mitarbeiter_uid){ return { method: 'get', - url: 'api/frontend/v1/vertraege/vertraege/getPersonAbteilung/' + person_id, + url: 'api/frontend/v1/vertraege/vertraege/getPersonAbteilung/' + mitarbeiter_uid, }; }, getLeitungOrg(oekurzbz){ diff --git a/public/js/components/DetailHeader/DetailHeader.js b/public/js/components/DetailHeader/DetailHeader.js index 0b50c5153..528133b5a 100644 --- a/public/js/components/DetailHeader/DetailHeader.js +++ b/public/js/components/DetailHeader/DetailHeader.js @@ -42,7 +42,7 @@ export default { if(this.person_id) { this.getHeader(this.person_id); - this.loadDepartmentData(this.person_id) + this.loadDepartmentData(this.mitarbeiter_uid) .then(() => { // Call getLeitungOrg only after departmentData is loaded this.getLeitungOrg(this.departmentData.oe_kurzbz); @@ -83,9 +83,9 @@ export default { }) .catch(this.$fhcAlert.handleSystemError); }, - loadDepartmentData(person_id) { + loadDepartmentData(mitarbeiter_uid) { return this.$api - .call(ApiDetailHeader.getPersonAbteilung(person_id)) + .call(ApiDetailHeader.getPersonAbteilung(mitarbeiter_uid)) .then(result => { this.departmentData = result.data; }) @@ -106,7 +106,9 @@ export default { }, template: `