From 230ebcaf9aa23352bb46b59abc6692e92cc61cfb Mon Sep 17 00:00:00 2001 From: KarpAlex Date: Mon, 20 Mar 2023 17:01:58 +0100 Subject: [PATCH 01/16] added table public.tbl_kennzeichen and kennzeichentyp for managing person Ids --- system/dbupdate_3.4.php | 1 + .../21620_neues_feld_zum_erfassen_des_ESI.php | 116 ++++++++++++++++++ 2 files changed, 117 insertions(+) create mode 100644 system/dbupdate_3.4/21620_neues_feld_zum_erfassen_des_ESI.php diff --git a/system/dbupdate_3.4.php b/system/dbupdate_3.4.php index bc8152a90..dcf882c43 100644 --- a/system/dbupdate_3.4.php +++ b/system/dbupdate_3.4.php @@ -32,6 +32,7 @@ require_once('dbupdate_3.4/26173_index_webservicelog.php'); require_once('dbupdate_3.4/24682_reihungstest_zugangscode_fuer_login.php'); require_once('dbupdate_3.4/17512_fehlercode_constraints.php'); require_once('dbupdate_3.4/19154_beurteilungsformulare_pruefungssenat.php'); +require_once('dbupdate_3.4/21620_neues_feld_zum_erfassen_des_ESI.php'); // *** Pruefung und hinzufuegen der neuen Attribute und Tabellen echo '

Pruefe Tabellen und Attribute!

