Fixed: delete/change restrictions in FAS now ONLY when config var is true

Before deleting/changing of lectors was disabled, no matter if the
regarding config entry was set to true or false. This is fixed now.
This commit is contained in:
Cris
2019-12-11 11:52:16 +01:00
committed by hainberg
parent e35e4f0398
commit d5df8d8584
4 changed files with 21 additions and 18 deletions
+5 -2
View File
@@ -693,8 +693,11 @@ if(!$error)
//Lehreinheitmitarbeiterzuteilung loeschen //Lehreinheitmitarbeiterzuteilung loeschen
if(isset($_POST['lehreinheit_id']) && is_numeric($_POST['lehreinheit_id']) && isset($_POST['mitarbeiter_uid'])) if(isset($_POST['lehreinheit_id']) && is_numeric($_POST['lehreinheit_id']) && isset($_POST['mitarbeiter_uid']))
{ {
// Wenn der Mitarbeiter schon einen Vertrag hat, wird das Loeschen verhindert // Wenn der Mitarbeiter schon einen Vertrag hat UND
if (isset($_POST['vertrag_id']) && is_numeric($_POST['vertrag_id'])) // der config Eintrag zum Anzeigen der Vertragsdetails true ist,
// wird das Loeschen verhindert
if (isset($_POST['vertrag_id']) && is_numeric($_POST['vertrag_id']) &&
(defined('FAS_LV_LEKTORINNENZUTEILUNG_VERTRAGSDETAILS_ANZEIGEN') && FAS_LV_LEKTORINNENZUTEILUNG_VERTRAGSDETAILS_ANZEIGEN))
{ {
$return = false; $return = false;
$errormsg = 'Löschen nur nach Stornierung des Vertrags möglich.'; $errormsg = 'Löschen nur nach Stornierung des Vertrags möglich.';
@@ -43,6 +43,10 @@ var lehrveranstaltungLvGesamtNotenSelectUID=null; //LehreinheitID des Noten Eint
var lehrveranstaltungNotenTreeloaded=false; var lehrveranstaltungNotenTreeloaded=false;
var lehrveranstaltungGesamtNotenTreeloaded=false; var lehrveranstaltungGesamtNotenTreeloaded=false;
var LehrveranstaltungAusbildungssemesterFilter=''; var LehrveranstaltungAusbildungssemesterFilter='';
// Config-Eintrag, ob Vertragsdetails angezeigt werden sollen
var lehrveranstaltung_vertragsdetails_anzeigen = Boolean(<?php echo (defined('FAS_LV_LEKTORINNENZUTEILUNG_VERTRAGSDETAILS_ANZEIGEN') && FAS_LV_LEKTORINNENZUTEILUNG_VERTRAGSDETAILS_ANZEIGEN) ? true : false ?>);
// ********** Observer und Listener ************* // // ********** Observer und Listener ************* //
// **** // ****
@@ -1331,12 +1335,9 @@ function LeMitarbeiterAuswahl()
document.getElementById('lehrveranstaltung-lehreinheitmitarbeiter-checkbox-bismelden').checked=false; document.getElementById('lehrveranstaltung-lehreinheitmitarbeiter-checkbox-bismelden').checked=false;
var gesamtkosten = LeMitarbeiterGesamtkosten(); var gesamtkosten = LeMitarbeiterGesamtkosten();
// Prüfe ob Vertragsdetails angezeigt werden
var vertragsdetails_anzeigen = new Boolean(<?php echo (defined('FAS_LV_LEKTORINNENZUTEILUNG_VERTRAGSDETAILS_ANZEIGEN') && FAS_LV_LEKTORINNENZUTEILUNG_VERTRAGSDETAILS_ANZEIGEN) ? 'true' : 'false' ?>);
// Wenn Vertragsdetails angezeigt werden // Wenn Vertragsdetails angezeigt werden
if (vertragsdetails_anzeigen) { if (lehrveranstaltung_vertragsdetails_anzeigen) {
// Reset attributes // Reset attributes
document.getElementById('lehrveranstaltung-lehreinheitmitarbeiter-label-vertragsstatus').setAttribute("style", "font-weight: normal"); document.getElementById('lehrveranstaltung-lehreinheitmitarbeiter-label-vertragsstatus').setAttribute("style", "font-weight: normal");
+2 -1
View File
@@ -4032,7 +4032,8 @@ if(!$error)
else else
{ {
// Wenn der Projektbetreuer schon einen Vertrag hat, wird das Loeschen verhindert // Wenn der Projektbetreuer schon einen Vertrag hat, wird das Loeschen verhindert
if (isset($_POST['vertrag_id']) && is_numeric($_POST['vertrag_id'])) if (isset($_POST['vertrag_id']) && is_numeric($_POST['vertrag_id']) &&
(defined('FAS_STUDIERENDE_PROJEKTARBEIT_VERTRAGSDETAILS_ANZEIGEN') && FAS_STUDIERENDE_PROJEKTARBEIT_VERTRAGSDETAILS_ANZEIGEN))
{ {
$return = false; $return = false;
$errormsg = 'Löschen nur nach Stornierung des Vertrags möglich.'; $errormsg = 'Löschen nur nach Stornierung des Vertrags möglich.';
+8 -10
View File
@@ -31,7 +31,9 @@ var StudentProjektbetreuerTreeDatasource=null; //Datasource des Projektbetreuer
var StudentProjektbetreuerSelectPersonID=null; var StudentProjektbetreuerSelectPersonID=null;
var StudentProjektbetreuerSelectProjektarbeitID=null; var StudentProjektbetreuerSelectProjektarbeitID=null;
var StudentProjektbetreuerSelectBetreuerartKurzbz=null; var StudentProjektbetreuerSelectBetreuerartKurzbz=null;
vertragsdetails_anzeigen = new Boolean(<?php echo (defined('FAS_STUDIERENDE_PROJEKTARBEIT_VERTRAGSDETAILS_ANZEIGEN') && FAS_STUDIERENDE_PROJEKTARBEIT_VERTRAGSDETAILS_ANZEIGEN) ? 'true' : 'false' ?>);
// Config-Eintrag, ob Vertragsdetails angezeigt werden sollen
var projektarbeit_vertragsdetails_anzeigen = Boolean(<?php echo (defined('FAS_STUDIERENDE_PROJEKTARBEIT_VERTRAGSDETAILS_ANZEIGEN') && FAS_STUDIERENDE_PROJEKTARBEIT_VERTRAGSDETAILS_ANZEIGEN) ? true : false ?>);
// ********** Observer und Listener ************* // // ********** Observer und Listener ************* //
@@ -117,7 +119,7 @@ var StudentProjektbetreuerTreeListener =
//ohne dem setTimeout //ohne dem setTimeout
window.setTimeout(StudentProjektbetreuerTreeSelectID,10); window.setTimeout(StudentProjektbetreuerTreeSelectID,10);
if(vertragsdetails_anzeigen) if(projektarbeit_vertragsdetails_anzeigen)
{ {
var mindEinVertragExistiert = StudentProjektbetreuer_VertragExistiert(); var mindEinVertragExistiert = StudentProjektbetreuer_VertragExistiert();
@@ -181,10 +183,9 @@ function StudentProjektarbeitTreeLoad(uid)
tree.builder.addListener(StudentProjektarbeitTreeListener); tree.builder.addListener(StudentProjektarbeitTreeListener);
StudentProjektarbeitDisableFields(false); StudentProjektarbeitDisableFields(false);
var vertragsdetails_anzeigen = new Boolean(<?php echo (defined('FAS_STUDIERENDE_PROJEKTARBEIT_VERTRAGSDETAILS_ANZEIGEN') && FAS_STUDIERENDE_PROJEKTARBEIT_VERTRAGSDETAILS_ANZEIGEN) ? 'true' : 'false' ?>);
// Wenn Vertragsdetails angezeigt werden // Wenn Vertragsdetails angezeigt werden
if (vertragsdetails_anzeigen) { if (projektarbeit_vertragsdetails_anzeigen) {
//Reset attributes //Reset attributes
document.getElementById('student-projektbetreuer-label-vertragsstatus').value = ''; document.getElementById('student-projektbetreuer-label-vertragsstatus').value = '';
document.getElementById('student-projektbetreuer-label-vertragsstunden').value = ''; document.getElementById('student-projektbetreuer-label-vertragsstunden').value = '';
@@ -891,12 +892,9 @@ function StudentProjektbetreuerAuswahl()
document.getElementById('student-projektbetreuer-checkbox-neu').checked=false; document.getElementById('student-projektbetreuer-checkbox-neu').checked=false;
var gesamtkosten = StudentProjektbetreuerGesamtkosten(); var gesamtkosten = StudentProjektbetreuerGesamtkosten();
// Prüfe ob Vertragsdetails angezeigt werden
var vertragsdetails_anzeigen = new Boolean(<?php echo (defined('FAS_STUDIERENDE_PROJEKTARBEIT_VERTRAGSDETAILS_ANZEIGEN') && FAS_STUDIERENDE_PROJEKTARBEIT_VERTRAGSDETAILS_ANZEIGEN) ? 'true' : 'false' ?>);
// Wenn Vertragsdetails angezeigt werden // Wenn Vertragsdetails angezeigt werden
if (vertragsdetails_anzeigen) { if (projektarbeit_vertragsdetails_anzeigen) {
//Reset attributes //Reset attributes
document.getElementById('student-projektbetreuer-label-vertragsstatus').setAttribute("style", "font-weight: normal"); document.getElementById('student-projektbetreuer-label-vertragsstatus').setAttribute("style", "font-weight: normal");
document.getElementById('student-projektbetreuer-button-vertrag-stornieren').setAttribute("tooltiptext", document.getElementById('student-projektbetreuer-button-vertrag-stornieren').setAttribute("tooltiptext",