fixed date format timzeone issue since native js date was still in use there; consolidated date helpers to use the luxon approach from abgabetool datehelpers and put them into the generic DateHelpers.js file; also replaced a few other sketchy date formats with the consolidated helpers function;

application/controllers/api/frontend/v1/Studium.php

@@ -167,9 +167,7 @@ class Studium extends FHCAPI_Controller
 		if($aktuelles_studienplan){
 			$lehrveranstaltungen = $this->computeStudienplanLehrveranstaltungen($aktuelles_studienplan->studienplan_id, $aktuelles_semester);
 			foreach($lehrveranstaltungen as $lehrv){
-				// a lehrveranstaltung without a modul has no children and gets its lektoren directly
-				$lvs = isset($lehrv->lehrveranstaltungen) ? $lehrv->lehrveranstaltungen : [$lehrv];
-				foreach($lvs as $lv){
+				foreach($lehrv->lehrveranstaltungen as $lv){
 					$lvLektoren =$this->computeLektorenFromLehrveranstaltung($lv->lehrveranstaltung_id,$aktuelles_semester, $aktuelles_studiengang->studiengang_kz, $aktuelles_studiensemester->studiensemester_kurzbz);
 					$lv->lektoren = $lvLektoren;
 				}
@@ -288,7 +286,6 @@ SELECT tbl_lehrveranstaltung.*,
 		$lehrveranstaltungen = $this->LehrveranstaltungModel->execReadOnlyQuery($query,[$studienplan_id, $semester]);
 		
 		$lehrveranstaltungen = $this->getDataOrTerminateWithError($lehrveranstaltungen);
-		
 		usort($lehrveranstaltungen, function($a, $b){
 			if($a->lehrtyp_kurzbz == "modul"){
 				return -1;
@@ -305,17 +302,12 @@ SELECT tbl_lehrveranstaltung.*,
 			}
 			else{
 				$parent =array_filter($carry, function($item)use($lehrv){
-					return isset($item->lehrveranstaltungen)
-						&& $item->studienplan_lehrveranstaltung_id == $lehrv->studienplan_lehrveranstaltung_id_parent;
+					return $item->studienplan_lehrveranstaltung_id == $lehrv->studienplan_lehrveranstaltung_id_parent;
 				});
 				$parent = current($parent);
 				if($parent){
 					$parent->lehrveranstaltungen[] = $lehrv;
 				}
-				else{
-					// lehrveranstaltung without a modul: keep it on the top level
-					array_push($carry, $lehrv);
-				}
 			}
 			return $carry;
 		}, []);

application/controllers/api/frontend/v1/stv/Grades.php

@@ -198,7 +198,7 @@ class Grades extends FHCAPI_Controller
 		$now = date('c');
 
 		// NOTE(chris): Stg Permissions
-		if (!$this->hasPermissionGradeUID($lehrveranstaltung_id, $student_uid))
+		if (!$this->hasPermissionUpdate($lehrveranstaltung_id, $student_uid))
 			return $this->_outputAuthError([$this->router->method => ['admin', 'assistenz']]);
 
 		$this->load->model('education/Zeugnisnote_model', 'ZeugnisnoteModel');
@@ -260,7 +260,7 @@ class Grades extends FHCAPI_Controller
 		$lehrveranstaltung_id = $this->input->post('lehrveranstaltung_id');
 
 		// NOTE(chris): Stg Permissions
-		if (!$this->hasPermissionGradeUID($lehrveranstaltung_id, $student_uid))
+		if (!$this->hasPermissionDelete($lehrveranstaltung_id, $student_uid))
 			return $this->_outputAuthError([$this->router->method => ['admin', 'assistenz']]);
 
 		$this->load->model('education/Zeugnisnote_model', 'ZeugnisnoteModel');
@@ -298,7 +298,7 @@ class Grades extends FHCAPI_Controller
 		$authUID = getAuthUID();
 		
 		// NOTE(chris): Stg Permissions
-		if (!$this->hasPermissionGradeUID($lehrveranstaltung_id, $student_uid))
+		if (!$this->hasPermissionCopy($lehrveranstaltung_id, $student_uid))
 			return $this->_outputAuthError([$this->router->method => 'student/noten']);
 
 		$this->load->model('education/Lvgesamtnote_model', 'LvgesamtnoteModel');
@@ -410,7 +410,7 @@ class Grades extends FHCAPI_Controller
 		$repeaterGrade = current($repeaterGrade);
 
 		// NOTE(chris): Stg Permissions
-		if (!$this->hasPermissionGradeUID($repeaterGrade->lehrveranstaltung_id, $repeaterGrade->student_uid))
+		if (!$this->hasPermissionCopy($repeaterGrade->lehrveranstaltung_id, $repeaterGrade->student_uid))
 			return $this->_outputAuthError([$this->router->method => 'student/noten']);
 
 		$data = [
@@ -565,57 +565,136 @@ class Grades extends FHCAPI_Controller
 	}
 
 	/**
-	 * Combined helper function to check permissions for
-	 * certificate-modifying functions
+	 * Helper function to check permissions for updateCertificate()
 	 *
 	 * @param integer				$lehrveranstaltung_id
 	 * @param string				$student_uid
 	 *
 	 * @return boolean
 	 */
-	protected function hasPermissionGradeUID($lehrveranstaltung_id, $student_uid)
+	protected function hasPermissionUpdate($lehrveranstaltung_id, $student_uid)
 	{
 		if ($lehrveranstaltung_id === null || $student_uid === null)
 			return true;
 
-
 		$this->load->model('crm/Student_model', 'StudentModel');
 		
-		$studentResult = $this->StudentModel->load([$student_uid]);
-		if (isError($studentResult) || !hasData($studentResult))
+		$result = $this->StudentModel->load([$student_uid]);
+		if (isError($result) || !hasData($result))
 			return false;
 
-		$student = current(getData($studentResult));
+		$student = current(getData($result));
 
+		if ($this->permissionlib->isBerechtigt('admin', 'suid', $student->studiengang_kz))
+			return true;
+		if ($this->permissionlib->isBerechtigt('assistenz', 'suid', $student->studiengang_kz))
+			return true;
 
 		$this->load->model('organisation/Studienplan_model', 'StudienplanModel');
 
-		$oesResult = $this->StudienplanModel->getAllOesForLv($lehrveranstaltung_id);
-		if (isError($oesResult))
+		$result = $this->StudienplanModel->getAllOesForLv($lehrveranstaltung_id);
+		if (isError($result))
 			return false;
 
-		$oes = hasData($oesResult) ? getData($oesResult) : [];
+		$oes = getData($result) ?: [];
 
 		$this->load->model('education/Lehrveranstaltung_model', 'LehrveranstaltungModel');
 
-		$lvResult = $this->LehrveranstaltungModel->load($lehrveranstaltung_id);
-		if (isError($lvResult))
+		$result = $this->LehrveranstaltungModel->getStg($lehrveranstaltung_id);
+		if (isError($result))
 			return false;
 
-		if (hasData($lvResult))
-			$oes[] = current(getData($lvResult));
-
-
-		if ($this->permissionlib->isBerechtigt('student/noten', 'suid', $student->studiengang_kz))
-			return true;
+		if (hasData($result))
+			$oes[] = current(getData($result));
 
 		foreach ($oes as $oe) {
-			if ($this->permissionlib->isBerechtigt('student/noten', 'suid', $oe->oe_kurzbz))
+			if ($this->permissionlib->isBerechtigt('admin', 'suid', $oe->oe_kurzbz))
 				return true;
-			if ($this->permissionlib->isBerechtigt('student/noten', 'suid', $oe->studiengang_kz))
+			if ($this->permissionlib->isBerechtigt('assistenz', 'suid', $oe->oe_kurzbz))
 				return true;
 		}
 
 		return false;
 	}
+
+	/**
+	 * Helper function to check permissions for deleteCertificate()
+	 *
+	 * @param integer				$lehrveranstaltung_id
+	 * @param string				$student_uid
+	 *
+	 * @return boolean
+	 */
+	protected function hasPermissionDelete($lehrveranstaltung_id, $student_uid)
+	{
+		if ($lehrveranstaltung_id === null || $student_uid === null)
+			return true;
+
+		$this->load->model('crm/Student_model', 'StudentModel');
+		
+		$result = $this->StudentModel->load([$student_uid]);
+		if (isError($result) || !hasData($result))
+			return false;
+
+		$student = current(getData($result));
+
+		if ($this->permissionlib->isBerechtigt('admin', 'suid', $student->studiengang_kz))
+			return true;
+		if ($this->permissionlib->isBerechtigt('assistenz', 'suid', $student->studiengang_kz))
+			return true;
+
+		$this->load->model('education/Lehrveranstaltung_model', 'LehrveranstaltungModel');
+
+		$result = $this->LehrveranstaltungModel->load($lehrveranstaltung_id);
+		if (isError($result) || !hasData($result))
+			return false;
+
+		$oe = current(getData($result));
+
+		if ($this->permissionlib->isBerechtigt('admin', 'suid', $oe->oe_kurzbz))
+			return true;
+		if ($this->permissionlib->isBerechtigt('assistenz', 'suid', $oe->oe_kurzbz))
+			return true;
+
+		return false;
+	}
+
+	/**
+	 * Helper function to check permissions for
+	 * copyTeacherProposalToCertificate() and copyRepeaterGradeToCertificate()
+	 *
+	 * @param integer				$lehrveranstaltung_id
+	 * @param string				$student_uid
+	 *
+	 * @return boolean
+	 */
+	protected function hasPermissionCopy($lehrveranstaltung_id, $student_uid)
+	{
+		if ($lehrveranstaltung_id === null || $student_uid === null)
+			return true;
+
+		$this->load->model('crm/Student_model', 'StudentModel');
+		
+		$result = $this->StudentModel->load([$student_uid]);
+		if (isError($result) || !hasData($result))
+			return false;
+		
+		$student = current(getData($result));
+
+		if ($this->permissionlib->isBerechtigt('student/noten', 'suid', $student->studiengang_kz))
+			return true;
+
+		$this->load->model('education/Lehrveranstaltung_model', 'LehrveranstaltungModel');
+		
+		$result = $this->LehrveranstaltungModel->load($lehrveranstaltung_id);
+		if (isError($result) || !hasData($result))
+			return false;
+		
+		$oe = current(getData($result));
+
+		if ($this->permissionlib->isBerechtigt('student/noten', 'suid', $oe->oe_kurzbz))
+			return true;
+
+		return false;
+	}
 }

application/controllers/api/frontend/v1/stv/Students.php

@@ -108,7 +108,6 @@ class Students extends FHCAPI_Controller
 			'studiensemester_kurzbz' => $studiensemester_kurzbz
 		]);
 		