'; diff --git a/system/dbupdate_3.4/21620_neues_feld_zum_erfassen_des_ESI.php b/system/dbupdate_3.4/21620_neues_feld_zum_erfassen_des_ESI.php new file mode 100644 index 000000000..a7d415b68 --- /dev/null +++ b/system/dbupdate_3.4/21620_neues_feld_zum_erfassen_des_ESI.php @@ -0,0 +1,116 @@ +db_query('SELECT 0 FROM public.tbl_kennzeichentyp WHERE 0 = 1')) +{ + $qry = 'CREATE TABLE public.tbl_kennzeichentyp ( + kennzeichentyp_kurzbz varchar(32) NOT NULL, + bezeichnung varchar(256) NOT NULL, + aktiv boolean NOT NULL DEFAULT TRUE + ); + + COMMENT ON TABLE public.tbl_kennzeichentyp IS \'Tabelle zur Verwaltung von Typen von externen Personenkennzeichen.\'; + COMMENT ON COLUMN public.tbl_kennzeichentyp.bezeichnung IS \'Voller Name des Kennzeichentyps.\'; + COMMENT ON COLUMN public.tbl_kennzeichentyp.aktiv IS \'Ob der Kennzeichentyp noch aktiv und verwendet wird.\'; + + ALTER TABLE public.tbl_kennzeichentyp ADD CONSTRAINT pk_tbl_kennzeichentyp PRIMARY KEY (kennzeichentyp_kurzbz)'; + + if (!$db->db_query($qry)) + echo 'public.tbl_kennzeichentyp: '.$db->db_last_error().'
'; + else + echo '
public.tbl_kennzeichentyp table created'; + + $qry = 'GRANT SELECT ON TABLE public.tbl_kennzeichentyp TO web;'; + if (!$db->db_query($qry)) + echo 'public.tbl_kennzeichentyp: '.$db->db_last_error().'
'; + else + echo '
Granted privileges to web on public.tbl_kennzeichentyp'; + + $qry = 'GRANT SELECT, UPDATE, INSERT, DELETE ON TABLE public.tbl_kennzeichentyp TO vilesci;'; + if (!$db->db_query($qry)) + echo 'public.tbl_kennzeichentyp: '.$db->db_last_error().'
'; + else + echo '
Granted privileges to vilesci on public.tbl_kennzeichentyp'; +} + +// SEQUENCE tbl_kennzeichen_id_seq +if ($result = $db->db_query("SELECT 0 FROM pg_class WHERE relname = 'tbl_kennzeichen_id_seq'")) +{ + if ($db->db_num_rows($result) == 0) + { + $qry = 'CREATE SEQUENCE public.tbl_kennzeichen_id_seq + START WITH 1 + INCREMENT BY 1 + NO MAXVALUE + NO MINVALUE + CACHE 1;'; + + if(!$db->db_query($qry)) + echo 'public.tbl_kennzeichen_id_seq '.$db->db_last_error().'
'; + else + echo '
Created sequence: public.tbl_kennzeichen_id_seq'; + + // GRANT SELECT, UPDATE ON SEQUENCE public.tbl_kennzeichen_id_seq TO vilesci; + $qry = 'GRANT SELECT, UPDATE ON SEQUENCE public.tbl_kennzeichen_id_seq TO vilesci;'; + if (!$db->db_query($qry)) + echo 'public.tbl_kennzeichen_id_seq '.$db->db_last_error().'
'; + else + echo '
Granted privileges to vilesci on public.tbl_kennzeichen_id_seq'; + + // GRANT SELECT, UPDATE ON SEQUENCE public.tbl_kennzeichen_id_seq TO fhcomplete; + $qry = 'GRANT SELECT, UPDATE ON SEQUENCE public.tbl_kennzeichen_id_seq TO fhcomplete;'; + if (!$db->db_query($qry)) + echo 'public.tbl_kennzeichen_id_seq '.$db->db_last_error().'
'; + else + echo '
Granted privileges to fhcomplete on public.tbl_kennzeichen_id_seq'; + } +} + +// Creates table public.tbl_kennzeichen if it doesn't exist and grants privileges +if (!$result = @$db->db_query('SELECT 0 FROM public.tbl_kennzeichen WHERE 0 = 1')) +{ + $qry = 'CREATE TABLE public.tbl_kennzeichen ( + kennzeichen_id integer NOT NULL DEFAULT NEXTVAL(\'public.tbl_kennzeichen_id_seq\'), + person_id integer NOT NULL, + kennzeichentyp_kurzbz varchar(32) NOT NULL, + inhalt text NOT NULL, + aktiv boolean NOT NULL DEFAULT TRUE, + insertamum timestamp DEFAULT NOW(), + insertvon varchar(32), + updateamum timestamp, + updatevon varchar(32) + ); + + COMMENT ON TABLE public.tbl_kennzeichen IS \'Tabelle zum Speichern von externen Personenkennzeichen.\'; + COMMENT ON COLUMN public.tbl_kennzeichen.kennzeichentyp_kurzbz IS \'Typ des externen Personen Kennzeichens.\'; + COMMENT ON COLUMN public.tbl_kennzeichen.inhalt IS \'Das externe Kennzeichen.\'; + COMMENT ON COLUMN public.tbl_kennzeichen.aktiv IS \'Ob das Kennzeichen noch aktiv ist und verwendet wird.\'; + + ALTER TABLE public.tbl_kennzeichen ADD CONSTRAINT pk_tbl_kennzeichen PRIMARY KEY (kennzeichen_id); + + ALTER TABLE public.tbl_kennzeichen ADD CONSTRAINT fk_kennzeichen_person FOREIGN KEY (person_id) REFERENCES public.tbl_person(person_id) ON UPDATE CASCADE ON DELETE RESTRICT; + ALTER TABLE public.tbl_kennzeichen ADD CONSTRAINT fk_kennzeichen_kennzeichentyp_kurzbz FOREIGN KEY (kennzeichentyp_kurzbz) REFERENCES public.tbl_kennzeichentyp(kennzeichentyp_kurzbz) ON UPDATE CASCADE ON DELETE RESTRICT; + + -- create unique constraint, no person can have the same kennzeichen twice + ALTER TABLE public.tbl_kennzeichen ADD CONSTRAINT uk_kennzeichen_person_id_inhalt UNIQUE (person_id, kennzeichentyp_kurzbz, inhalt); + -- create unique index - person can only have one active kennzeichen of each type + CREATE UNIQUE INDEX kennzeichen_aktiv_constraint ON public.tbl_kennzeichen (person_id, kennzeichentyp_kurzbz) WHERE aktiv;'; + + if (!$db->db_query($qry)) + echo 'public.tbl_kennzeichen: '.$db->db_last_error().'
'; + else + echo '
public.tbl_kennzeichen table created'; + + $qry = 'GRANT SELECT, UPDATE, INSERT, DELETE ON TABLE public.tbl_kennzeichen TO web;'; + if (!$db->db_query($qry)) + echo 'public.tbl_kennzeichen: '.$db->db_last_error().'
'; + else + echo '
Granted privileges to web on public.tbl_kennzeichen'; + + $qry = 'GRANT SELECT, UPDATE, INSERT, DELETE ON TABLE public.tbl_kennzeichen TO vilesci;'; + if (!$db->db_query($qry)) + echo 'public.tbl_kennzeichen: '.$db->db_last_error().'
'; + else + echo '
Granted privileges to vilesci on public.tbl_kennzeichen'; +} From cd6443d9c05a67c65d5b31623d1c1e726adefeb4 Mon Sep 17 00:00:00 2001 From: KarpAlex Date: Wed, 22 Mar 2023 17:41:15 +0100 Subject: [PATCH 02/16] -added scheduler for generating and saving ESI -first version of generateESI job --- application/controllers/jobs/ESIJob.php | 169 ++++++++++++++++++ .../jobs/schedulers/ESIScheduler.php | 108 +++++++++++ .../models/person/Kennzeichen_model.php | 36 ++++ .../21620_neues_feld_zum_erfassen_des_ESI.php | 34 +++- 4 files changed, 345 insertions(+), 2 deletions(-) create mode 100644 application/controllers/jobs/ESIJob.php create mode 100644 application/controllers/jobs/schedulers/ESIScheduler.php create mode 100644 application/models/person/Kennzeichen_model.php diff --git a/application/controllers/jobs/ESIJob.php b/application/controllers/jobs/ESIJob.php new file mode 100644 index 000000000..74c5a925d --- /dev/null +++ b/application/controllers/jobs/ESIJob.php @@ -0,0 +1,169 @@ +load->model('Person_model', 'PersonModel'); + $this->load->model('Kennzeichen_model', 'KennzeichenModel'); + + // load libraries + //$this->load->library('extensions/FHC-Core-DVUH/DVUHIssueLib'); + } + + //------------------------------------------------------------------------------------------------------------------ + // Public methods + + /** + * Initialises generateESI job, handles job queue, logs infos/errors + */ + public function generateESI() + { + //$jobType = 'DVUHSendPruefungsaktivitaeten'; + $this->logInfo(ESIScheduler::JOB_TYPE_GENERATE_ESI.' job start'); + + // Gets the latest jobs + $lastJobs = $this->getLastJobs(ESIScheduler::JOB_TYPE_GENERATE_ESI); + if (isError($lastJobs)) + { + $this->logError(getCode($lastJobs).': '.getError($lastJobs), ESIScheduler::JOB_TYPE_GENERATE_ESI); + } + else + { + $this->updateJobs( + getData($lastJobs), // Jobs to be updated + array(JobsQueueLib::PROPERTY_START_TIME), // Job properties to be updated + array(date('Y-m-d H:i:s')) // Job properties new values + ); + + $person_arr = $this->_getInputObjArray(getData($lastJobs)); + + foreach ($person_arr as $persobj) + { + if (!isset($persobj->person_id)) + $this->logError("Error when generating ESI: invalid parameters"); + else + { + $person_id = $persobj->person_id; + + // check if there already is an active ESI + $this->KennzeichenModel->addSelect('1'); + $activeKennzeichenRes = $this->KennzeichenModel->loadWhere( + array('person_id' => $person_id, 'kennzeichentyp_kurzbz' => ESIScheduler::KENNZEICHENTYP_KURZBZ, 'aktiv' => true) + ); + + if (hasData($activeKennzeichenRes)) + { + $this->logError("Active ESI for person Id $prestudent_id already exists"); + continue; + } + + // get Matrikelnr for person for which ESI should be generated + $this->PersonModel->addSelect('matr_nr'); + $personRes = $this->PersonModel->load($person_id); + + if (!hasData($personRes)) + { + $this->logError("Person with Id $person_id not found"); + continue; + } + + $matr_nr = getData($personRes)[0]->matr_nr; + + if (isEmptyString($matr_nr)) + { + $this->logError("Matrikelnummer for person with Id $person_id is empty"); + continue; + } + + $esi = self::ESI_PREFIX.$matrikelnr; + + // check if ESI was already used + $this->KennzeichenModel->addSelect('1'); + $existingKennzeichenRes = $this->KennzeichenModel->loadWhere( + array('person_id' => $person_id, 'kennzeichentyp_kurzbz' => ESIScheduler::KENNZEICHENTYP_KURZBZ, 'inhalt' => $esi) + ); + + if (hasData($existingKennzeichenRes)) + { + $this->logError("ESI $esi for person Id $prestudent_id already exists"); + continue; + } + + // if everything ok, save the esi for the person + $saveEsiResult = $this->Kennzeichen_model->insert( + array( + 'person_id' => $person_id, + 'kennzeichentyp_kurzbz' => ESIScheduler::KENNZEICHENTYP_KURZBZ, + 'inhalt' => $esi, + 'aktiv' => true + ) + ); + + if (isError($saveEsiResult)) + { + $this->logError("Error when sending ESI, person Id $person_id ".getError($saveEsiResult)); + } + elseif (hasData($saveEsiResult)) + { + // TODO everything ok + } + } + } + + // Update jobs properties values + $this->updateJobs( + getData($lastJobs), // Jobs to be updated + array(JobsQueueLib::PROPERTY_STATUS, JobsQueueLib::PROPERTY_END_TIME), // Job properties to be updated + array(JobsQueueLib::STATUS_DONE, date('Y-m-d H:i:s')) // Job properties new values + ); + + if (hasData($lastJobs)) $this->updateJobsQueue($jobType, getData($lastJobs)); + } + + $this->logInfo(ESIScheduler::JOB_TYPE_GENERATE_ESI.' job stop'); + } + + // -------------------------------------------------------------------------------------------- + // Private methods + + /** + * Extracts input data from jobs. + * @param $jobs + * @return array with jobinput + */ + private function _getInputObjArray($jobs) + { + $mergedUsersArray = array(); + + if (count($jobs) == 0) return $mergedUsersArray; + + foreach ($jobs as $job) + { + $decodedInput = json_decode($job->input); + if ($decodedInput != null) + { + foreach ($decodedInput as $el) + { + $mergedUsersArray[] = $el; + } + } + } + return $mergedUsersArray; + } +} diff --git a/application/controllers/jobs/schedulers/ESIScheduler.php b/application/controllers/jobs/schedulers/ESIScheduler.php new file mode 100644 index 000000000..3ab858937 --- /dev/null +++ b/application/controllers/jobs/schedulers/ESIScheduler.php @@ -0,0 +1,108 @@ +load->model('organisation/Studiensemester_model', 'StudiensemesterModel'); + } + + //------------------------------------------------------------------------------------------------------------------ + // Public methods + + /** + * Creates jobs queue entries for generateESI job. + * @param string $studiensemester_kurzbz semester for which ESIs should be generated + */ + public function generateESI($studiensemester_kurzbz = null) + { + // if no semester given, get current studiensemester + if (!isset($studiensemester_kurzbz)) + { + $semRes = $this->StudiensemesterModel->getAkt(); + + if (hasData($semRes)) + { + $studiensemester_kurzbz = getData($semRes)[0]->studiensemester_kurzbz; + } + } + + if (isset($studiensemester_kurzbz)) + { + $this->logInfo('Start job queue scheduler '.self::JOB_TYPE_GENERATE_ESI); + + $qry = " + SELECT + DISTINCT person_id + FROM + public.tbl_person pers + JOIN public.tbl_prestudent ps USING (person_id) + JOIN public.tbl_prestudentstatus pss USING (prestudent_id) + WHERE + pss.studiensemester_kurzbz = ? + AND pers.matr_nr IS NOT NULL + AND pss.status_kurzbz IN ? + AND NOT EXISTS ( -- has no ESI yet + SELECT 1 + FROM + public.tbl_kennzeichen + WHERE + person_id = pers.person_id + AND kennzeichentyp_kurzbz = ? + AND aktiv + ) + AND NOT EXISTS ( -- making sure it's not an incoming + SELECT 1 + FROM + public.tbl_prestudentstatus + WHERE + prestudent_id = ps.prestudent_id + AND status_kurzbz = 'Incoming' + )"; + + $db = new DB_Model(); + $jobInputResult = $db->execReadOnlyQuery($qry, array($studiensemester_kurzbz, $this->_active_status_kurzbz, self::KENNZEICHENTYP_KURZBZ)); + + // If an error occured then log it + if (isError($jobInputResult)) + { + $this->logError(getError($jobInputResult)); + } + elseif (hasData($jobInputResult)) // if persons found + { + // Add the new job to the jobs queue + $addNewJobResult = $this->addNewJobsToQueue( + self::JOB_TYPE_GENERATE_ESI, // job type + $this->generateJobs( // gnerate the structure of the new job + JobsQueueLib::STATUS_NEW, + json_encode(getData($jobInputResult)) + ) + ); + + // If error occurred return it + if (isError($addNewJobResult)) $this->logError(getError($addNewJobResult)); + } + } + else + { + $this->logError('Error when getting Studiensemester'); + } + + $this->logInfo('End job queue scheduler '.self::JOB_TYPE_GENERATE_ESI); + } +} diff --git a/application/models/person/Kennzeichen_model.php b/application/models/person/Kennzeichen_model.php new file mode 100644 index 000000000..7d00a68b4 --- /dev/null +++ b/application/models/person/Kennzeichen_model.php @@ -0,0 +1,36 @@ +dbTable = 'public.tbl_kennzeichen'; + $this->pk = 'kennzeichen_id'; + } + + /** + * + * @param + * @return object success or error + */ + public function _() + { + + } + + /** + * Get Zustelladress of given person. + * @param string $person_id + * @param string $select + * @return array + */ + //~ public function getZustellAdresse($person_id, $select = '*') + //~ { + //~ $this->addSelect($select); + //~ return $this->loadWhere(array('person_id' => $person_id, 'zustelladresse'=> true)); + //~ } +} diff --git a/system/dbupdate_3.4/21620_neues_feld_zum_erfassen_des_ESI.php b/system/dbupdate_3.4/21620_neues_feld_zum_erfassen_des_ESI.php index a7d415b68..5b6b22918 100644 --- a/system/dbupdate_3.4/21620_neues_feld_zum_erfassen_des_ESI.php +++ b/system/dbupdate_3.4/21620_neues_feld_zum_erfassen_des_ESI.php @@ -89,8 +89,10 @@ if (!$result = @$db->db_query('SELECT 0 FROM public.tbl_kennzeichen WHERE 0 = 1' ALTER TABLE public.tbl_kennzeichen ADD CONSTRAINT pk_tbl_kennzeichen PRIMARY KEY (kennzeichen_id); - ALTER TABLE public.tbl_kennzeichen ADD CONSTRAINT fk_kennzeichen_person FOREIGN KEY (person_id) REFERENCES public.tbl_person(person_id) ON UPDATE CASCADE ON DELETE RESTRICT; - ALTER TABLE public.tbl_kennzeichen ADD CONSTRAINT fk_kennzeichen_kennzeichentyp_kurzbz FOREIGN KEY (kennzeichentyp_kurzbz) REFERENCES public.tbl_kennzeichentyp(kennzeichentyp_kurzbz) ON UPDATE CASCADE ON DELETE RESTRICT; + ALTER TABLE public.tbl_kennzeichen ADD CONSTRAINT fk_kennzeichen_person FOREIGN KEY (person_id) + REFERENCES public.tbl_person(person_id) ON UPDATE CASCADE ON DELETE RESTRICT; + ALTER TABLE public.tbl_kennzeichen ADD CONSTRAINT fk_kennzeichen_kennzeichentyp_kurzbz FOREIGN KEY (kennzeichentyp_kurzbz) + REFERENCES public.tbl_kennzeichentyp(kennzeichentyp_kurzbz) ON UPDATE CASCADE ON DELETE RESTRICT; -- create unique constraint, no person can have the same kennzeichen twice ALTER TABLE public.tbl_kennzeichen ADD CONSTRAINT uk_kennzeichen_person_id_inhalt UNIQUE (person_id, kennzeichentyp_kurzbz, inhalt); @@ -114,3 +116,31 @@ if (!$result = @$db->db_query('SELECT 0 FROM public.tbl_kennzeichen WHERE 0 = 1' else echo '
Granted privileges to vilesci on public.tbl_kennzeichen'; } + +// public.tbl_kennzeichentyp: add type esi +if ($result = $db->db_query("SELECT 1 FROM public.tbl_kennzeichentyp WHERE kennzeichentyp_kurzbz='esi'")) +{ + if($db->db_num_rows($result)==0) + { + $qry = "INSERT INTO public.tbl_kennzeichentyp(kennzeichentyp_kurzbz, bezeichnung) VALUES('esi', 'European Student Identifier');"; + + if(!$db->db_query($qry)) + echo 'Kennzeichentyp: '.$db->db_last_error().'
'; + else + echo '
Neuer Kennzeichentyp esi in public.tbl_kennzeichentyp hinzugefügt'; + } +} + +// system.tbl_jobtypes: add type esi +if ($result = $db->db_query("SELECT 1 FROM system.tbl_jobtypes WHERE type='generateESI'")) +{ + if($db->db_num_rows($result)==0) + { + $qry = "INSERT INTO system.tbl_jobtypes(type, description) VALUES('generateESI', 'Generate and save European Student Identifier');"; + + if(!$db->db_query($qry)) + echo 'Jobtyp: '.$db->db_last_error().'
'; + else + echo '
Neuer Jobtyp generateESI in system.tbl_jobtypes hinzugefügt'; + } +} From 903f3d2f378537d0f94be9dcd05d4beb8bb57e66 Mon Sep 17 00:00:00 2001 From: KarpAlex Date: Thu, 23 Mar 2023 17:50:32 +0100 Subject: [PATCH 03/16] generateESI Job bugfixes (correct models, Ids in error logs), added insertvon --- application/controllers/jobs/ESIJob.php | 26 +++++++++++-------------- 1 file changed, 11 insertions(+), 15 deletions(-) diff --git a/application/controllers/jobs/ESIJob.php b/application/controllers/jobs/ESIJob.php index 74c5a925d..c9a558a46 100644 --- a/application/controllers/jobs/ESIJob.php +++ b/application/controllers/jobs/ESIJob.php @@ -10,6 +10,7 @@ require_once('schedulers/ESIScheduler.php'); class ESIJob extends JQW_Controller { const ESI_PREFIX = 'urn:schac:personalUniqueCode:int:esi:at:'; + const INSERT_VON = 'generateEsiJob'; /** * Controller initialization @@ -19,11 +20,8 @@ class ESIJob extends JQW_Controller parent::__construct(); // load models - $this->load->model('Person_model', 'PersonModel'); - $this->load->model('Kennzeichen_model', 'KennzeichenModel'); - - // load libraries - //$this->load->library('extensions/FHC-Core-DVUH/DVUHIssueLib'); + $this->load->model('person/Person_model', 'PersonModel'); + $this->load->model('person/Kennzeichen_model', 'KennzeichenModel'); } //------------------------------------------------------------------------------------------------------------------ @@ -39,6 +37,7 @@ class ESIJob extends JQW_Controller // Gets the latest jobs $lastJobs = $this->getLastJobs(ESIScheduler::JOB_TYPE_GENERATE_ESI); + if (isError($lastJobs)) { $this->logError(getCode($lastJobs).': '.getError($lastJobs), ESIScheduler::JOB_TYPE_GENERATE_ESI); @@ -69,7 +68,7 @@ class ESIJob extends JQW_Controller if (hasData($activeKennzeichenRes)) { - $this->logError("Active ESI for person Id $prestudent_id already exists"); + $this->logError("Active ESI for person Id $person_id already exists"); continue; } @@ -91,7 +90,7 @@ class ESIJob extends JQW_Controller continue; } - $esi = self::ESI_PREFIX.$matrikelnr; + $esi = self::ESI_PREFIX.$matr_nr; // check if ESI was already used $this->KennzeichenModel->addSelect('1'); @@ -101,17 +100,18 @@ class ESIJob extends JQW_Controller if (hasData($existingKennzeichenRes)) { - $this->logError("ESI $esi for person Id $prestudent_id already exists"); + $this->logError("ESI $esi for person Id $person_id already exists"); continue; } // if everything ok, save the esi for the person - $saveEsiResult = $this->Kennzeichen_model->insert( + $saveEsiResult = $this->KennzeichenModel->insert( array( 'person_id' => $person_id, 'kennzeichentyp_kurzbz' => ESIScheduler::KENNZEICHENTYP_KURZBZ, 'inhalt' => $esi, - 'aktiv' => true + 'aktiv' => true, + 'insertvon' => self::INSERT_VON ) ); @@ -119,10 +119,6 @@ class ESIJob extends JQW_Controller { $this->logError("Error when sending ESI, person Id $person_id ".getError($saveEsiResult)); } - elseif (hasData($saveEsiResult)) - { - // TODO everything ok - } } } @@ -133,7 +129,7 @@ class ESIJob extends JQW_Controller array(JobsQueueLib::STATUS_DONE, date('Y-m-d H:i:s')) // Job properties new values ); - if (hasData($lastJobs)) $this->updateJobsQueue($jobType, getData($lastJobs)); + if (hasData($lastJobs)) $this->updateJobsQueue(ESIScheduler::JOB_TYPE_GENERATE_ESI, getData($lastJobs)); } $this->logInfo(ESIScheduler::JOB_TYPE_GENERATE_ESI.' job stop'); From d9eb0f5704732ce05549ffeb4e24a149ca3c5a99 Mon Sep 17 00:00:00 2001 From: ma0048 Date: Fri, 16 Jun 2023 11:30:03 +0200 Subject: [PATCH 04/16] - kennzeichen beim zusammenlegen uebernehmen --- .../models/person/Kennzeichen_model.php | 21 ----- vilesci/stammdaten/personen_wartung.php | 84 +++++++++++++++++++ 2 files changed, 84 insertions(+), 21 deletions(-) diff --git a/application/models/person/Kennzeichen_model.php b/application/models/person/Kennzeichen_model.php index 7d00a68b4..fe8a9ac62 100644 --- a/application/models/person/Kennzeichen_model.php +++ b/application/models/person/Kennzeichen_model.php @@ -12,25 +12,4 @@ class Kennzeichen_model extends DB_Model $this->pk = 'kennzeichen_id'; } - /** - * - * @param - * @return object success or error - */ - public function _() - { - - } - - /** - * Get Zustelladress of given person. - * @param string $person_id - * @param string $select - * @return array - */ - //~ public function getZustellAdresse($person_id, $select = '*') - //~ { - //~ $this->addSelect($select); - //~ return $this->loadWhere(array('person_id' => $person_id, 'zustelladresse'=> true)); - //~ } } diff --git a/vilesci/stammdaten/personen_wartung.php b/vilesci/stammdaten/personen_wartung.php index ee95854ed..c77b81b2b 100644 --- a/vilesci/stammdaten/personen_wartung.php +++ b/vilesci/stammdaten/personen_wartung.php @@ -319,6 +319,90 @@ if (isset($personToDelete) && isset($personToKeep) && $personToDelete >= 0 && $p } } } + + if($result = @$db->db_query("SELECT 1 FROM public.tbl_kennzeichen LIMIT 1")) + { + $kennzeichen_has_personToKeep = array(); + $kennzeichen_has_personToDelete = array(); + + $kennzeichen_query = " + SELECT * + FROM public.tbl_kennzeichen + WHERE ( + person_id = " . $db->db_add_param($personToKeep, FHC_INTEGER) . " OR + person_id = " . $db->db_add_param($personToDelete, FHC_INTEGER) . " + ) + ORDER BY kennzeichentyp_kurzbz, aktiv DESC"; + + if ($result = $db->db_query($kennzeichen_query)) + { + while ($row = $db->db_fetch_object($result)) + { + if ($row->person_id === $personToKeep) + { + $kennzeichen_has_personToKeep[] = $row; + } + else if ($row->person_id === $personToDelete) + { + $kennzeichen_has_personToDelete[] = $row; + } + } + } + + if (!empty($kennzeichen_has_personToDelete)) + { + foreach ($kennzeichen_has_personToDelete as $kennzeichen_toDelete) + { + $kennzeichen_toKeep_Kurzbz = array_column($kennzeichen_has_personToKeep, 'kennzeichentyp_kurzbz'); + + if (in_array($kennzeichen_toDelete->kennzeichentyp_kurzbz, $kennzeichen_toKeep_Kurzbz)) + { + $kennzeichen_toKeep_Keys = array_keys($kennzeichen_toKeep_Kurzbz, $kennzeichen_toDelete->kennzeichentyp_kurzbz); + + foreach ($kennzeichen_toKeep_Keys as $key) + { + if (($kennzeichen_has_personToKeep[$key]->aktiv === 't' && $kennzeichen_toDelete->aktiv === 'f') || + ($kennzeichen_has_personToKeep[$key]->aktiv === 'f' && $kennzeichen_toDelete->aktiv === 't') || + ($kennzeichen_has_personToKeep[$key]->aktiv === 'f' && $kennzeichen_toDelete->aktiv === 'f')) + { + if ($kennzeichen_has_personToKeep[$key]->inhalt !== $kennzeichen_toDelete->inhalt) + { + $sql_query_upd1 .= "UPDATE public.tbl_kennzeichen SET person_id=" . $db->db_add_param($personToKeep, FHC_INTEGER) . " WHERE kennzeichen_id=" . $db->db_add_param($kennzeichen_toDelete->kennzeichen_id, FHC_INTEGER) . ";"; + $kennzeichen_has_personToKeep[] = $kennzeichen_toDelete; + continue 2; + } + else + { + if ($kennzeichen_toDelete->aktiv === 'f') + { + $sql_query_upd1 .= "DELETE FROM public.tbl_kennzeichen WHERE kennzeichen_id=" . $db->db_add_param($kennzeichen_toDelete->kennzeichen_id, FHC_INTEGER) . ";"; + $msg_warning[] = "Das nicht aktive Kennzeichen mit der ID '" . $kennzeichen_toDelete->kennzeichen_id . "' wurde gelöscht,
+ da es der gleiche Inhalt wie beim Kennzeichen mit der ID '". $kennzeichen_has_personToKeep[$key]->kennzeichen_id ."' ist."; + continue 2; + } + $msg_error[] = 'Beide Personen haben ein Kennzeichen mit dem gleichen Typ ('. $kennzeichen_toDelete->kennzeichentyp_kurzbz.') und den gleichen Inhalt. Können nicht zusammengelegt werden.
+ Sie müssen die Datensätze manuell bereinigen, bevor Sie die Personen zusammenlegen können.'; + $error = true; + break 2; + } + } + else if ($kennzeichen_has_personToKeep[$key]->aktiv === 't' && $kennzeichen_toDelete->aktiv === 't') + { + $msg_error[] = 'Beide Personen haben ein aktives Kennzeichen mit dem gleichen Typ ('. $kennzeichen_toDelete->kennzeichentyp_kurzbz.'). Können nicht zusammengelegt werden.
+ Sie müssen die Datensätze manuell bereinigen, bevor Sie die Personen zusammenlegen können.'; + $error = true; + break 2; + } + } + } + else + { + $sql_query_upd1 .= "UPDATE public.tbl_kennzeichen SET person_id=" . $db->db_add_param($personToKeep, FHC_INTEGER) . " WHERE kennzeichen_id=" . $db->db_add_param($kennzeichen_toDelete->kennzeichen_id, FHC_INTEGER) . ";"; + } + } + } + } + if ($error == false) { // Wenn bei einer der Personen das Foto gesperrt ist, dann die Sperre uebernehmen From e60157dd9beb45f7d7441f445470459cd9734934 Mon Sep 17 00:00:00 2001 From: KarpAlex Date: Mon, 18 Dec 2023 17:16:20 +0100 Subject: [PATCH 05/16] Personen zusammenlegen: uhstat1daten are checked --- vilesci/stammdaten/personen_wartung.php | 58 +++++++++++++++++++++++-- 1 file changed, 55 insertions(+), 3 deletions(-) diff --git a/vilesci/stammdaten/personen_wartung.php b/vilesci/stammdaten/personen_wartung.php index c77b81b2b..99f83c454 100644 --- a/vilesci/stammdaten/personen_wartung.php +++ b/vilesci/stammdaten/personen_wartung.php @@ -229,7 +229,7 @@ if (isset($personToDelete) && isset($personToKeep) && $personToDelete >= 0 && $p die('Es sind bereits beide Personen in SAP vorhanden. Bitte zuerst direkt in der tbl_sap_students lösen.'); } } - + $personToDelete_obj = new person(); if ($personToDelete_obj->load($personToDelete)) { @@ -333,7 +333,7 @@ if (isset($personToDelete) && isset($personToKeep) && $personToDelete >= 0 && $p person_id = " . $db->db_add_param($personToDelete, FHC_INTEGER) . " ) ORDER BY kennzeichentyp_kurzbz, aktiv DESC"; - + if ($result = $db->db_query($kennzeichen_query)) { while ($row = $db->db_fetch_object($result)) @@ -403,6 +403,58 @@ if (isset($personToDelete) && isset($personToKeep) && $personToDelete >= 0 && $p } } + if($result = @$db->db_query("SELECT 1 FROM bis.tbl_uhstat1daten LIMIT 1")) + { + $uhstat_has_personToKeep = array(); + $uhstat_has_personToDelete = array(); + + $uhstat_query = " + SELECT uhstat1daten_id, person_id + FROM bis.tbl_uhstat1daten + WHERE ( + person_id = " . $db->db_add_param($personToKeep, FHC_INTEGER) . " OR + person_id = " . $db->db_add_param($personToDelete, FHC_INTEGER) . " + ) + ORDER BY updateamum DESC NULLS LAST, insertamum DESC NULLS LAST"; + + // Herausfinden, ob UHSTAT Daten der löschenden oder zu belassenden Person zugeordnet sind + if ($result = $db->db_query($uhstat_query)) + { + while ($row = $db->db_fetch_object($result)) + { + if ($row->person_id === $personToKeep) + { + $uhstat_has_personToKeep[] = $row; + } + else if ($row->person_id === $personToDelete) + { + $uhstat_has_personToDelete[] = $row; + } + } + } + + // wenn UHSTAT Daten an Person, die gelöscht werden soll, hängen + if (!empty($uhstat_has_personToDelete)) + { + // Wenn es auch UHSTAT Daten für die Person, die bleibt, gibt + if (!empty($uhstat_has_personToKeep)) + { + // Unklar: welche Version der Daten soll behalten werden? + $msg_error[] = 'Beide Personen haben UHSTAT1 Daten. Können nicht zusammengelegt werden.
+ Sie müssen die Datensätze manuell bereinigen, bevor Sie die Personen zusammenlegen können.'; + $error = true; + } + else + { + // Es gibt nur UHSTAT Daten für die zu löschende Person: Update + foreach ($uhstat_has_personToDelete as $uhstat_toDelete) + { + $sql_query_upd1 .= "UPDATE bis.tbl_uhstat1daten SET person_id=" . $db->db_add_param($personToKeep, FHC_INTEGER) . " WHERE uhstat1daten_id=" . $db->db_add_param($uhstat_toDelete->uhstat1daten_id, FHC_INTEGER) . ";"; + } + } + } + } + if ($error == false) { // Wenn bei einer der Personen das Foto gesperrt ist, dann die Sperre uebernehmen @@ -521,7 +573,7 @@ if (isset($personToDelete) && isset($personToKeep) && $personToDelete >= 0 && $p Alte Anmerkungen: ".$personToDelete_obj->anmerkungen; $anmerkung .= " - + Zusammengelegt mit Person-ID ".$personToDelete_obj->person_id." am ".date('d.m.Y H:i:s')." von ".$uid; // Letztbenutzten Zugangscode abfragen und übernehmen From 8b9c023781d1e5c46c32f633249de102dd037e89 Mon Sep 17 00:00:00 2001 From: KarpAlex Date: Mon, 18 Dec 2023 17:51:05 +0100 Subject: [PATCH 06/16] Personen zusammenlegen: Rueckstellungen are checked --- vilesci/stammdaten/personen_wartung.php | 52 +++++++++++++++++++++++++ 1 file changed, 52 insertions(+) diff --git a/vilesci/stammdaten/personen_wartung.php b/vilesci/stammdaten/personen_wartung.php index 99f83c454..b2aef6055 100644 --- a/vilesci/stammdaten/personen_wartung.php +++ b/vilesci/stammdaten/personen_wartung.php @@ -455,6 +455,58 @@ if (isset($personToDelete) && isset($personToKeep) && $personToDelete >= 0 && $p } } + if($result = @$db->db_query("SELECT 1 FROM public.tbl_rueckstellung LIMIT 1")) + { + $rueckstellung_has_personToKeep = array(); + $rueckstellung_has_personToDelete = array(); + + $rueckstellung_query = " + SELECT rueckstellung_id, person_id + FROM public.tbl_rueckstellung + WHERE ( + person_id = " . $db->db_add_param($personToKeep, FHC_INTEGER) . " OR + person_id = " . $db->db_add_param($personToDelete, FHC_INTEGER) . " + ) + ORDER BY insertamum DESC NULLS LAST"; + + // Herausfinden, ob Rueckstellung Daten der löschenden oder zu belassenden Person zugeordnet sind + if ($result = $db->db_query($rueckstellung_query)) + { + while ($row = $db->db_fetch_object($result)) + { + if ($row->person_id === $personToKeep) + { + $rueckstellung_has_personToKeep[] = $row; + } + else if ($row->person_id === $personToDelete) + { + $rueckstellung_has_personToDelete[] = $row; + } + } + } + + // wenn Rueckstellung Daten an Person, die gelöscht werden soll, hängen + if (!empty($rueckstellung_has_personToDelete)) + { + // Wenn es auch Rueckstellung Daten für die Person, die bleibt, gibt + if (!empty($rueckstellung_has_personToKeep)) + { + // Unklar: welche Version der Daten soll behalten werden? + $msg_error[] = 'Beide Personen haben Rückstellung Daten. Können nicht zusammengelegt werden.
+ Sie müssen die Datensätze manuell bereinigen, bevor Sie die Personen zusammenlegen können.'; + $error = true; + } + else + { + // Es gibt nur Rueckstellung Daten für die zu löschende Person: Update + foreach ($rueckstellung_has_personToDelete as $rueckstellung_toDelete) + { + $sql_query_upd1 .= "UPDATE public.tbl_rueckstellung SET person_id=" . $db->db_add_param($personToKeep, FHC_INTEGER) . " WHERE rueckstellung_id=" . $db->db_add_param($rueckstellung_toDelete->rueckstellung_id, FHC_INTEGER) . ";"; + } + } + } + } + if ($error == false) { // Wenn bei einer der Personen das Foto gesperrt ist, dann die Sperre uebernehmen From a2905c3e93acd0309a6ad505ae361c42f15aa05e Mon Sep 17 00:00:00 2001 From: ma0048 Date: Mon, 15 Jan 2024 12:25:34 +0100 Subject: [PATCH 07/16] - scheduler angepasst - config erweitert um mahnsperre und zahlungsbedingungen - fas config hinzugefuegt --- config/vilesci.config-default.inc.php | 11 ++++++++++- content/student/studentDBDML.php | 5 ++++- 2 files changed, 14 insertions(+), 2 deletions(-) diff --git a/config/vilesci.config-default.inc.php b/config/vilesci.config-default.inc.php index ddcf8d563..a372bc9da 100644 --- a/config/vilesci.config-default.inc.php +++ b/config/vilesci.config-default.inc.php @@ -283,7 +283,7 @@ define('FAS_DOPPELTE_BUCHUNGSTYPEN_CHECK', serialize( 'StudiengebuehrRestzahlung' => array('StudiengebuehrErhoeht', 'Studiengebuehr', 'StudiengebuehrRestzahlung'), 'OEH' => array('OEH') )) -)); +); // Spezialnoten die am Zeunigs und Diplomasupplement ignoriert werden define('ZEUGNISNOTE_NICHT_ANZEIGEN',serialize(array('iar', 'nz'))); @@ -295,4 +295,13 @@ define ('DEFAULT_LEHRMODUS','regulaer'); //Echter Dienstvertrag define ('DEFAULT_ECHTER_DIENSTVERTRAG',[103,110]); +//Buchungstypen die fix auf eine bestimmte Kostenstelle gebucht werden sollen +//Buchungstyp => Studiengang_kz +define('FAS_BUCHUNGSTYP_FIXE_KOSTENSTELLE', serialize( + array( + 'Test_1' => 0, + 'Test_2' => 2 + ) +)); + ?> diff --git a/content/student/studentDBDML.php b/content/student/studentDBDML.php index 4e5f5452c..88c11676a 100644 --- a/content/student/studentDBDML.php +++ b/content/student/studentDBDML.php @@ -2584,13 +2584,16 @@ if(!$error) } else { + if(defined('FAS_BUCHUNGSTYP_FIXE_KOSTENSTELLE') && isset(unserialize(FAS_BUCHUNGSTYP_FIXE_KOSTENSTELLE)[$_POST['buchungstyp_kurzbz']])) + $kostenstelle = unserialize(FAS_BUCHUNGSTYP_FIXE_KOSTENSTELLE)[$_POST['buchungstyp_kurzbz']]; + foreach ($person_ids as $person_id) { if($person_id!='') { $buchung = new konto(); $buchung->person_id = $person_id; - $buchung->studiengang_kz = $_POST['studiengang_kz']; + $buchung->studiengang_kz = isset($kostenstelle) ? $kostenstelle : $_POST['studiengang_kz']; $buchung->studiensemester_kurzbz = $_POST['studiensemester_kurzbz']; $buchung->buchungsnr_verweis=''; $buchung->betrag = $_POST['betrag']; From 5e99ff9c8bc420912407cb9c38641aae51485a74 Mon Sep 17 00:00:00 2001 From: ma0048 Date: Mon, 29 Jan 2024 14:29:39 +0100 Subject: [PATCH 08/16] - uebersicht erweitert - job angepasst --- system/filtersupdate.php | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/system/filtersupdate.php b/system/filtersupdate.php index cc9abfb98..a044a3616 100644 --- a/system/filtersupdate.php +++ b/system/filtersupdate.php @@ -378,7 +378,8 @@ $filters = array( {"name" : "FoerderfallId"}, {"name" : "LeistungsdatenId"}, {"name" : "startjahr"}, - {"name" : "endjahr"} + {"name" : "endjahr"}, + {"name" : "Uebermittelt"} ], "filters": [] } From cbf71a416f9fd49b3d12d48ac9bc84b698d176d6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Andreas=20=C3=96sterreicher?= Date: Fri, 16 Feb 2024 11:23:20 +0100 Subject: [PATCH 09/16] Deprecated Spalte Gesamtstunden aus Projektarbeitsexport entfernt --- content/statistik/projektarbeitexport.xls.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/content/statistik/projektarbeitexport.xls.php b/content/statistik/projektarbeitexport.xls.php index 3b8e6b31e..2422292e7 100644 --- a/content/statistik/projektarbeitexport.xls.php +++ b/content/statistik/projektarbeitexport.xls.php @@ -64,7 +64,7 @@ $format_title->setAlign('merge'); //Zeilenueberschriften ausgeben $headline = array('Typ der Projektarbeit','Titel der Projektarbeit','Student', - 'Note','Punkte','Beginn','Ende','Freigegeben','Gesperrt bis','Gesamtstunden','Themenbereich', + 'Note','Punkte','Beginn','Ende','Freigegeben','Gesperrt bis','Themenbereich', 'Anmerkung','Projektarbeit ID'); $i = 0; @@ -81,7 +81,7 @@ $qry = "SELECT trim(COALESCE(titelpre,'') || ' ' || COALESCE(vorname,'') || ' ' || COALESCE(nachname,'') || ' ' || COALESCE(titelpost,'')), (SELECT anmerkung FROM lehre.tbl_note WHERE note=tbl_projektarbeit.note) as anmerkung, punkte, beginn, - ende, CASE WHEN freigegeben THEN 'Ja' ELSE 'Nein' END, gesperrtbis, gesamtstunden, themenbereich, + ende, CASE WHEN freigegeben THEN 'Ja' ELSE 'Nein' END, gesperrtbis, themenbereich, tbl_projektarbeit.anmerkung, projektarbeit_id FROM lehre.tbl_projektarbeit, lehre.tbl_lehreinheit, lehre.tbl_lehrveranstaltung, From 126defadb7e5d4e184533ca4ae684abd0c95809b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Andreas=20=C3=96sterreicher?= Date: Fri, 16 Feb 2024 14:59:53 +0100 Subject: [PATCH 10/16] =?UTF-8?q?Link=20hinzugef=C3=BCgt=20beim=20Zusammen?= =?UTF-8?q?legen=20von=20Personen=20wenn=20beide=20Personen=20UHSTAT=20aus?= =?UTF-8?q?gef=C3=BCllt=20haben=20damit=20diese=20direkt=20verglichen=20un?= =?UTF-8?q?d=20gel=C3=B6scht=20werden=20k=C3=B6nnen.=20Fehlende=20DB=20che?= =?UTF-8?q?cks=20hinzugef=C3=BCgt?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- system/dbupdate_3.4.php | 5 +++++ vilesci/stammdaten/personen_wartung.php | 9 +++++++-- 2 files changed, 12 insertions(+), 2 deletions(-) diff --git a/system/dbupdate_3.4.php b/system/dbupdate_3.4.php index 70f27d53a..964f02bae 100644 --- a/system/dbupdate_3.4.php +++ b/system/dbupdate_3.4.php @@ -87,6 +87,7 @@ $tabellen=array( "bis.tbl_nationengruppe" => array("nationengruppe_kurzbz","nationengruppe_bezeichnung","aktiv"), "bis.tbl_oehbeitrag" => array("oehbeitrag_id","studierendenbeitrag","versicherung","von_studiensemester_kurzbz","bis_studiensemester_kurzbz","insertamum","insertvon","updateamum","updatevon"), "bis.tbl_orgform" => array("orgform_kurzbz","code","bezeichnung","rolle","bisorgform_kurzbz","bezeichnung_mehrsprachig"), + "bis.tbl_uhstat1daten" => array("uhstat1daten_id", "mutter_geburtsstaat", "mutter_bildungsstaat", "mutter_geburtsjahr", "mutter_bildungmax", "vater_geburtsstaat", "vater_bildungsstaat", "vater_geburtsjahr", "vater_bildungmax", "person_id", "insertamum", "insertvon", "updateamum", "updatevon"), "bis.tbl_verwendung" => array("verwendung_code","verwendungbez"), "bis.tbl_zgv" => array("zgv_code","zgv_bez","zgv_kurzbz","bezeichnung","aktiv"), "bis.tbl_zgvmaster" => array("zgvmas_code","zgvmas_bez","zgvmas_kurzbz","bezeichnung","aktiv"), @@ -262,6 +263,8 @@ $tabellen=array( "public.tbl_geschaeftsjahr" => array("geschaeftsjahr_kurzbz","start","ende","bezeichnung"), "public.tbl_gruppe" => array("gruppe_kurzbz","studiengang_kz","semester","bezeichnung","beschreibung","sichtbar","lehre","aktiv","sort","mailgrp","generiert","updateamum","updatevon","insertamum","insertvon","ext_id","orgform_kurzbz","gid","content_visible","gesperrt","zutrittssystem","aufnahmegruppe","direktinskription"), "public.tbl_gruppe_manager" => array("gruppe_manager_id","gruppe_kurzbz","uid","insertamum","insertvon"), + "public.tbl_kennzeichen" => array("kennzeichen_id", "person_id", "kennzeichentyp_kurzbz", "inhalt", "aktiv", "insertamum", "insertvon", "updateamum", "updatevon"), + "public.tbl_kennzeichentyp" => array("kennzeichentyp_kurzbz", "bezeichnung", "aktiv"), "public.tbl_kontakt" => array("kontakt_id","person_id","kontakttyp","anmerkung","kontakt","zustellung","updateamum","updatevon","insertamum","insertvon","ext_id","standort_id"), "public.tbl_kontaktmedium" => array("kontaktmedium_kurzbz","beschreibung"), "public.tbl_kontakttyp" => array("kontakttyp","beschreibung","bezeichnung_mehrsprachig"), @@ -296,6 +299,8 @@ $tabellen=array( "public.tbl_prestudentstatus" => array("prestudent_id","status_kurzbz","studiensemester_kurzbz","ausbildungssemester","datum","orgform_kurzbz","insertamum","insertvon","updateamum","updatevon","ext_id","studienplan_id","bestaetigtam","bestaetigtvon","fgm","faktiv", "anmerkung","bewerbung_abgeschicktamum","rt_stufe","statusgrund_id"), "public.tbl_raumtyp" => array("raumtyp_kurzbz","beschreibung","kosten"), "public.tbl_reihungstest" => array("reihungstest_id","studiengang_kz","ort_kurzbz","anmerkung","datum","uhrzeit","updateamum","updatevon","insertamum","insertvon","ext_id","freigeschaltet","max_teilnehmer","oeffentlich","studiensemester_kurzbz","aufnahmegruppe_kurzbz","stufe","anmeldefrist","zugangs_ueberpruefung","zugangscode"), + "public.tbl_rueckstellung" => array("rueckstellung_id","person_id","status_kurzbz","datum_bis","insertamum","insertvon"), + "public.tbl_rueckstellung_status" => array("status_kurzbz", "bezeichnung_mehrsprachig", "sort", "aktiv"), "public.tbl_rt_ort" => array("rt_id","ort_kurzbz","uid"), "public.tbl_rt_person" => array("rt_person_id","person_id","rt_id","studienplan_id","anmeldedatum","teilgenommen","ort_kurzbz","punkte","insertamum","insertvon","updateamum","updatevon"), "public.tbl_rt_studienplan" => array("reihungstest_id","studienplan_id"), diff --git a/vilesci/stammdaten/personen_wartung.php b/vilesci/stammdaten/personen_wartung.php index b2aef6055..d2b5c3446 100644 --- a/vilesci/stammdaten/personen_wartung.php +++ b/vilesci/stammdaten/personen_wartung.php @@ -441,7 +441,12 @@ if (isset($personToDelete) && isset($personToKeep) && $personToDelete >= 0 && $p { // Unklar: welche Version der Daten soll behalten werden? $msg_error[] = 'Beide Personen haben UHSTAT1 Daten. Können nicht zusammengelegt werden.
- Sie müssen die Datensätze manuell bereinigen, bevor Sie die Personen zusammenlegen können.'; + Sie müssen die Datensätze manuell bereinigen, bevor Sie die Personen zusammenlegen können:
+ zum UHSTAT von Person A + / + zum UHSTAT von Person B +
'; + $error = true; } else @@ -1523,7 +1528,7 @@ if ($filter != '' || ($person_id_1 != '' && $person_id_2 != '')) $messageOutput .= '
'.$value; } } - echo '

