From ef631e328db28cf8f058955dcf2485e72ea27e94 Mon Sep 17 00:00:00 2001 From: Alexei Date: Wed, 25 Mar 2020 16:16:30 +0100 Subject: [PATCH 1/3] vilesci/stammdaten/auswertung_fhtw: - fixed bug where no results are shown when filtering by other criteria than reihungstestids --- vilesci/stammdaten/auswertung_fhtw.php | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/vilesci/stammdaten/auswertung_fhtw.php b/vilesci/stammdaten/auswertung_fhtw.php index 4d82d0603..ad80d3454 100644 --- a/vilesci/stammdaten/auswertung_fhtw.php +++ b/vilesci/stammdaten/auswertung_fhtw.php @@ -1120,7 +1120,7 @@ if ($prestudent_id != '' && !is_numeric($prestudent_id)) { die('PrestudentID ist ungueltig'); } -if (($reihungstest == '' && isset($_REQUEST['reihungstest'])) && $studiengang == '' && $semester == '' && $prestudent_id == '' && $datum_von == '' && $datum_bis == '') +if (isset($_POST['rtauswsubmit']) && $reihungstest == '' && $studiengang == '' && $semester == '' && $prestudent_id == '' && $datum_von == '' && $datum_bis == '') { die('Waehlen Sie bitte mindestens eine der Optionen aus'); } @@ -1181,7 +1181,7 @@ while ($row = $db->db_fetch_object($result)) $rtest[$row->reihungstest_id]->freigeschaltet = $db->db_parse_bool($row->freigeschaltet); } -if (isset($_REQUEST['reihungstest'])) +if (isset($_REQUEST['reihungstest']) || isset($_POST['rtauswsubmit'])) { // Vorkommende Gebiete laden $query = " @@ -2693,7 +2693,7 @@ else echo 'PrestudentIn: '; echo ' '; - echo '

'; + echo '

'; echo ''; - if (isset($_REQUEST['reihungstest'])) + if (isset($_REQUEST['reihungstest']) || isset($_POST['rtauswsubmit'])) { echo ' From 54c23570164f6ca115a4c45d3758a4b60effad99 Mon Sep 17 00:00:00 2001 From: Alexei Date: Wed, 25 Mar 2020 18:50:50 +0100 Subject: [PATCH 2/3] vilesci/stammdaten/auswertung_fhtw: - preselect RTs of current date if no selection - fixed prestudent_id selection event bug --- vilesci/stammdaten/auswertung_fhtw.php | 27 +++++++++++++++++++++++++- 1 file changed, 26 insertions(+), 1 deletion(-) diff --git a/vilesci/stammdaten/auswertung_fhtw.php b/vilesci/stammdaten/auswertung_fhtw.php index ad80d3454..35afe98f0 100644 --- a/vilesci/stammdaten/auswertung_fhtw.php +++ b/vilesci/stammdaten/auswertung_fhtw.php @@ -2071,6 +2071,19 @@ else } }); + $("#prestudent").on("input", function(){ + var numchecked = $("#rtcheckboxes input[type=checkbox]:checked").length; + if (numchecked > 0) + { + $("#rtcheckboxes input[type=checkbox]").prop("checked", false); + showSelectedRts(); + } + }); + + $("#prestudent").keyup( function(){ + $("#prestudent_id").val(this.value); + }); + $("#zuteilungAutocomplete").autocomplete({ source: "auswertung_fhtw.php?autocomplete=prestudentAdd&'.http_build_query(array('studiensemester_kurzbz' => $rtStudiensemester)).'", minLength:2, @@ -2601,6 +2614,7 @@ else $selectedrtstr = ''; $checkbxstr = ''; $first = true; + $noparamsselected = $prestudent_id == '' && $reihungstest == '' && $datum_von == '' && $datum_bis == '' && $studiengang == '' && $semester == ''; //$maxeachline = 1; foreach ($rtest as $rt) { @@ -2624,6 +2638,17 @@ else } } } + elseif($noparamsselected && $rt->datum == date('Y-m-d')) + { + //wenn nichts ausgewählt, heute Reihungstests vorausgewählt + $checked = ' checked'; + + if (!$first) + $selectedrtstr .= '
'; + + $selectedrtstr .= $rtstr; + $first = false; + } $checkbxstr .= '
'; $checkbxstr .= '
'; @@ -2690,7 +2715,7 @@ else echo ''; echo ''; echo '
'; - echo 'PrestudentIn: '; + echo 'PrestudentIn: '; echo '
'; echo '

'; From 5b9c1f7cccc3e81d2eca14640e998b56887c1b3f Mon Sep 17 00:00:00 2001 From: Alexei Date: Wed, 25 Mar 2020 20:34:14 +0100 Subject: [PATCH 3/3] vilesci/stammdaten/auswertung_fhtw: - correct order of RT beendet text in Testende mail --- vilesci/stammdaten/auswertung_fhtw.php | 30 +++++++++++++++++++++----- 1 file changed, 25 insertions(+), 5 deletions(-) diff --git a/vilesci/stammdaten/auswertung_fhtw.php b/vilesci/stammdaten/auswertung_fhtw.php index 35afe98f0..71714f169 100644 --- a/vilesci/stammdaten/auswertung_fhtw.php +++ b/vilesci/stammdaten/auswertung_fhtw.php @@ -482,12 +482,8 @@ if ($rtFreischalten) $testende = filter_input(INPUT_POST, 'testende', FILTER_VALIDATE_BOOLEAN); if ($testende) { - /*if (isset($_POST['reihungstest_ids']) && is_array($_POST['reihungstest_ids'])) - { - $reihungstest = new reihungstest($_POST['reihungstest_id']);*/ // Alle Bachelor-Studiengänge holen, bei denen der Bewerber Interessent ist, die Bewerbung abgeschickt hat und bestätigt wurde // Mail an alle diese Studiengänge senden - if (isset($_POST['prestudents'])) { // Array mit allen Prestudenten aufbauen @@ -540,6 +536,7 @@ if ($testende) $sendError = false; $empfaengerArray = array(); $rtidArray = array(); + $rtdatumstrArray = array(); $rtdatumstr = ''; foreach ($prestudentsrt as $psrt) @@ -548,11 +545,34 @@ if ($testende) { $rtidArray[] = $psrt['reihungstest_id']; $rt = new reihungstest($psrt['reihungstest_id']); + $idx = 0; - $rtdatumstr .= 'Der Reihungstest vom '.$datum_obj->convertISODate($rt->datum).' um '.$datum_obj->formatDatum($rt->uhrzeit, 'H:i').' Uhr ist beendet.
'; + //sort by date and time for correct order in mailtext + foreach ($rtdatumstrArray as $ds) + { + if ($ds->datum < $rt->datum) + $idx++; + elseif ($ds->datum == $rt->datum) + { + if ($ds->uhrzeit < $rt->uhrzeit) + $idx++; + else + break; + } + else + break; + } + $rtdatum = new stdClass(); + $rtdatum->datum = $rt->datum; + $rtdatum->uhrzeit = $rt->uhrzeit; + array_splice($rtdatumstrArray, $idx, 0, array($rtdatum)); } } + foreach ($rtdatumstrArray as $rtdatumobj) { + $rtdatumstr .= 'Der Reihungstest vom '.$datum_obj->convertISODate($rtdatumobj->datum).' um '.$datum_obj->formatDatum($rtdatumobj->uhrzeit, 'H:i').' Uhr ist beendet.
'; + } + $rtidparams = http_build_query(array('reihungstest' => $rtidArray)); foreach ($prestudentArray AS $studiengang_kz => $OrgFormPrestudent)