-		$studiensemester_kurzbz = $this->getStudiensemesterKurzbz($studiensemester_kurzbz);
 
 		$this->studentlistlib->addJoin(
 			"(
@@ -154,8 +153,7 @@ class Students extends FHCAPI_Controller
 		$this->addMeta('ci_params', [
 			'studiensemester_kurzbz' => $studiensemester_kurzbz
 		]);
-
-		$studiensemester_kurzbz = $this->getStudiensemesterKurzbz($studiensemester_kurzbz);		
+		
 
 		$this->studentlistlib->addJoin(
 			"(
@@ -215,7 +213,6 @@ class Students extends FHCAPI_Controller
 			'studiensemester_kurzbz' => $studiensemester_kurzbz
 		]);
 		
-		$studiensemester_kurzbz = $this->getStudiensemesterKurzbz($studiensemester_kurzbz);
 
 		$this->studentlistlib->addJoin(
 			"(

application/views/lehre/anrechnung/approveAnrechnungUebersichtData.php

@@ -16,7 +16,7 @@ $query = '
 			anrechnung.begruendung_id,
 			anrechnung.dms_id,
 			CASE
-				WHEN stg.oe_kurzbz IN (' . $ORGANISATIONSEINHEITEN_SCHREIBBERECHTIGT . ') THEN TRUE
+				WHEN stg.typ || stg.kurzbz IN (' . $ORGANISATIONSEINHEITEN_SCHREIBBERECHTIGT . ') THEN TRUE
 				ELSE FALSE
 			END "schreibberechtigt",
 			anrechnung.studiensemester_kurzbz,

application/views/system/infocenter/infocenterDetails.php

@@ -57,7 +57,7 @@
 				<div class="row<?php if ($lockedbyother) echo ' alert-danger' ?>">
 					<div class="col-lg-8">
 						<h3 class="page-header">
-							Infocenter Details: <a target="_blank" title="Studentenverwaltung" href="<?php echo site_url('/studvw/stdsem/' . $studiensemester . '/person/' . $stammdaten->person_id) ?>"><?php echo $stammdaten->vorname.' '.$stammdaten->nachname ?> <i class="fa fa-external-link" style="font-size:small"></i></a>
+							Infocenter Details: <a target="_blank" title="Studentenverwaltung" href="<?php echo site_url('/Studentenverwaltung/' . $studiensemester . '/person/' . $stammdaten->person_id) ?>"><?php echo $stammdaten->vorname.' '.$stammdaten->nachname ?> <i class="fa fa-external-link" style="font-size:small"></i></a>
 						</h3>
 					</div>
 					<div class="col-lg-4">

cis/private/lehre/notenliste.xls.php

@@ -351,7 +351,7 @@ if (!$berechtigung->isBerechtigt('admin')
 					{
 						$worksheet->write($lines,3,$elem->vorname);
 					}
-					$worksheet->writeString($lines,4,$elem->semester.$elem->verband.$elem->gruppe);
+					$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);
 

include/vertrag.class.php

@@ -121,7 +121,7 @@ class vertrag extends basis_db
 											JOIN lehre.tbl_lehreinheit USING(lehreinheit_id)
 										WHERE
 											vertrag_id=tbl_vertrag.vertrag_id
-											AND studiensemester_kurzbz in (SELECT studiensemester_kurzbz FROM public.tbl_studiensemester WHERE start>".$this->db_add_param($datum).")
+											AND studiensemester_kurzbz in (SELECT studiensemester_kurzbz FROM public.tbl_studiensemester WHERE start>=".$this->db_add_param($datum).")
 								)
 						)";
 		}

