Code Quality

This commit is contained in:
chfhtw
2025-10-03 07:46:07 +02:00
parent ba1401f98d
commit cecaa458bf
6 changed files with 104 additions and 21 deletions
@@ -666,4 +666,4 @@ class Prestudentstatus_model extends DB_Model
'status_kurzbz' => self::STATUS_BEWERBER
]);
}
}
}
@@ -204,8 +204,15 @@ class Pruefung_model extends DB_Model
$this->addSelect('campus.get_status_studierendenantrag(a.studierendenantrag_id) status');
$this->addSelect('pss.ausbildungssemester');
$this->addJoin('(SELECT MAX(datum) AS datum, lehreinheit_id AS le_id, student_uid AS stud_uid FROM lehre.tbl_pruefung p WHERE pruefungstyp_kurzbz IN (\'kommPruef\', \'zusKommPruef\') GROUP BY lehreinheit_id, student_uid) lpd',
'p.datum = lpd.datum AND p.lehreinheit_id = lpd.le_id AND p.student_uid = lpd.stud_uid');
$this->addJoin(
'(
SELECT MAX(datum) AS datum, lehreinheit_id AS le_id, student_uid AS stud_uid
FROM lehre.tbl_pruefung p
WHERE pruefungstyp_kurzbz IN (\'kommPruef\', \'zusKommPruef\')
GROUP BY lehreinheit_id, student_uid
) lpd',
'p.datum = lpd.datum AND p.lehreinheit_id = lpd.le_id AND p.student_uid = lpd.stud_uid'
);
$this->addJoin('lehre.tbl_lehreinheit le', 'lehreinheit_id');
$this->addJoin('lehre.tbl_lehrveranstaltung lv', 'lehrveranstaltung_id');
$this->addJoin('public.tbl_student s', 'student_uid');
+18 -3
View File
@@ -9,7 +9,6 @@ $db = new basis_db();
if (isset($_REQUEST["xmlformat"]) && $_REQUEST["xmlformat"] == "xml")
{
if(isset($_GET['id'])) {
$id = $_GET['id'];
@@ -36,7 +35,19 @@ else
$query = "
SELECT stg.bezeichnung, bezeichnung_mehrsprachig[(SELECT index FROM public.tbl_sprache WHERE sprache=" . $db->db_add_param(getSprache(), FHC_STRING) . ")], studierendenantrag_id, matrikelnr, studienjahr_kurzbz, a.studiensemester_kurzbz, vorname, nachname, studiengang_kz, pss.ausbildungssemester AS semester, pss.datum, a.grund
SELECT
stg.bezeichnung,
bezeichnung_mehrsprachig[(SELECT index FROM public.tbl_sprache WHERE sprache=" . $db->db_add_param(getSprache(), FHC_STRING) . ")],
studierendenantrag_id,
matrikelnr,
studienjahr_kurzbz,
a.studiensemester_kurzbz,
vorname,
nachname,
studiengang_kz,
pss.ausbildungssemester AS semester,
pss.datum,
a.grund
FROM
campus.tbl_studierendenantrag a
JOIN public.tbl_student USING (prestudent_id)
@@ -44,7 +55,11 @@ $query = "
JOIN public.tbl_person USING (person_id)
JOIN public.tbl_studiengang stg USING (studiengang_kz)
JOIN public.tbl_studiensemester USING (studiensemester_kurzbz)
LEFT JOIN public.tbl_prestudentstatus pss ON (pss.prestudent_id = a.prestudent_id AND pss.studiensemester_kurzbz=a.studiensemester_kurzbz AND pss.status_kurzbz=get_rolle_prestudent(a.prestudent_id, a.studiensemester_kurzbz))
LEFT JOIN public.tbl_prestudentstatus pss ON (
pss.prestudent_id = a.prestudent_id AND
pss.studiensemester_kurzbz=a.studiensemester_kurzbz AND
pss.status_kurzbz=get_rolle_prestudent(a.prestudent_id, a.studiensemester_kurzbz)
)
LEFT JOIN lehre.tbl_studienplan plan USING (studienplan_id)
JOIN bis.tbl_orgform ON (tbl_orgform.orgform_kurzbz = public.get_orgform_prestudent(a.prestudent_id, a.studiensemester_kurzbz))" . $where;
+24 -5
View File
@@ -9,12 +9,13 @@ $db = new basis_db();
if (isset($_REQUEST["xmlformat"]) && $_REQUEST["xmlformat"] == "xml")
{
if(isset($_GET['id'])) {
$id = $_GET['id'];
$where = " WHERE studierendenantrag_id = " . $db->db_add_param($id) . "
AND a.typ = 'AbmeldungStgl' AND campus.get_status_studierendenantrag(a.studierendenantrag_id) IN ('Genehmigt', 'Beeinsprucht', 'EinspruchAbgelehnt', 'Abgemeldet');";
AND a.typ = 'AbmeldungStgl'
AND campus.get_status_studierendenantrag(a.studierendenantrag_id)
IN ('Genehmigt', 'Beeinsprucht', 'EinspruchAbgelehnt', 'Abgemeldet');";
$not_found_error = 'Studierendenantrag not found'. $id;
} elseif(isset($_GET['uid']) && isset($_GET['prestudent_id'])) {
$uid = $_GET['uid'];
@@ -26,7 +27,9 @@ if (isset($_REQUEST["xmlformat"]) && $_REQUEST["xmlformat"] == "xml")
$prestudent_id = (array_filter($prestudent_id, 'strlen'));
$where = " WHERE a.prestudent_id in (" . $db->db_implode4SQL($prestudent_id) . ")
AND a.typ = 'AbmeldungStgl' AND campus.get_status_studierendenantrag(a.studierendenantrag_id) IN ('Genehmigt', 'Beeinsprucht', 'EinspruchAbgelehnt', 'Abgemeldet');";
AND a.typ = 'AbmeldungStgl'
AND campus.get_status_studierendenantrag(a.studierendenantrag_id)
IN ('Genehmigt', 'Beeinsprucht', 'EinspruchAbgelehnt', 'Abgemeldet');";
$not_found_error = 'Studierendenantrag not found for: ' . implode(',', $uid);
} else
die('<error>wrong parameters</error>');
@@ -36,7 +39,19 @@ else
$query = "
SELECT stg.bezeichnung, bezeichnung_mehrsprachig[(SELECT index FROM public.tbl_sprache WHERE sprache=" . $db->db_add_param(getSprache(), FHC_STRING) . ")], studierendenantrag_id, matrikelnr, studienjahr_kurzbz, a.studiensemester_kurzbz, vorname, nachname, studiengang_kz, pss.ausbildungssemester AS semester, pss.bestaetigtam, a.grund
SELECT
stg.bezeichnung,
bezeichnung_mehrsprachig[(SELECT index FROM public.tbl_sprache WHERE sprache=" . $db->db_add_param(getSprache(), FHC_STRING) . ")],
studierendenantrag_id,
matrikelnr,
studienjahr_kurzbz,
a.studiensemester_kurzbz,
vorname,
nachname,
studiengang_kz,
pss.ausbildungssemester AS semester,
pss.bestaetigtam,
a.grund
FROM
campus.tbl_studierendenantrag a
JOIN public.tbl_student USING (prestudent_id)
@@ -44,7 +59,11 @@ $query = "
JOIN public.tbl_person USING (person_id)
JOIN public.tbl_studiengang stg USING (studiengang_kz)
JOIN public.tbl_studiensemester USING (studiensemester_kurzbz)
LEFT JOIN public.tbl_prestudentstatus pss ON (pss.prestudent_id = a.prestudent_id AND pss.studiensemester_kurzbz=a.studiensemester_kurzbz AND pss.status_kurzbz=get_rolle_prestudent(a.prestudent_id, a.studiensemester_kurzbz))
LEFT JOIN public.tbl_prestudentstatus pss ON (
pss.prestudent_id = a.prestudent_id AND
pss.studiensemester_kurzbz=a.studiensemester_kurzbz AND
pss.status_kurzbz=get_rolle_prestudent(a.prestudent_id, a.studiensemester_kurzbz)
)
LEFT JOIN lehre.tbl_studienplan plan USING (studienplan_id)
JOIN bis.tbl_orgform ON (tbl_orgform.orgform_kurzbz = public.get_orgform_prestudent(a.prestudent_id, a.studiensemester_kurzbz))" . $where;
+19 -3
View File
@@ -10,7 +10,6 @@ $db = new basis_db();
if (isset($_REQUEST["xmlformat"]) && $_REQUEST["xmlformat"] == "xml")
{
if(isset($_GET['id'])) {
$id = $_GET['id'];
@@ -37,7 +36,20 @@ else
$query = "
SELECT stg.bezeichnung, bezeichnung_mehrsprachig[(SELECT index FROM public.tbl_sprache WHERE sprache=" . $db->db_add_param(getSprache(), FHC_STRING) . ")], studierendenantrag_id, matrikelnr, studienjahr_kurzbz, a.studiensemester_kurzbz, vorname, nachname, studiengang_kz, pss.ausbildungssemester AS semester, a.grund, datum_wiedereinstieg, a.datum
SELECT
stg.bezeichnung,
bezeichnung_mehrsprachig[(SELECT index FROM public.tbl_sprache WHERE sprache=" . $db->db_add_param(getSprache(), FHC_STRING) . ")],
studierendenantrag_id,
matrikelnr,
studienjahr_kurzbz,
a.studiensemester_kurzbz,
vorname,
nachname,
studiengang_kz,
pss.ausbildungssemester AS semester,
a.grund,
datum_wiedereinstieg,
a.datum
FROM
campus.tbl_studierendenantrag a
JOIN public.tbl_student USING (prestudent_id)
@@ -45,7 +57,11 @@ $query = "
JOIN public.tbl_person USING (person_id)
JOIN public.tbl_studiengang stg USING (studiengang_kz)
JOIN public.tbl_studiensemester USING (studiensemester_kurzbz)
LEFT JOIN public.tbl_prestudentstatus pss ON (pss.prestudent_id = a.prestudent_id AND pss.studiensemester_kurzbz=a.studiensemester_kurzbz AND pss.status_kurzbz=get_rolle_prestudent(a.prestudent_id, a.studiensemester_kurzbz))
LEFT JOIN public.tbl_prestudentstatus pss ON (
pss.prestudent_id = a.prestudent_id AND
pss.studiensemester_kurzbz=a.studiensemester_kurzbz AND
pss.status_kurzbz=get_rolle_prestudent(a.prestudent_id, a.studiensemester_kurzbz)
)
LEFT JOIN lehre.tbl_studienplan plan USING (studienplan_id)
JOIN bis.tbl_orgform ON (tbl_orgform.orgform_kurzbz = public.get_orgform_prestudent(a.prestudent_id, a.studiensemester_kurzbz))" . $where;
+33 -7
View File
@@ -12,7 +12,7 @@ define('ENVIRONMENT', isset($_SERVER['CI_ENV']) ? $_SERVER['CI_ENV'] : 'developm
// Get BASEPATH Var
$system_path = dirname(__FILE__).'/../vendor/codeigniter/framework/system';
if (($_temp = realpath($system_path)) !== FALSE)
if (($_temp = realpath($system_path)) !== false)
$system_path = $_temp.'/';
else
$system_path = rtrim($system_path, '/').'/';
@@ -21,12 +21,12 @@ define('BASEPATH', str_replace('\\', '/', $system_path));
// Get APPPATH Var
$application_folder = dirname(__FILE__).'/../application';
if (is_dir($application_folder)) {
if (($_temp = realpath($application_folder)) !== FALSE)
if (($_temp = realpath($application_folder)) !== false)
$application_folder = $_temp;
define('APPPATH', $application_folder.DIRECTORY_SEPARATOR);
} else {
if (!is_dir(BASEPATH.$application_folder.DIRECTORY_SEPARATOR)) {
header('HTTP/1.1 503 Service Unavailable.', TRUE, 503);
header('HTTP/1.1 503 Service Unavailable.', true, 503);
echo 'Your application folder path does not appear to be set correctly. Please open the following file and correct this: '.SELF;
exit(3); // EXIT_CONFIG
}
@@ -45,7 +45,6 @@ $db = new basis_db();
if (isset($_REQUEST["xmlformat"]) && $_REQUEST["xmlformat"] == "xml")
{
if(isset($_GET['id'])) {
$id = $_GET['id'];
@@ -77,7 +76,22 @@ if ($config['note_blacklist_wiederholung']) {
$query = "
SELECT stg.bezeichnung, tbl_orgform.bezeichnung_mehrsprachig[(SELECT index FROM public.tbl_sprache WHERE sprache=" . $db->db_add_param(getSprache(), FHC_STRING) . ")], studierendenantrag_id, matrikelnr, studienjahr_kurzbz, a.studiensemester_kurzbz, vorname, nachname, studiengang_kz, pss.ausbildungssemester AS semester, (
SELECT
stg.bezeichnung,
tbl_orgform.bezeichnung_mehrsprachig[(
SELECT index
FROM public.tbl_sprache
WHERE sprache=" . $db->db_add_param(getSprache(), FHC_STRING) . "
)],
studierendenantrag_id,
matrikelnr,
studienjahr_kurzbz,
a.studiensemester_kurzbz,
vorname,
nachname,
studiengang_kz,
pss.ausbildungssemester AS semester,
(
SELECT
insertamum::date
FROM
@@ -87,7 +101,15 @@ $query = "
ORDER BY
insertamum DESC
LIMIT 1
) AS abmeldedatum, (SELECT pt.text FROM system.tbl_phrase p JOIN system.tbl_phrasentext pt USING(phrase_id) WHERE p.category=" . $db->db_add_param('studierendenantrag', FHC_STRING) . " AND p.phrase=" . $db->db_add_param('grund_Wiederholung_deadline', FHC_STRING) . " AND pt.sprache=" . $db->db_add_param(getSprache(), FHC_STRING) . " LIMIT 1) AS grund
) AS abmeldedatum,
(
SELECT pt.text
FROM system.tbl_phrase p
JOIN system.tbl_phrasentext pt USING(phrase_id)
WHERE p.category=" . $db->db_add_param('studierendenantrag', FHC_STRING) . "
AND p.phrase=" . $db->db_add_param('grund_Wiederholung_deadline', FHC_STRING) . "
AND pt.sprache=" . $db->db_add_param(getSprache(), FHC_STRING) . " LIMIT 1
) AS grund
FROM
campus.tbl_studierendenantrag a
JOIN public.tbl_student USING (prestudent_id)
@@ -95,7 +117,11 @@ $query = "
JOIN public.tbl_person USING (person_id)
JOIN public.tbl_studiengang stg USING (studiengang_kz)
JOIN public.tbl_studiensemester USING (studiensemester_kurzbz)
LEFT JOIN public.tbl_prestudentstatus pss ON (pss.prestudent_id = a.prestudent_id AND pss.studiensemester_kurzbz=a.studiensemester_kurzbz AND pss.status_kurzbz=get_rolle_prestudent(a.prestudent_id, a.studiensemester_kurzbz))
LEFT JOIN public.tbl_prestudentstatus pss ON (
pss.prestudent_id = a.prestudent_id AND
pss.studiensemester_kurzbz=a.studiensemester_kurzbz AND
pss.status_kurzbz=get_rolle_prestudent(a.prestudent_id, a.studiensemester_kurzbz)
)
LEFT JOIN lehre.tbl_studienplan plan USING (studienplan_id)
JOIN bis.tbl_orgform ON (tbl_orgform.orgform_kurzbz = public.get_orgform_prestudent(a.prestudent_id, a.studiensemester_kurzbz))" . $where;