From 4d1994ca63ca912c5e395ec7a5b17faf9c2eb31c Mon Sep 17 00:00:00 2001 From: Harald Bamberger Date: Tue, 2 Nov 2021 17:50:20 +0100 Subject: [PATCH 1/3] menulayout, coding styles --- cis/private/lehre/anwesenheitsliste.php | 25 +++++++++++++++++-------- cis/private/lehre/covidstatusliste.php | 9 ++++++--- include/covid/covidhelper.class.php | 6 ++++-- include/lehrelisthelper.class.php | 9 ++++++--- locale/de-AT/anwesenheitsliste.php | 1 + locale/en-US/anwesenheitsliste.php | 1 + 6 files changed, 35 insertions(+), 16 deletions(-) diff --git a/cis/private/lehre/anwesenheitsliste.php b/cis/private/lehre/anwesenheitsliste.php index 6859c1a2e..0eda67a65 100644 --- a/cis/private/lehre/anwesenheitsliste.php +++ b/cis/private/lehre/anwesenheitsliste.php @@ -96,6 +96,7 @@ $covidhelper = new CovidHelper(); isUdfDefined()) ? 'Covid-Status' : ''; - $aw_content .= "".$p->t('anwesenheitsliste/gesamtliste')." $lv->bezeichnung" . $covid_content . ""; + $covid_content .= ($covidhelper->isUdfDefined()) ? '' . $p->t('anwesenheitsliste/gesamtliste') . ' ' . $lv->bezeichnung . '' : ''; + $aw_content .= "".$p->t('anwesenheitsliste/gesamtliste')." $lv->bezeichnung"; $awbild_content .= "".$p->t('anwesenheitsliste/gesamtliste')." $lv->bezeichnung"; $nt_content .= "".$p->t('anwesenheitsliste/gesamtliste')." $lv->bezeichnung"; @@ -164,8 +165,8 @@ $covidhelper = new CovidHelper(); } } - $covid_content = ($covidhelper->isUdfDefined()) ? 'Covid-Status' : ''; - $aw_content .= "   $kurzbz - $lehrform - $gruppen ($lektoren)" . $covid_content . ""; + $covid_content .= ($covidhelper->isUdfDefined()) ? '   ' . $kurzbz . ' - ' . $lehrform . ' - ' . $gruppen . ' (' . $lektoren . ')' : ''; + $aw_content .= "   $kurzbz - $lehrform - $gruppen ($lektoren)"; $awbild_content .= "   $kurzbz - $lehrform - $gruppen ($lektoren)"; $nt_content .= "   $kurzbz - $lehrform - $gruppen ($lektoren)"; @@ -200,8 +201,8 @@ $covidhelper = new CovidHelper(); } } - $covid_content = ($covidhelper->isUdfDefined()) ? 'Covid-Status' : ''; - $aw_content .= "   $kurzbz - $lehrform - $gruppen ($lektoren)" . $covid_content . ""; + $covid_content .= ($covidhelper->isUdfDefined()) ? '   ' . $kurzbz . ' - ' . $lehrform . ' - ' . $gruppen . ' (' . $lektoren . ')' : ''; + $aw_content .= "   $kurzbz - $lehrform - $gruppen ($lektoren)"; $awbild_content .= "   $kurzbz - $lehrform - $gruppen ($lektoren)"; $nt_content .= "   $kurzbz - $lehrform - $gruppen ($lektoren)"; } @@ -213,6 +214,11 @@ $covidhelper = new CovidHelper(); } else { + if($covid_content!='') + $covid_content = "".$covid_content."

".$p->t('anwesenheitsliste/covidstatuslisten')."

"; + else + $covid_content = $p->t('anwesenheitsliste/keineStudentenVorhanden'); + if($aw_content!='') $aw_content = "".$aw_content."

".$p->t('anwesenheitsliste/anwesenheitslisten')."

"; else @@ -236,17 +242,20 @@ $covidhelper = new CovidHelper(); $aw_content=''; if(defined('CIS_ANWESENHEITSLISTE_ANWESENHEITSLISTE_BILD_ANZEIGEN') && !CIS_ANWESENHEITSLISTE_ANWESENHEITSLISTE_BILD_ANZEIGEN) $awbild_content=''; - + echo " - + + + +
$aw_content$covid_content
$awbild_content
$nt_content
"; diff --git a/cis/private/lehre/covidstatusliste.php b/cis/private/lehre/covidstatusliste.php index 8e9b07fa1..85fbb643e 100644 --- a/cis/private/lehre/covidstatusliste.php +++ b/cis/private/lehre/covidstatusliste.php @@ -113,8 +113,10 @@ header('Content-Type: text/html; charset=utf-8');
diff --git a/include/covid/covidhelper.class.php b/include/covid/covidhelper.class.php index 3c9ef12da..0ac546689 100644 --- a/include/covid/covidhelper.class.php +++ b/include/covid/covidhelper.class.php @@ -52,11 +52,13 @@ class CovidHelper extends basis_db $this->checkIfUdfValuesAreDefined(); } - public function isUdfDefined() { + public function isUdfDefined() + { return $this->isUdfDefined; } - public function fetchCovidStatus(array $uids) { + public function fetchCovidStatus(array $uids) + { $this->uids = $uids; $this->covidstatus = array(); $this->fetchCovidValidStatus(); diff --git a/include/lehrelisthelper.class.php b/include/lehrelisthelper.class.php index 619b112f1..631d6fb86 100644 --- a/include/lehrelisthelper.class.php +++ b/include/lehrelisthelper.class.php @@ -85,7 +85,8 @@ class LehreListHelper return $this->studentuids; } - public function getArr_Lehrende() { + public function getArr_Lehrende() + { return $this->arr_lehrende; } @@ -175,7 +176,8 @@ class LehreListHelper ); } - protected function loadLehrende() { + protected function loadLehrende() + { //Lehrende der LV laden und in ein Array schreiben $lehrende = new lehreinheitmitarbeiter(); $lehrende->getMitarbeiterLV($this->lvid, $this->studiensemester, $this->lehreinheit); @@ -193,7 +195,8 @@ class LehreListHelper } } - protected function loadStudierende() { + protected function loadStudierende() + { //Studierende der LV laden und in ein Array schreiben $qry = 'SELECT diff --git a/locale/de-AT/anwesenheitsliste.php b/locale/de-AT/anwesenheitsliste.php index d6d98226d..977649a0a 100644 --- a/locale/de-AT/anwesenheitsliste.php +++ b/locale/de-AT/anwesenheitsliste.php @@ -1,6 +1,7 @@ phrasen['anwesenheitsliste/studiensemesterIstUngueltig']='Studiensemester ist ungueltig'; $this->phrasen['anwesenheitsliste/anwesenheitslisten']='Anwesenheitslisten'; +$this->phrasen['anwesenheitsliste/covidstatuslisten']='Covid-Statuslisten'; $this->phrasen['anwesenheitsliste/notenlisten']='Notenlisten'; $this->phrasen['anwesenheitsliste/notenliste']='Notenliste'; $this->phrasen['anwesenheitsliste/anwesenheitslistenMitBildern']='Fotolisten'; diff --git a/locale/en-US/anwesenheitsliste.php b/locale/en-US/anwesenheitsliste.php index 1fc5de993..b5205dfbe 100644 --- a/locale/en-US/anwesenheitsliste.php +++ b/locale/en-US/anwesenheitsliste.php @@ -1,6 +1,7 @@ phrasen['anwesenheitsliste/studiensemesterIstUngueltig']='No valid semester'; $this->phrasen['anwesenheitsliste/anwesenheitslisten']='Attendance lists'; +$this->phrasen['anwesenheitsliste/covidstatuslisten']='Covid status lists'; $this->phrasen['anwesenheitsliste/notenlisten']='Grade lists'; $this->phrasen['anwesenheitsliste/notenliste']='Grade list'; $this->phrasen['anwesenheitsliste/anwesenheitslistenMitBildern']='Picture lists'; From 811b59ef1b485171b755461878e4f3c2429e5284 Mon Sep 17 00:00:00 2001 From: Harald Bamberger Date: Tue, 2 Nov 2021 17:56:33 +0100 Subject: [PATCH 2/3] styling --- cis/private/lehre/anwesenheitsliste.php | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/cis/private/lehre/anwesenheitsliste.php b/cis/private/lehre/anwesenheitsliste.php index 0eda67a65..25942ebae 100644 --- a/cis/private/lehre/anwesenheitsliste.php +++ b/cis/private/lehre/anwesenheitsliste.php @@ -71,14 +71,9 @@ $covidhelper = new CovidHelper(); - @@ -247,7 +242,7 @@ $covidhelper = new CovidHelper(); $aw_content - $covid_content + $covid_content $awbild_content From 8d4a79f9b93b6bc7de320233a9b2b567b9594be5 Mon Sep 17 00:00:00 2001 From: Harald Bamberger Date: Tue, 2 Nov 2021 18:00:41 +0100 Subject: [PATCH 3/3] only show messages if covid udf is defined --- cis/private/lehre/anwesenheitsliste.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cis/private/lehre/anwesenheitsliste.php b/cis/private/lehre/anwesenheitsliste.php index 25942ebae..2cdf56d5f 100644 --- a/cis/private/lehre/anwesenheitsliste.php +++ b/cis/private/lehre/anwesenheitsliste.php @@ -212,7 +212,7 @@ $covidhelper = new CovidHelper(); if($covid_content!='') $covid_content = "".$covid_content."

".$p->t('anwesenheitsliste/covidstatuslisten')."

"; else - $covid_content = $p->t('anwesenheitsliste/keineStudentenVorhanden'); + $covid_content = ($covidhelper->isUdfDefined()) ? $p->t('anwesenheitsliste/keineStudentenVorhanden') : ''; if($aw_content!='') $aw_content = "".$aw_content."

".$p->t('anwesenheitsliste/anwesenheitslisten')."

";