public/js/apps/Cis/Cis.js

@@ -22,6 +22,7 @@ import OtherLvPlan from "../../components/Cis/LvPlan/OtherLvPlan.js";
 
 import ApiRouteInfo from '../../api/factory/routeinfo.js';
 import {capitalize} from "../../helpers/StringHelpers.js";
+import { getTodayISO, toDateTime } from "../../helpers/DateHelpers.js";
 
 const ciPath = FHC_JS_DATA_STORAGE_OBJECT.app_root.replace(/(https:|)(^|\/\/)(.*?\/)/g, '') + FHC_JS_DATA_STORAGE_OBJECT.ci_router;
 const isMobile = window.matchMedia("(max-width: 767px)").matches;
@@ -88,7 +89,7 @@ const router = VueRouter.createRouter({
 					params: { // in this case always populate other params since they are not optional
 						ort_kurzbz: to.params.ort_kurzbz,
 						mode: isMobile ? DEFAULT_MODE_RAUMINFO_MOBILE : DEFAULT_MODE_RAUMINFO_DESKTOP,
-						focus_date: new Date().toISOString().split("T")[0]
+						focus_date: getTodayISO()
 					},
 				};
 			},
@@ -107,8 +108,7 @@ const router = VueRouter.createRouter({
 					: (isMobile ? DEFAULT_MODE_RAUMINFO_MOBILE : DEFAULT_MODE_RAUMINFO_DESKTOP);
 
 				// default to today date if not provided
-				const d = new Date(route.params.focus_date)
-				const focus_date = !isNaN(d) ? route.params.focus_date : new Date().toISOString().split("T")[0];
+				const focus_date = toDateTime(route.params.focus_date) ? route.params.focus_date : getTodayISO();
 
 				// for consistency reasons format the props into one object but actually use a new name to we dont collide with
 				// existing viewData declaration written from codeigniter 3 into routerview tag
@@ -127,7 +127,7 @@ const router = VueRouter.createRouter({
 						name: "RoomInformation",
 						params: {
 							mode: to.params.mode || DEFAULT_MODE_RAUMINFO,
-							focus_date: to.params.focus_date || new Date().toISOString().split("T")[0],
+							focus_date: to.params.focus_date || getTodayISO(),
 							ort_kurzbz: route.params.ort_kurzbz
 						}
 					});

public/js/components/Cis/Abgabetool/AbgabeMitarbeiterDetail.js

@@ -2,7 +2,7 @@ import BsModal from '../../Bootstrap/Modal.js';
 import VueDatePicker from '../../vueDatepicker.js.php';
 import ApiAbgabe from '../../../api/factory/abgabe.js'
 import { getDateStyleClass } from "./getDateStyleClass.js";
-import {compareISODateValues, formatDateTime, formatISODate, getViennaTodayISO} from "./dateUtils.js";
+import { compareISODateValues, formatDate, formatDateTime, getTodayISO } from "../../../helpers/DateHelpers.js";
 
 
 export const AbgabeMitarbeiterDetail = {
@@ -174,7 +174,7 @@ export const AbgabeMitarbeiterDetail = {
 							'fixtermin': false,
 							'invertedFixtermin': true,
 							'kurzbz': '', // todo kurzbz textfield value vorschlag für qualgates
-							'datum': getViennaTodayISO(),
+							'datum': getTodayISO(),
 							'note': this.allowedNotenOptions.find(opt => opt.note == 9),
 							'beurteilungsnotiz': '',
 							'upload_allowed': false,
@@ -351,9 +351,7 @@ export const AbgabeMitarbeiterDetail = {
 				}
 			}
 		},
-		formatDate(dateParam) {
-			return formatISODate(dateParam)
-		},
+		formatDate,
 		openCreateNewAbgabeModal() {
 			if(this.projektarbeit?.betreuerart_kurzbz == 'Zweitbegutachter') {
 				// maybe alert that this is not allowed or smth
@@ -370,7 +368,7 @@ export const AbgabeMitarbeiterDetail = {
 					'fixtermin': false,
 					'invertedFixtermin': true,
 					'kurzbz': '',
-					'datum': getViennaTodayISO(),
+					'datum': getTodayISO(),
 					'note': this.allowedNotenOptions.find(opt => opt.note == 9),
 					'beurteilungsnotiz': '',
 					'upload_allowed': typ.upload_allowed_default,
@@ -404,7 +402,7 @@ export const AbgabeMitarbeiterDetail = {
 				'fixtermin': false,
 				'invertedFixtermin': true,
 				'kurzbz': '',
-				'datum': getViennaTodayISO(),
+				'datum': getTodayISO(),
 				'note': this.allowedNotenOptions.find(opt => opt.note == 9),
 				'beurteilungsnotiz': '',
 				'upload_allowed': false,
@@ -603,7 +601,7 @@ export const AbgabeMitarbeiterDetail = {
 				'fixtermin': false,
 				'invertedFixtermin': true,
 				'kurzbz': '',
-				'datum': getViennaTodayISO(),
+				'datum': getTodayISO(),
 				'note': this.allowedNotenOptions.find(opt => opt.note == 9),
 				'beurteilungsnotiz': '',
 				'upload_allowed': typ.upload_allowed_default,

public/js/components/Cis/Abgabetool/AbgabeStudentDetail.js

@@ -3,7 +3,7 @@ import BsModal from '../../Bootstrap/Modal.js';
 import VueDatePicker from '../../vueDatepicker.js.php';
 import ApiAbgabe from '../../../api/factory/abgabe.js'
 import FhcOverlay from "../../Overlay/FhcOverlay.js";
-import { formatISODate, getViennaTodayISO } from "./dateUtils.js";
+import { formatDate, getTodayISO } from "../../../helpers/DateHelpers.js";
 import { validateThesisTitle } from './titleValidation.js'
 
 
@@ -248,9 +248,7 @@ export const AbgabeStudentDetail = {
 
 			window.open(FHC_JS_DATA_STORAGE_OBJECT.app_root + FHC_JS_DATA_STORAGE_OBJECT.ci_router + url)
 		},
-		formatDate(dateParam) {
-			return formatISODate(dateParam)
-		},
+		formatDate,
 		async upload(termin) {
 
 			if (! await this.validate(termin))
@@ -286,7 +284,7 @@ export const AbgabeStudentDetail = {
 			if(res.meta.status == "success") {
 				this.$fhcAlert.alertSuccess(this.$capitalize(this.$p.t('abgabetool/c4fileUploadSuccessv3')))
 
-				termin.abgabedatum = getViennaTodayISO();
+				termin.abgabedatum = getTodayISO();
 				if(res?.data?.signatur !== undefined) {
 					termin.signatur = res.data.signatur
 				}

public/js/components/Cis/Abgabetool/AbgabeStudentTimeline.js

@@ -1,5 +1,5 @@
 import AbgabeterminStatusLegende from "./StatusLegende.js";
-import {formatDateTime} from "./dateUtils.js";
+import { formatDateTime } from "../../../helpers/DateHelpers.js";
 
 export const AbgabeStudentTimeline = {
 	name: "AbgabeStudentTimeline",

public/js/components/Cis/Abgabetool/AbgabetoolAssistenz.js

@@ -11,7 +11,7 @@ import AbgabeStudentTimeline from "./AbgabeStudentTimeline.js";
 import { splitMailsHelper } from "../../../helpers/EmailHelpers.js"
 import { getDateStyleClass} from "./getDateStyleClass.js";
 import { dateFilter } from '../../../tabulator/filters/DatesManual.js';
-import { compareISODateValues, formatISODate, getViennaTodayISO, toViennaDate } from "./dateUtils.js";
+import { compareISODateValues, formatDate, getNow, getTodayISO, toDateTime } from "../../../helpers/DateHelpers.js";
 
 export const AbgabetoolAssistenz = {
 	name: "AbgabetoolAssistenz",
@@ -114,7 +114,7 @@ export const AbgabetoolAssistenz = {
 				fixtermin: false,
 			},
 			serienTermin: Vue.reactive({
-				datum: getViennaTodayISO(),
+				datum: getTodayISO(),
 				bezeichnung: {
 					paabgabetyp_kurzbz: 'zwischen',
 					bezeichnung: 'Zwischenabgabe'
@@ -643,7 +643,7 @@ export const AbgabetoolAssistenz = {
 				upload_allowed: false,
 				fixtermin: false,
 			}
-			this.serienEdit.datum = getViennaTodayISO()
+			this.serienEdit.datum = getTodayISO()
 			this.serienEdit.bezeichnung = this.abgabeTypeOptions.find(opt => opt.paabgabetyp_kurzbz === 'zwischen')
 			this.serienEdit.kurzbz = ''
 			this.serienEdit.upload_allowed = false
@@ -1091,23 +1091,9 @@ export const AbgabetoolAssistenz = {
 				return false;
 			}
 
-			const toLuxon = (val) => {
-				if (!val) return null;
-				let dt;
-				if (val instanceof Date) {
-					dt = luxon.DateTime.fromJSDate(val);
-				} else if (typeof val === "string") {
-					dt = toViennaDate(val);
-				} else { // fallback
-					dt = luxon.DateTime.fromMillis(Number(val));
-				}
-
-				return dt.isValid ? dt : null;
-			};
-
-			const rowDate = toLuxon(rowVal);
-			const von = toLuxon(filterVal[0]);
-			const bis = toLuxon(filterVal[1]);
+			const rowDate = toDateTime(rowVal);
+			const von = toDateTime(filterVal[0]);
+			const bis = toDateTime(filterVal[1]);
 
 			// specific day
 			if (von && !bis) {
@@ -1128,22 +1114,8 @@ export const AbgabetoolAssistenz = {
 
 			const rowDate = rowVal.luxonDate;
 
-			const toLuxon = (val) => {
-				if (!val) return null;
-				let dt;
-				if (val instanceof Date) {
-					dt = luxon.DateTime.fromJSDate(val);
-				} else if (typeof val === "string") {
-					dt = toViennaDate(val);
-				} else { // fallback
-					dt = luxon.DateTime.fromMillis(Number(val));
-				}
-
-				return dt.isValid ? dt : null;
-			};
-
-			const von = toLuxon(filterVal[0]);
-			const bis = toLuxon(filterVal[1]);
+			const von = toDateTime(filterVal[0]);
+			const bis = toDateTime(filterVal[1]);
 
 			// specific day
 			if (von && !bis) {
@@ -1372,7 +1344,7 @@ export const AbgabetoolAssistenz = {
 			
 		},
 		checkAbgabetermineProjektarbeit(projekt) {
-			const now = luxon.DateTime.now()
+			const now = getNow()
 			
 			// calculate Abgabetermin time diff to now and assign last and next to projekt
 			projekt.abgabetermine.forEach(termin => {
@@ -1387,7 +1359,7 @@ export const AbgabetoolAssistenz = {
 				// while already looping through each termin, calculate datestyle beforehand
 				termin.dateStyle = getDateStyleClass(termin, this.notenOptions)
 
-				const date = toViennaDate(termin.datum).endOf('day')
+				const date = toDateTime(termin.datum).endOf('day')
 				termin.luxonDate = date
 				termin.diffMs = date.toMillis() - now.toMillis(); // positive = future, negative = past
 
@@ -1654,9 +1626,7 @@ export const AbgabetoolAssistenz = {
 		getNotenFilterOptionLabel(option) {
 			return option.bezeichnung	
 		},
-		formatDate(dateParam) {
-			return formatISODate(dateParam);
-		},
+		formatDate,
 		formAction(cell) {
 			const actionButtons = document.createElement('div');
 			actionButtons.className = "d-flex gap-3";
@@ -1851,9 +1821,7 @@ export const AbgabetoolAssistenz = {
 			return str
 		},
 		isPastDate(date) {
-			const deadline = luxon.DateTime.fromISO(date, { zone: 'Europe/Vienna' }).endOf('day');
-			const nowInVienna = luxon.DateTime.now().setZone('Europe/Vienna');
-			return nowInVienna > deadline;
+			return getNow() > toDateTime(date).endOf("day");
 		},
 		setDetailComponent(details){
 			const pa = this.projektarbeiten.find(projektarbeit => projektarbeit.projektarbeit_id == details.projektarbeit_id)

public/js/components/Cis/Abgabetool/AbgabetoolMitarbeiter.js

@@ -7,7 +7,7 @@ import FhcOverlay from "../../Overlay/FhcOverlay.js";
 import { getDateStyleClass } from "./getDateStyleClass.js";
 import { dateFilter } from '../../../tabulator/filters/DatesManual.js';
 import {splitMailsHelper} from "../../../helpers/EmailHelpers.js";
-import { formatISODate, getViennaTodayISO, toViennaDate } from "./dateUtils.js";
+import { formatDate, getNow, getTodayISO, toDateTime } from "../../../helpers/DateHelpers.js";
 
 export const AbgabetoolMitarbeiter = {
 	name: "AbgabetoolMitarbeiter",
@@ -54,7 +54,7 @@ export const AbgabetoolMitarbeiter = {
 			allowedNotenOptions: null,
 			notenOptionsNonFinal: null,
 			serienTermin: Vue.reactive({
-				datum: getViennaTodayISO(),
+				datum: getTodayISO(),
 				bezeichnung: {
 					paabgabetyp_kurzbz: 'zwischen',
 					bezeichnung: 'Zwischenabgabe'
@@ -668,22 +668,8 @@ export const AbgabetoolMitarbeiter = {
 			
 			const rowDate = rowVal.luxonDate;
 			
-			const toLuxon = (val) => {
-				if (!val) return null;
-				let dt;
-				if (val instanceof Date) {
-					dt = luxon.DateTime.fromJSDate(val);
-				} else if (typeof val === "string") {
-					dt = toViennaDate(val);
-				} else { // fallback
-					dt = luxon.DateTime.fromMillis(Number(val));
-				}
-
-				return dt.isValid ? dt : null;
-			};
-
-			const von = toLuxon(filterVal[0]);
-			const bis = toLuxon(filterVal[1]);
+			const von = toDateTime(filterVal[0]);
+			const bis = toDateTime(filterVal[1]);
 
 			// specific day
 			if (von && !bis) {
@@ -899,14 +885,14 @@ export const AbgabetoolMitarbeiter = {
 			}
 		},
 		checkAbgabetermineProjektarbeit(projekt) {
-			const now = luxon.DateTime.now()
+			const now = getNow()
 			// calculate Abgabetermin time diff to now and assign last and next to projekt
 			projekt.abgabetermine.forEach(termin => {
 				
 				// while already looping through each termin, calculate datestyle beforehand
 				termin.dateStyle = getDateStyleClass(termin, this.notenOptions)
 
-				const date = toViennaDate(termin.datum).endOf('day')
+				const date = toDateTime(termin.datum).endOf('day')
 				termin.luxonDate = date
 				termin.diffMs = date.toMillis() - now.toMillis(); // positive = future, negative = past
 
@@ -1017,9 +1003,7 @@ export const AbgabetoolMitarbeiter = {
 		getOptionLabelAbgabetyp(option){
 			return this.$p.t('abgabetool/c4paatyp' + option.paabgabetyp_kurzbz)
 		},
-		formatDate(dateParam) {
-			return formatISODate(dateParam);
-		},
+		formatDate,
 		undoSelection(cell) {
 			// checks if cells row is selected and unselects -> imitates columns which dont trigger row selection
 			// but actually just revert it after the fact
@@ -1104,9 +1088,7 @@ export const AbgabetoolMitarbeiter = {
 			return str
 		},
 		isPastDate(date) {
-			const deadline = luxon.DateTime.fromISO(date, { zone: 'Europe/Vienna' }).endOf('day');
-			const nowInVienna = luxon.DateTime.now().setZone('Europe/Vienna');
-			return nowInVienna > deadline;
+			return getNow() > toDateTime(date).endOf("day");
 		},
 		setDetailComponent(details){
 			this.loading=true

public/js/components/Cis/Abgabetool/AbgabetoolStudent.js

@@ -4,6 +4,7 @@ import ApiAuthinfo from '../../../api/factory/authinfo.js';
 import BsModal from "../../Bootstrap/Modal.js";
 import FhcOverlay from "../../Overlay/FhcOverlay.js";
 import { getDateStyleClass} from "./getDateStyleClass.js";
+import { getNow, toDateTime } from "../../../helpers/DateHelpers.js";
 import { validateThesisTitle } from './titleValidation.js'
 
 export const AbgabetoolStudent = {
@@ -171,9 +172,7 @@ export const AbgabetoolStudent = {
 			return qgate1positiv && qgate2positiv
 		},
 		isPastDate(date) {
-			const deadline = luxon.DateTime.fromISO(date, { zone: 'Europe/Vienna' }).endOf('day');
-			const nowInVienna = luxon.DateTime.now().setZone('Europe/Vienna');
-			return nowInVienna > deadline;
+			return getNow() > toDateTime(date).endOf("day");
 		},
 		setDetailComponent(details){
 			this.loading = true

public/js/components/Cis/Abgabetool/dateUtils.js

@@ -1,33 +0,0 @@
-const zone = 'Europe/Vienna';
-
-export function getViennaTodayISO() {
-	return luxon.DateTime.now().setZone(zone).toISODate();
-}
-
-export function formatISODate(dateParam) {
-	if (!dateParam) return '';
-
-	const date = luxon.DateTime.fromISO(String(dateParam), { zone });
-	return date.isValid ? date.toFormat('dd.MM.yyyy') : '';
-}
-
-export function formatDateTime(dateParam) {
-	if (!dateParam) return '';
-
-	const date = luxon.DateTime.fromSQL(dateParam);
-	return date.isValid ? date.toFormat("dd.MM.yyyy HH:mm") : '';
-}
-
-export function toViennaDate(dateParam) {
-	if (!dateParam) return null;
-
-	return luxon.DateTime.fromISO(String(dateParam), { zone });
-}
-
-export function compareISODateValues(a, b) {
-	if (!a && !b) return 0;
-	if (!a) return 1;
-	if (!b) return -1;
-
-	return String(a).localeCompare(String(b));
-}

public/js/components/Cis/Abgabetool/getDateStyleClass.js

@@ -1,12 +1,12 @@
 
-const zone = 'Europe/Vienna';
+import { getNow, toDateTime } from "../../../helpers/DateHelpers.js";
 
 export function getDateStyleClass(termin, notenOptions) {
-	const today = luxon.DateTime.now().setZone(zone);
-	const datum = luxon.DateTime.fromISO(termin.datum, { zone }).endOf('day');
-	const abgabedatum = termin.abgabedatum ? luxon.DateTime.fromISO(termin.abgabedatum, { zone }) : null;
-	termin.diffindays = datum.diff(today, 'days').days;
-	const isLate = abgabedatum && abgabedatum > datum;
+	const today = getNow();
+	const datum = toDateTime(termin.datum)?.endOf('day');
+	const abgabedatum = toDateTime(termin.abgabedatum);
+	termin.diffindays = datum ? datum.diff(today, 'days').days : NaN;
+	const isLate = abgabedatum && datum && abgabedatum > datum;
 
 	// GRADE STATUS
 	if (termin.note) {

public/js/components/Cis/Mylv/LvInfo.js

@@ -1,4 +1,4 @@
-import { numberPadding, formatDate } from "../../../helpers/DateHelpers.js"
+import { formatDate, formatTime } from "../../../helpers/DateHelpers.js"
 
 export default {
 	props: {
@@ -25,27 +25,14 @@ export default {
 			return FHC_JS_DATA_STORAGE_OBJECT.app_root + FHC_JS_DATA_STORAGE_OBJECT.ci_router + `/CisVue/Cms/content/${this.event.ort_content_id}`
 		},
 		start_time: function () {
-			if (!this.event.start) return 'N/A';
-			if (!this.event.start instanceof Date) {
-				return this.event.start;
-			}
-			return numberPadding(this.event.start.getHours()) + ":" + numberPadding(this.event.start.getMinutes());
+			return this.event.start ? formatTime(this.event.start) : 'N/A';
 		},
 		end_time: function () {
-			if (!this.event.end) return 'N/A';
-			if (!this.event.end instanceof Date) {
-				return this.event.end;
-			}
-			return numberPadding(this.event.end.getHours()) + ":" + numberPadding(this.event.end.getMinutes());
+			return this.event.end ? formatTime(this.event.end) : 'N/A';
 		}
 	},
 	methods: {
-		mehtodNumberPadding: function (number) {
-			return numberPadding(number);
-		},
-		methodFormatDate: function (d) {
-			return formatDate(d);
-		},
+		methodFormatDate: formatDate,
 	},
 	template:/*html*/`
 		<table class="table table-hover mb-4">

public/js/components/Cis/Mylv/RoomInformation.js

@@ -1,6 +1,7 @@
 import FhcCalendar from "../../Calendar/LvPlan.js";
 
 import ApiLvPlan from '../../../api/factory/lvPlan.js';
+import { getTodayISO } from "../../../helpers/DateHelpers.js";
 
 export const DEFAULT_MODE_RAUMINFO_MOBILE = 'List';
 export const DEFAULT_MODE_RAUMINFO_DESKTOP = 'Week';
@@ -17,7 +18,7 @@ export default {
 	inject: ["isMobile"],
 	computed: {
 		currentDay() {
-			return this.propsViewData?.focus_date || luxon.DateTime.now().setZone(FHC_JS_DATA_STORAGE_OBJECT.timezone).toISODate();
+			return this.propsViewData?.focus_date || getTodayISO();
 		},
 		currentMode() {
 			const defaultMode = this.isMobile ? DEFAULT_MODE_RAUMINFO_MOBILE : DEFAULT_MODE_RAUMINFO_DESKTOP;

public/js/components/Cis/Raumsuche/Raumsuche.js

@@ -1,6 +1,7 @@
 import {CoreFilterCmpt} from "../../../components/filter/Filter.js";
 import VueDatePicker from '../../vueDatepicker.js.php';
 import ApiOrt from '../../../api/factory/ort.js'
+import { getNow, getTodayISO } from "../../../helpers/DateHelpers.js";
 export const Raumsuche =  {
 	name: "Raumsuche",
 	components: {
@@ -10,6 +11,8 @@ export const Raumsuche =  {
 	},
 	inject: ["isMobile"],
 	data() {
+		const now = getNow();
+
 		return {
 			phrasenPromise: null,
 			phrasenResolved: false,
@@ -23,14 +26,14 @@ export const Raumsuche =  {
 			},
 			anzahl: 1,
 			selectedType: null,
-			datum: new Date(),
+			datum: getTodayISO(),
 			von: Vue.ref({
-				hours: new Date().getHours(),
-				minutes: new Date().getMinutes()
+				hours: now.hour,
+				minutes: now.minute
 			}),
 			bis: Vue.ref({
-				hours: new Date().getHours() + 1,
-				minutes: new Date().getMinutes()
+				hours: now.plus({ hours: 1 }).hour,
+				minutes: now.minute
 			}),
 			datepickerTextInputOptions: {
 				enterSubmit: true,
@@ -136,7 +139,7 @@ export const Raumsuche =  {
 			})
 		},
 		loadRooms() {
-			this.$api.call(ApiOrt.getRooms(this.datum.toISOString(), this.getTimeString(this.von), this.getTimeString(this.bis), this.selectedType?.raumtyp_kurzbz ?? '', this.anzahl))
+			this.$api.call(ApiOrt.getRooms(this.datum, this.getTimeString(this.von), this.getTimeString(this.bis), this.selectedType?.raumtyp_kurzbz ?? '', this.anzahl))
 				.then(res => {
 					if(res?.data?.retval) this.setupData(res.data.retval)
 			})
@@ -150,17 +153,6 @@ export const Raumsuche =  {
 		setRoute(val) {
 			// TODO: router push
 		},
-		dateFormat(date) {
-			const day = String(date.getDate()).padStart(2, '0');
-			const month = String(date.getMonth() + 1).padStart(2, '0');
-			const year = date.getFullYear();
-			return `${day}.${month}.${year}`
-		},
-		timeFormat(date) {
-			const hours = String(date.getHours()).padStart(2, '0');
-			const minutes = String(date.getMinutes()).padStart(2, '0');
-			return `${hours}:${minutes}`;
-		},
 		async setupMounted() {
 			
 			this.tableBuiltPromise = new Promise(this.tableResolve)
@@ -199,7 +191,8 @@ export const Raumsuche =  {
 				:dark="isDarkMode"
 				:clearable="false"
 				:enable-time-picker="false"
-				:format="dateFormat"
+				format="dd.MM.yyyy"
+				model-type="yyyy-MM-dd"
 				:text-input="datepickerTextInputOptions"
 				:min-date="new Date()"
 				date-picker
@@ -213,7 +206,7 @@ export const Raumsuche =  {
 				v-model="von"
 				:dark="isDarkMode"
 				:clearable="false"
-				:format="timeFormat"
+				format="HH:mm"
 				:text-input="timepickerTextInputOptions"
 				:is-24="true"
 				time-picker
@@ -227,7 +220,7 @@ export const Raumsuche =  {
 				v-model="bis"
 				:dark="isDarkMode"
 				:clearable="false"
-				:format="timeFormat"
+				format="HH:mm"
 				:text-input="timepickerTextInputOptions"
 				:is-24="true"
 				time-picker

public/js/components/Cis/Studium/Studium.js

@@ -206,8 +206,6 @@ export default {
 			});
 
 			this.lehrveranstaltungen.forEach((lehrveranstaltung)=>{
-				// a lehrveranstaltung without a modul has no children
-				if(!Array.isArray(lehrveranstaltung.lehrveranstaltungen)) return;
 				lehrveranstaltung.lehrveranstaltungen.sort((lv1,lv2)=>{
 					if (lv1.bezeichnung.toLowerCase() > lv2.bezeichnung.toLowerCase()) {
 						return 1;
@@ -395,16 +393,6 @@ export default {
 				</ul>
 			</div>
 		</div>
-		<div class="card" v-else-if="!Array.isArray(lehrveranstaltung.lehrveranstaltungen)">
-			<div class="card-body">
-				<ul class="list-group list-group-flush">
-					<li class="d-flex list-group-item">
-						<a class="fhc-link-color d-block me-auto" href="#" @click="openLvUebersicht(lehrveranstaltung)">{{isGermanLanguage ? lehrveranstaltung.bezeichnung : lehrveranstaltung.bezeichnung_english}}</a>
-						<p>{{lehrveranstaltung.lehrform_kurzbz}}</p>
-					</li>
-				</ul>
-			</div>
-		</div>
 	</template>
 	</div>
 

public/js/components/DashboardWidget/Abstract.js

@@ -1,3 +1,5 @@
+import { formatDate, formatDateTime } from "../../helpers/DateHelpers.js";
+
 export default {
 	props: [
 		"config",
@@ -30,14 +32,7 @@ export default {
 		},
 	},
 	methods: {
-		formatDateTime: function(dateTime) {
-			const dt = new Date(dateTime);
-			return dt.getDate() + '.' + (dt.getMonth()+1) + '.' + dt.getFullYear() + ' | '
-				+ dt.getHours() + ':' + dt.getMinutes();
-		},
-		getDate: function(dateTime){
-			const dt = new Date(dateTime);
-			return dt.getDate() + '.' + (dt.getMonth()+1) + '.' + dt.getFullYear();
-		}
+		formatDateTime,
+		getDate: formatDate
 	}
 }

public/js/components/DashboardWidget/News.js

@@ -1,6 +1,6 @@
 import AbstractWidget from './Abstract.js';
 import BsModal from '../Bootstrap/Modal.js';
-import { numberPadding } from '../../helpers/DateHelpers.js';
+import { formatDate, formatTime } from '../../helpers/DateHelpers.js';
 import ApiCms from '../../api/factory/cms.js';
 
 const MAX_LOADED_NEWS = 30;
@@ -84,14 +84,8 @@ export default {
 				});
 			})
 		},
-		formatDate: function (dateTime) {
-			const dt = new Date(dateTime);
-			return numberPadding(dt.getDate()) + '.' + numberPadding((dt.getMonth() + 1)) + '.' + dt.getFullYear();				
-		},
-		formatTime: function (dateTime) {
-			const dt = new Date(dateTime);
-			return numberPadding(dt.getHours()) + ':' + numberPadding(dt.getMinutes());
-		},
+		formatDate,
+		formatTime,
 		isString(value){
 			return Object.prototype.toString.call(value) === '[object String]';
 		},

public/js/components/Stv/Studentenverwaltung/Details/Noten/Zeugnis.js

@@ -255,7 +255,7 @@ export default {
 							deleteButton.className = 'btn btn-outline-secondary';
 							const icon = document.createElement('i');
 							icon.className = 'fa fa-trash';
-							icon.title = this.$p.t('lehre/info_deleteMarkEntry');
+							icon.title = this.$p.t('ui/loeschen');
 							deleteButton.append(icon);
 							deleteButton.addEventListener('click', evt => {
 								evt.stopPropagation();

public/js/helpers/DateHelpers.js

@@ -1,61 +1,69 @@
 
-// HELPER FILE -- that contains multiple functions which create, handle and manipulate js dates
+// Everything parses and formats with luxon in the server timezone. new Date("2026-09-02")
+// reads UTC midnight and shows the day before west of Greenwich.
 
-// custom Error class for DateHelpers.js misuse
-class DateHelperError extends Error {
-	constructor(message) {
-		super(message);
-		this.name = "DateHelperError";
-	}
+
+function getZone() {
+	return globalThis.FHC_JS_DATA_STORAGE_OBJECT?.timezone || 'Europe/Vienna';
 }
 
 /**
- * adds padding to a number 
+ * converts a date value into a luxon DateTime in the server timezone
  *
- * @param {number|string} number - the number on which padding should be added.
- * @returns {string} number with padding.
+ * @param {string|number|Date|Object} value - ISO string, SQL string, milliseconds, Date or luxon DateTime.
+ * @returns {Object|null} luxon DateTime, or null if the value is empty or unparsable.
  */
+export function toDateTime(value) {
+	if (!value) return null;
+
+	const zone = getZone();
+	let date;
+
+	if (value instanceof luxon.DateTime) date = value.setZone(zone);
+	else if (value instanceof Date) date = luxon.DateTime.fromJSDate(value, { zone });
+	else if (typeof value === 'number') date = luxon.DateTime.fromMillis(value, { zone });
+	// fromISO rejects the space that SQL puts between date and time
+	else if (typeof value === 'string') date = value.includes(' ')
+		? luxon.DateTime.fromSQL(value, { zone })
+		: luxon.DateTime.fromISO(value, { zone });
+	else return null;
+
+	return date.isValid ? date : null;
+}
+
+// current time in the server timezone
+export function getNow() {
+	return luxon.DateTime.now().setZone(getZone());
+}
+
+// today in the server timezone, as yyyy-MM-dd
+export function getTodayISO() {
+	return getNow().toISODate();
+}
+
+// the three formatters return '' for an empty or unparsable value
+export function formatDate(value) {
+	return toDateTime(value)?.toFormat('dd.MM.yyyy') ?? '';
+}
+
+export function formatTime(value) {
+	return toDateTime(value)?.toFormat('HH:mm') ?? '';
+}
+
+export function formatDateTime(value) {
+	return toDateTime(value)?.toFormat('dd.MM.yyyy HH:mm') ?? '';
+}
+
+// tabulator sorter for ISO dates, sorts empty values last
+export function compareISODateValues(a, b) {
+	if (!a && !b) return 0;
+	if (!a) return 1;
+	if (!b) return -1;
+
+	return String(a).localeCompare(String(b));
+}
+
+// pads hours and minutes to two digits
 export function numberPadding(number) {
-	if(typeof number !== "string" && typeof number !== "number")
-	{
-		throw new TypeError("function numberPadding in file DateHelpers.js is only usable with strings or numbers");
-	}
-	if(number.toString().length > 2) 
-	{
-		throw new DateHelperError("The number on which the padding should be added should not be longer than to 2 characters, please refere to the function numberPadding in the helper file DateHelpers.js");
-	}
-	return number.toString().length == 1 ? '0' + number.toString() : number.toString();
-}
-
-/**
- * formats date to dd.mm.yyyy
- *
- * @param {string|Date} d - the date that should be formatted.
- * @returns {string} formatted date string.
- */
-export function formatDate(d) {
-	// parameter is of type Date
-	if(d instanceof Date)
-	{
-		if (isNaN(date.valueOf())) {
-			return 'N/A';
-		}
-		// if the date is an invalid string then creating a date from the string will fail and N/A is returned
-		return `${numberPadding(d.getDate())}.${numberPadding(d.getMonth() + 1)}.${d.getFullYear()}`;
-	}
-	// parameter is of type string
-	else if (typeof d === "string")
-	{
-		let date = new Date(d);
-		// if the date is an invalid string then creating a date from the string will fail and N/A is returned
-		if (isNaN(date.valueOf())) {
-			return 'N/A';
-		}
-		return `${numberPadding(date.getDate())}.${numberPadding(date.getMonth() + 1)}.${date.getFullYear()}`;
-	}
-	// parameter is not of type string or Date and an exception is thrown
-	else
-	{
-		throw new TypeError("The parameter provided for this function is not a string or a Date object, please refere to the function formatDate in the DateHelpers.js file");
-	}
+	return String(number).padStart(2, '0');
 }

system/phrasesupdate.php

@@ -62411,26 +62411,6 @@ I have been informed that I am under no obligation to consent to the transmissio
 			)
 		)
 	),
-	array(
-		'app' => 'core',
-		'category' => 'lehre',
-		'phrase' => 'info_deleteMarkEntry',
-		'insertvon' => 'system',
-		'phrases' => array(
-			array(
-				'sprache' => 'German',
-				'text' => 'Noteneintrag entfernen',
-				'description' => '',
-				'insertvon' => 'system'
-			),
-			array(
-				'sprache' => 'English',
-				'text' => 'Remove grade entry',
-				'description' => '',
-				'insertvon' => 'system'
-			)
-		)
-	),
 );