' . $messageOutput . '

'; + echo '

' . $messageOutput . '

'; // Tabellen anzeigen echo '
'; From e433ca3d9b81eeda7fc3002e819fadc275a0356b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Andreas=20=C3=96sterreicher?= Date: Mon, 19 Feb 2024 09:22:54 +0100 Subject: [PATCH 11/16] Fehlende Gegenpruefungen im DB Check hinzugefuegt --- system/dbupdate_3.4.php | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/system/dbupdate_3.4.php b/system/dbupdate_3.4.php index 964f02bae..73fa311eb 100644 --- a/system/dbupdate_3.4.php +++ b/system/dbupdate_3.4.php @@ -56,6 +56,7 @@ require_once('dbupdate_3.4/21620_neues_feld_zum_erfassen_des_ESI.php'); echo '

Pruefe Tabellen und Attribute!

'; $tabellen=array( + "bis.tbl_abschluss" => array("ausbildung_code","abschluss_bez","bezeichnung","aktiv","in_oesterreich"), "bis.tbl_bisorgform" => array("bisorgform_kurzbz","code","bezeichnung"), "bis.tbl_archiv" => array("archiv_id","studiensemester_kurzbz","meldung","html","studiengang_kz","insertamum","insertvon","typ"), "bis.tbl_aufenthaltfoerderung" => array("aufenthaltfoerderung_code", "bezeichnung"), @@ -136,6 +137,10 @@ $tabellen=array( "campus.tbl_pruefungsanmeldung" => array("pruefungsanmeldung_id","uid","pruefungstermin_id","lehrveranstaltung_id","status_kurzbz","wuensche","reihung","kommentar","statusupdatevon","statusupdateamum","anrechnung_id","pruefungstyp_kurzbz","insertamum"), "campus.tbl_pruefungsstatus" => array("status_kurzbz","bezeichnung"), "campus.tbl_reservierung" => array("reservierung_id","ort_kurzbz","studiengang_kz","uid","stunde","datum","titel","beschreibung","semester","verband","gruppe","gruppe_kurzbz","veranstaltung_id","insertamum","insertvon"), + "campus.tbl_studierendenantrag" => array("studierendenantrag_id","prestudent_id","studiensemester_kurzbz","datum","typ","insertamum","insertvon","datum_wiedereinstieg","grund","dms_id"), + "campus.tbl_studierendenantrag_lehrveranstaltung" => array("studierendenantrag_lehrveranstaltung_id","studierendenantrag_id","lehrveranstaltung_id","studiensemester_kurzbz","note","anmerkung","insertamum","insertvon"), + "campus.tbl_studierendenantrag_status" => array("studierendenantrag_status_id","studierendenantrag_id","studierendenantrag_statustyp_kurzbz","insertamum","insertvon", "grund"), + "campus.tbl_studierendenantrag_statustyp"=> array("studierendenantrag_statustyp_kurzbz","bezeichnung"), "campus.tbl_studentbeispiel" => array("student_uid","beispiel_id","vorbereitet","probleme","updateamum","updatevon","insertamum","insertvon"), "campus.tbl_studentuebung" => array("student_uid","mitarbeiter_uid","abgabe_id","uebung_id","note","mitarbeitspunkte","punkte","anmerkung","benotungsdatum","updateamum","updatevon","insertamum","insertvon"), "campus.tbl_template" => array("template_kurzbz","bezeichnung","xsd","xslt_xhtml","xslfo_pdf"), @@ -159,8 +164,12 @@ $tabellen=array( "fue.tbl_ressource" => array("ressource_id","student_uid","mitarbeiter_uid","betriebsmittel_id","firma_id","bezeichnung","beschreibung","insertamum","insertvon","updateamum","updatevon"), "fue.tbl_scrumteam" => array("scrumteam_kurzbz","bezeichnung","punkteprosprint","tasksprosprint","gruppe_kurzbz"), "fue.tbl_scrumsprint" => array("scrumsprint_id","scrumteam_kurzbz","sprint_kurzbz","sprintstart","sprintende","insertamum","insertvon","updateamum","updatevon"), + "hr.tbl_audit_log" => array("audit_log_id","mtime","action","username","table_name","diff_data","row_data"), "hr.tbl_sachaufwand" => array("sachaufwand_id","mitarbeiter_uid","sachaufwandtyp_kurzbz","dienstverhaeltnis_id","beginn","ende","anmerkung","insertamum","insertvon","updateamum","updatevon"), "hr.tbl_sachaufwandtyp" => array("sachaufwandtyp_kurzbz","bezeichnung","sort", "aktiv"), + "hr.tbl_stundensatz" => array("stundensatz_id","uid","stundensatztyp","stundensatz","oe_kurzbz","gueltig_von","gueltig_bis","insertamum","insertvon","updateamum","updatevon"), + "hr.tbl_stundensatztyp" => array("stundensatztyp","bezeichnung","aktiv","insertamum","insertvon","updateamum","updatevon"), + "hr.tbl_tmp_store" => array("tmp_store_id","typ","mitarbeiter_uid","formdata","insertvon","insertamum","updatevon","updateamum"), "hr.tbl_dienstverhaeltnis" => array("dienstverhaeltnis_id","mitarbeiter_uid","vertragsart_kurzbz","oe_kurzbz","von","bis","insertamum","insertvon","updateamum","updatevon"), "hr.tbl_vertragsart" => array("vertragsart_kurzbz","bezeichnung","anmerkung","dienstverhaeltnis","vertragsart_kurzbz_parent","aktiv","sort"), "hr.tbl_vertragsbestandteil" => array("vertragsbestandteil_id","dienstverhaeltnis_id","vertragsbestandteiltyp_kurzbz","von", "bis","insertamum", "insertvon","updateamum","updatevon"), @@ -297,7 +306,7 @@ $tabellen=array( "public.tbl_preoutgoing_status" => array("preoutgoing_status_kurzbz","bezeichnung"), "public.tbl_prestudent" => array("prestudent_id","aufmerksamdurch_kurzbz","person_id","studiengang_kz","berufstaetigkeit_code","ausbildungcode","zgv_code","zgvort","zgvdatum","zgvmas_code","zgvmaort","zgvmadatum","aufnahmeschluessel","facheinschlberuf","reihungstest_id","anmeldungreihungstest","reihungstestangetreten","rt_gesamtpunkte","rt_punkte1","rt_punkte2","bismelden","anmerkung","dual","insertamum","insertvon","updateamum","updatevon","ext_id","ausstellungsstaat","rt_punkte3", "zgvdoktor_code", "zgvdoktorort", "zgvdoktordatum","mentor","zgvnation","zgvmanation","zgvdoktornation","gsstudientyp_kurzbz","aufnahmegruppe_kurzbz","udf_values","priorisierung","foerderrelevant","standort_code","zgv_erfuellt","zgvmas_erfuellt","zgvdoktor_erfuellt"), "public.tbl_prestudentstatus" => array("prestudent_id","status_kurzbz","studiensemester_kurzbz","ausbildungssemester","datum","orgform_kurzbz","insertamum","insertvon","updateamum","updatevon","ext_id","studienplan_id","bestaetigtam","bestaetigtvon","fgm","faktiv", "anmerkung","bewerbung_abgeschicktamum","rt_stufe","statusgrund_id"), - "public.tbl_raumtyp" => array("raumtyp_kurzbz","beschreibung","kosten"), + "public.tbl_raumtyp" => array("raumtyp_kurzbz","beschreibung","kosten","aktiv"), "public.tbl_reihungstest" => array("reihungstest_id","studiengang_kz","ort_kurzbz","anmerkung","datum","uhrzeit","updateamum","updatevon","insertamum","insertvon","ext_id","freigeschaltet","max_teilnehmer","oeffentlich","studiensemester_kurzbz","aufnahmegruppe_kurzbz","stufe","anmeldefrist","zugangs_ueberpruefung","zugangscode"), "public.tbl_rueckstellung" => array("rueckstellung_id","person_id","status_kurzbz","datum_bis","insertamum","insertvon"), "public.tbl_rueckstellung_status" => array("status_kurzbz", "bezeichnung_mehrsprachig", "sort", "aktiv"), From 6f92698631154eaec54103b6a454f05dff24d902 Mon Sep 17 00:00:00 2001 From: ma0048 Date: Tue, 20 Feb 2024 17:01:49 +0100 Subject: [PATCH 12/16] - bewerbungstool payments uebersetzungen anpassungen --- system/phrasesupdate.php | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/system/phrasesupdate.php b/system/phrasesupdate.php index b6b5b2ffc..35b49d1f4 100644 --- a/system/phrasesupdate.php +++ b/system/phrasesupdate.php @@ -4424,7 +4424,7 @@ When on hold, the date is only a reminder.', 'sprache' => 'German', 'text' => 'Wir möchten Sie darauf aufmerksam machen, dass bei der Überweisung *immer* die Rechnungsnummer als Zahlungsreferenz anzuführen ist. Andernfalls erfolgt keine automatische Zahlungszuordnung und es kann zu einer Verzögerung der Darstellung des aktuellen Zahlungsstatus - der Rechnung im CIS kommen. + der Rechnung im Campus Informations-System kommen.

Im Falle dass der Betrag an ein falsches Konto überwiesen wurde, bitten wir Sie höflichst sich an Ihre Bank zu wenden. @@ -4439,7 +4439,7 @@ When on hold, the date is only a reminder.', array( 'sprache' => 'English', 'text' => 'We would like to draw your attention to the fact that the invoice number must *always* be quoted as the payment reference when making a bank transfer. - Otherwise, no automatic payment allocation will take place and there may be a delay in displaying the current payment status of the invoice in CIS. + Otherwise, no automatic payment allocation will take place and there may be a delay in displaying the current payment status of the invoice in Campus Informations-System.

In the event that the amount has been transferred to an incorrect account, we kindly ask you to contact your bank. @@ -4513,7 +4513,7 @@ When on hold, the date is only a reminder.', 'sprache' => 'English', 'text' => 'Foreign bank transfers:
- In the case of foreign bank transfers, the charges are to be paid by the + In the case of foreign bank transfers, the charges are to be paid by
the payer in addition to the invoice amounts.', 'description' => '', @@ -4622,7 +4622,7 @@ Die Rechnung wird Ihnen erneut zugesendet. Erst nach Erhalt der Rechn array( 'sprache' => 'English', 'text' => 'In this case, please check your spam folder. If the invoice has not been sent, please inform us at billing@technikum-wien.at. -The invoice will be sent to you again. The amount is only to be transferred after receipt of the invoice', +The invoice will be sent to you again. The amount is only to be transferred after receipt of the invoice!', 'description' => '', 'insertvon' => 'system' ) @@ -4714,7 +4714,7 @@ The invoice will be sent to you again. The amount is only to be trans ), array( 'sprache' => 'English', - 'text' => 'Please contact your study program assistant. The outstanding invoice will be canceled.', + 'text' => 'Please contact your degree program assistant. The outstanding invoice will be canceled.', 'description' => '', 'insertvon' => 'system' ) From aa6c5c50c2af8bb9b72d5a597a26291064bbd414 Mon Sep 17 00:00:00 2001 From: ma0048 Date: Mon, 26 Feb 2024 09:03:06 +0100 Subject: [PATCH 13/16] - fas filter gefixt --- rdf/lehrverbandsgruppe.rdf.php | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/rdf/lehrverbandsgruppe.rdf.php b/rdf/lehrverbandsgruppe.rdf.php index 755444b6d..4df07637a 100644 --- a/rdf/lehrverbandsgruppe.rdf.php +++ b/rdf/lehrverbandsgruppe.rdf.php @@ -85,7 +85,6 @@ if(isset($_GET['prestudent']) && $_GET['prestudent']=='false') else $berechtigt_studiengang = array_merge($berechtigt_studiengang, $berechtigung->getStgKz('assistenz')); -//var_dump($berechtigung); array_unique($berechtigt_studiengang); $stg_kz_query=''; if (count($berechtigt_studiengang)>0) @@ -382,13 +381,13 @@ function draw_orgformsubmenu($stg_kz, $orgform) $orgform_sequence[$stg_kz].= "\n\t\t\t\n"; $orgform_sequence[$stg_kz].= "\t\t\t"; - $orgform_sequence[$stg_kz].= "\n\t\t\t\tstudiensemester_kurzbz/bewerber\">\n"; + $orgform_sequence[$stg_kz].= "\n\t\t\t\tstudiensemester_kurzbz/bewerber\">\n"; $orgform_sequence[$stg_kz].= "\t\t\tstudiensemester_kurzbz/bewerber/reihungstestnichtangemeldet\" />\n"; $orgform_sequence[$stg_kz].= "\t\t\t"; $orgform_sequence[$stg_kz].= "\t\t\tstudiensemester_kurzbz/bewerber/reihungstestangemeldet\">\n"; - $orgform_sequence[$stg_kz].= "\t\t\t\tstudiensemester_kurzbz/bewerber/reihungstestangemeldet/reihungstestnichtangemeldet/teilgenommen\" />\n"; - $orgform_sequence[$stg_kz].= "\t\t\t\tstudiensemester_kurzbz/bewerber/reihungstestangemeldet/reihungstestnichtangemeldet/nichtteilgenommen\" />\n"; + $orgform_sequence[$stg_kz].= "\t\t\t\tstudiensemester_kurzbz/bewerber/reihungstestangemeldet/teilgenommen\" />\n"; + $orgform_sequence[$stg_kz].= "\t\t\t\tstudiensemester_kurzbz/bewerber/reihungstestangemeldet/nichtteilgenommen\" />\n"; $orgform_sequence[$stg_kz].= "\t\t\t\t"; $orgform_sequence[$stg_kz].= "\n\t\t\t\n"; @@ -915,7 +914,7 @@ draw_orgformpart($stg_kz); echo "\n\t\t\t\n"; echo "\t\t\t"; - echo "\t\t\t\tstudiensemester_kurzbz/bewerber\">\n"; + echo "\n\t\t\t\tstudiensemester_kurzbz/bewerber\">\n"; echo "\t\t\t\tstudiensemester_kurzbz/bewerber/reihungstestnichtangemeldet\" />\n"; echo "\t\t\t"; echo "\t\t\t\tstudiensemester_kurzbz/bewerber/reihungstestangemeldet\">\n"; From 45f2593b133957366d457417dff10ce4c296ffef Mon Sep 17 00:00:00 2001 From: Harald Bamberger Date: Tue, 27 Feb 2024 13:54:58 +0100 Subject: [PATCH 14/16] fix bug with excel showing file error when switching to editmode --- cis/private/lehre/notenliste.xls.php | 23 ++++++++++++++++------- 1 file changed, 16 insertions(+), 7 deletions(-) diff --git a/cis/private/lehre/notenliste.xls.php b/cis/private/lehre/notenliste.xls.php index 63cf45d29..6c8db5246 100644 --- a/cis/private/lehre/notenliste.xls.php +++ b/cis/private/lehre/notenliste.xls.php @@ -114,12 +114,15 @@ else $format_highlight->setFgColor(15); $format_highlight->setBorder(1); $format_highlight->setBorderColor('white'); + $format_highlight->setAlign('left'); + $format_highlight->setNumFormat(49); $format_highlightright=& $workbook->addFormat(); $format_highlightright->setFgColor(15); $format_highlightright->setBorder(1); $format_highlightright->setBorderColor('white'); $format_highlightright->setAlign('right'); + $format_highlightright->setNumFormat(49); $format_highlightright_date=& $workbook->addFormat(); $format_highlightright_date->setFgColor(15); @@ -318,17 +321,23 @@ else $worksheet->write($lines,1,$elem->uid); $worksheet->write($lines,2,$elem->nachname.$inc); - $worksheet->write($lines,3,$elem->vorname); - //wenn Wahlname vorhanden überschreibt dieser den Vornamen - $worksheet->write($lines,3,$elem->wahlname); - $worksheet->write($lines,4,'="'.$elem->semester.$elem->verband.$elem->gruppe.'"'); - $worksheet->write($lines,5,'="'.trim($elem->matrikelnr).'"',$format_highlight); + if( NULL !== $elem->wahlname ) + { + //wenn Wahlname vorhanden überschreibt dieser den Vornamen + $worksheet->write($lines,3,$elem->wahlname); + } + else + { + $worksheet->write($lines,3,$elem->vorname); + } + $worksheet->write($lines,4,$elem->semester.$elem->verband.$elem->gruppe); + $worksheet->write($lines,5,trim($elem->matrikelnr),$format_highlight); $worksheet->write($lines,6, $note, $format_highlightright); // Nachprüfung if (defined('CIS_GESAMTNOTE_PRUEFUNG_TERMIN2') && CIS_GESAMTNOTE_PRUEFUNG_TERMIN2) { - $worksheet->write($lines,8, '="'.trim($elem->matrikelnr).'"', $format_highlight); + $worksheet->write($lines,8, trim($elem->matrikelnr), $format_highlight); $pr = new Pruefung(); $pr->getPruefungen($elem->uid, "Termin2", $lvid, $sem); $output2 = $pr->result; @@ -349,7 +358,7 @@ else // Nachprüfung if (defined('CIS_GESAMTNOTE_PRUEFUNG_TERMIN3') && CIS_GESAMTNOTE_PRUEFUNG_TERMIN3) { - $worksheet->write($lines,12, '="'.trim($elem->matrikelnr).'"', $format_highlight); + $worksheet->write($lines,12, trim($elem->matrikelnr), $format_highlight); $pr = new Pruefung(); $pr->getPruefungen($elem->uid, "Termin3", $lvid, $sem); $output3 = $pr->result; From c4d4f55d06083eccd70298d6810c59b6c62d462a Mon Sep 17 00:00:00 2001 From: Harald Bamberger Date: Wed, 28 Feb 2024 18:22:35 +0100 Subject: [PATCH 15/16] use tabulator5 simple theme to get smaller tables --- public/css/Tabulator5.css | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/public/css/Tabulator5.css b/public/css/Tabulator5.css index 48124335f..b755f0d76 100644 --- a/public/css/Tabulator5.css +++ b/public/css/Tabulator5.css @@ -1,4 +1,4 @@ -@import '../../vendor/olifolkerd/tabulator5/dist/css/tabulator_bootstrap5.min.css'; +@import '../../vendor/olifolkerd/tabulator5/dist/css/tabulator_simple.min.css'; /* Apply borders and background to Cell instead of the Row * otherwise frozen columns won't look good (columns behind @@ -38,3 +38,9 @@ .tabulator .tabulator-col-resize-handle:last-of-type { z-index: 999999; } + +.tabulator-cell .btn { + padding: 0 .5rem; + max-height: 22px; + min-width: 30px; +} From 621429798c087e0ffc4bf8482cce7d5a33e1194b Mon Sep 17 00:00:00 2001 From: kindlm Date: Thu, 29 Feb 2024 09:50:58 +0100 Subject: [PATCH 16/16] Abschlussgrad in Studienblatt erst aus Studienordnung ermitteln --- rdf/studienblatt.xml.php | 20 ++++++++++++++++---- 1 file changed, 16 insertions(+), 4 deletions(-) diff --git a/rdf/studienblatt.xml.php b/rdf/studienblatt.xml.php index f0caadeb8..3ee805c04 100644 --- a/rdf/studienblatt.xml.php +++ b/rdf/studienblatt.xml.php @@ -269,11 +269,23 @@ foreach($uid_arr as $uid) echo "\t\tsprache."]]>\n"; echo "\t\tregelstudiendauer."]]>\n"; - $akadgrad = new akadgrad(); - $akadgrad->getAkadgradStudent($student->uid); + // Abschlussgrad ermitteln + // Erst aus Studienordnung, wenn nicht gesetzt aus akadgrad-Tabelle - echo "\t\ttitel."]]>\n"; - echo "\t\takadgrad_kurzbz."]]>\n"; + if ($studienordnung->akadgrad_id != '') + { + $akadgrad = new akadgrad($studienordnung->akadgrad_id); + echo "\t\ttitel."]]>\n"; + echo "\t\takadgrad_kurzbz."]]>\n"; + } + else + { + $akadgrad = new akadgrad(); + $akadgrad->getAkadgradStudent($student->uid); + + echo "\t\ttitel."]]>\n"; + echo "\t\takadgrad_kurzbz."]]>\n"; + } //für ao. Studierende wird die StgKz der Lehrveranstaltungen benötigt, die sie besuchen $lv_studiengang_kz='';