From 9606ec381f02de9f39c90ba4fb0c128b38853685 Mon Sep 17 00:00:00 2001
From: Markus Pospischil <pospisch@techikum-wien.at>
Date: Mon, 7 Jun 2010 10:01:51 +0000
Subject: [PATCH] =?UTF-8?q?Wenn=20eine=20Nachpr=C3=BCfung=20eingetragen=20?=
 =?UTF-8?q?wird,=20wird=20ein=20automatisches=20Mail=20and=20die=20Assiste?=
 =?UTF-8?q?nz=20und=20alle=20Lektoren=20generiert.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Das Datumsformat beim Eintragen der Nachprüfungsnoten wurde von YYYY-MM-DD auf DD.MM.YYYY geändert.
---
 .../benotungstool/lvgesamtnoteverwalten.php   | 18 ++---
 .../benotungstool/nachpruefungeintragen.php   | 68 +++++++++++++++++--
 cis/private/lehre/lesson.php                  | 10 +--
 include/benutzerfunktion.class.php            |  4 +-
 4 files changed, 80 insertions(+), 20 deletions(-)

diff --git a/cis/private/lehre/benotungstool/lvgesamtnoteverwalten.php b/cis/private/lehre/benotungstool/lvgesamtnoteverwalten.php
index e916795e2..6486c9008 100644
--- a/cis/private/lehre/benotungstool/lvgesamtnoteverwalten.php
+++ b/cis/private/lehre/benotungstool/lvgesamtnoteverwalten.php
@@ -264,7 +264,7 @@ $uid = (isset($_GET['uid'])?$_GET['uid']:'');
 	
 		str += "<tr><td colspan='2'><b>Pr&uuml;fung f&uuml;r "+uid+" anlegen:</b></td></tr>";
 		str += "<tr><td>Datum:</td>";
-		str += "<td><input type='hidden' name='uid' value='"+uid+"'><input type='hidden' name='le_id' value='"+lehreinheit_id+"'><input type='text' name='datum' value='"+datum+"'> [YYYY-MM-DD]</td>";
+		str += "<td><input type='hidden' name='uid' value='"+uid+"'><input type='hidden' name='le_id' value='"+lehreinheit_id+"'><input type='text' name='datum' value='"+datum+"'> [DD.MM.YYYY]</td>";
 		str += "</tr><tr><td>Note:</td>";
 		str += "<td><input type='text' name='note' value='"+note+"'></td>";
 		str += "</tr><tr><td colspan='2' align='center'><input type='button' name='speichern' value='speichern' onclick='pruefungSpeichern();'></td></tr>";
@@ -284,10 +284,10 @@ $uid = (isset($_GET['uid'])?$_GET['uid']:'');
 			alert("Bitte geben Sie eine Note von 1 - 5 bzw. 7 (nicht beurteilt), 8 (teilgenommen), 9 (noch nicht eingetragen) ein oder lassen Sie das Feld leer!");
 			document.getElementById(uid).note.value="";
 		}		
-		var datum = 	document.nachpruefung_form.datum.value;		
-		var datum_test = datum.split("-");
-		if (datum_test[0].length != 4 || datum_test[1].length!=2 || datum_test[2].length!=2 || isNaN(datum_test[0]) || datum_test[1]>12 || datum_test[2]>31)
-			alert("Invalid Date. Format: YYYY-MM-DD");
+		var datum = document.nachpruefung_form.datum.value;		
+		var datum_test = datum.split(".");
+		if (datum_test[0].length != 2 || datum_test[1].length != 2 || datum_test[2].length!=4 || isNaN(datum_test[2]) || datum_test[1]>12 || datum_test[1]<1 || datum_test[0]>31 || datum_test[0]<1)
+			alert("Invalid Date. Format: DD.MM.YYYY");
 		else
 		{
 			var anlegendiv = document.getElementById("nachpruefung_div");			
@@ -353,7 +353,7 @@ $uid = (isset($_GET['uid'])?$_GET['uid']:'');
 					anlegendiv.style.visibility = "hidden";
 					//if (note == 9)
 					//	note = " ";
-					document.getElementById("span_"+uid).innerHTML = "<table><tr><td class='td_datum'>"+datum+"</td><td class='td_note'>"+note+"<td><input type='button' name='anlegen' value='ändern' onclick='pruefungAnlegen(\""+uid+"\",\""+datum+"\",\""+note+"\",\""+lehreinheit_id+"\")'></td></tr></table>"
+					document.getElementById("span_"+uid).innerHTML = "<table><tr><td class='td_datum'>"+datum+"</td><td class='td_note'>"+note+"<td><input type='button' name='anlegen' value='&Auml;ndern' onclick='pruefungAnlegen(\""+uid+"\",\""+datum+"\",\""+note+"\",\""+lehreinheit_id+"\")'></td></tr></table>";
                 }
                 else
          		{
@@ -443,7 +443,7 @@ $uid = (isset($_GET['uid'])?$_GET['uid']:'');
 	// *******************************************************************************
 	function readNotenAusZwischenablage()
 	{
-		var data = getDataFromClipboard()
+		var data = getDataFromClipboard();
 		
 		//Reihen ermitteln
 		var rows = data.split("\n");
@@ -502,7 +502,7 @@ $uid = (isset($_GET['uid'])?$_GET['uid']:'');
 	        else alert("Request status:" + anfrage.status);
 	    }
 	}
--->
+//-->
 </script>
 <style type="text/css">
 .transparent {
@@ -1013,7 +1013,7 @@ echo '<table>';
 						
 						echo "<tr><td class='td_datum'>";
 						echo $pr_datum."</td><td class='td_note'>".$pr_note."</td><td>";
-						echo "<input type='button' name='anlegen' value='Ändern' onclick='pruefungAnlegen(\"".$row_stud->uid."\",\"".$pr_datum."\",\"".$pr_note."\",\"".$pr_le_id."\")'>";					
+						echo "<input type='button' name='anlegen' value='&Auml;ndern' onclick='pruefungAnlegen(\"".$row_stud->uid."\",\"".$pr_datum."\",\"".$pr_note."\",\"".$pr_le_id."\")'>";					
 						echo "</td></tr>";
 					}
 					echo "</table>";			
diff --git a/cis/private/lehre/benotungstool/nachpruefungeintragen.php b/cis/private/lehre/benotungstool/nachpruefungeintragen.php
index faab2075f..08345da86 100755
--- a/cis/private/lehre/benotungstool/nachpruefungeintragen.php
+++ b/cis/private/lehre/benotungstool/nachpruefungeintragen.php
@@ -42,6 +42,10 @@ require_once('../../../../include/legesamtnote.class.php');
 require_once('../../../../include/lvgesamtnote.class.php');
 require_once('../../../../include/zeugnisnote.class.php');
 require_once('../../../../include/pruefung.class.php');
+require_once('../../../../include/mail.class.php');
+require_once('../../../../include/benutzerfunktion.class.php');
+require_once('../../../../include/benutzer.class.php');
+require_once('../../../../include/student.class.php');
 
 $user = get_uid();
 
@@ -65,8 +69,11 @@ if(isset($_GET['lehreinheit_id_pr']) && is_numeric($_GET['lehreinheit_id_pr']))
 	$lehreinheit_id = $_GET['lehreinheit_id_pr'];
 	
 if(isset($_GET['datum']))
+{
 	$datum = $_GET['datum'];
-	
+	$datum_obj = new datum();
+	$datum = $datum_obj->checkformatDatum($datum, 'Y-m-d', true) OR die('Invalid date format');
+}
 else
 	die('Fehlerhafte Parameteruebergabe');
 
@@ -82,9 +89,8 @@ if(isset($_GET['stsem']))
 	$stsem = $_GET['stsem'];
 else
 	$stsem = '';
-
+	
 //Vars
-$datum_obj = new datum();
 
 $uebung_id = (isset($_GET['uebung_id'])?$_GET['uebung_id']:'');
 $uid = (isset($_GET['uid'])?$_GET['uid']:'');
@@ -109,6 +115,7 @@ if ( (($note>0) && ($note < 6)) || ($note == 7) || ($note==8) )
 else
 	$note = 9;
 
+$old_note = $note;
 
 // lvgesamtnote für studenten speichern
 if (isset($_REQUEST["submit"]) && ($_REQUEST["student_uid"] != '')  ){
@@ -178,6 +185,7 @@ if (isset($_REQUEST["submit"]) && ($_REQUEST["student_uid"] != '')  ){
 			$pr_2->new = null;
 			$pr_2->updateamum = $jetzt;
 			$pr_2->updatevon = $user;
+			$old_note = $pr_2->note;
 			$pr_2->note = $note;
 			$pr_2->datum = $datum;
 			$pr_2->anmerkung = "";
@@ -196,7 +204,8 @@ if (isset($_REQUEST["submit"]) && ($_REQUEST["student_uid"] != '')  ){
 			$pr_2->updateamum = Null;
 			$pr_2->updatevon = Null;
 			$pr_2->ext_id = Null;
-			$pr_2->new = 1;		
+			$pr_2->new = 1;
+			$old_note = -1;		
 		}
 		$pr_2->save();
 	}
@@ -251,7 +260,56 @@ else
 	echo "Fehler beim Eintragen der Pr&uuml;fungen";
 
 
+if ($old_note != $note)
+{
+	    $qry = "SELECT distinct on(uid) vorname, nachname, tbl_benutzer.uid as uid, oe_kurzbz 
+	    		FROM lehre.tbl_lehreinheit
+	    		JOIN lehre.tbl_lehreinheitmitarbeiter USING (lehreinheit_id)
+	    		JOIN public.tbl_benutzer ON (uid=mitarbeiter_uid)
+	    		JOIN public.tbl_person USING (person_id)
+	    		JOIN lehre.tbl_lehrveranstaltung USING (lehrveranstaltung_id)
+	    		JOIN public.tbl_studiengang USING (studiengang_kz)
+	    		WHERE 
+	    			lehrveranstaltung_id='$lvid' AND 
+	    			tbl_lehreinheitmitarbeiter.mitarbeiter_uid NOT like '_Dummy%' AND 
+	    			tbl_benutzer.aktiv=true AND tbl_person.aktiv=true AND 
+	    			studiensemester_kurzbz='$stsem'";
 
+    $mailto = '';
+    if(($result = $db->db_query($qry)) != FALSE)
+    {
+        if ($db->db_num_rows($result) > 0)
+        {
+            $row_lector = $db->db_fetch_object($result);
+            $oe_kurzbz = $row_lector->oe_kurzbz;
+            $mailto = $row_lector->uid.'@'.DOMAIN;
+            while (($row_lector = $db->db_fetch_object($result)) != FALSE)
+                $mailto .= ','.$row_lector->uid.'@'.DOMAIN;
+        }        
+    }
+    
+    $ass = new benutzerfunktion();
+    if ($ass->getBenutzerFunktionen("ass",$oe_kurzbz))
+        foreach ($ass->result as $res) $mailto .= (empty($mailto) ? "" : ",").$res->uid.'@'.DOMAIN;
+        
+    if($mailto != '')
+    {
+        $culprit = new benutzer($user);
+        $victim = new student($student_uid);
 
-
+        $mail = new mail(	$mailto,
+        					'CIS-System@do.not.reply',
+        					'[CIS-System] Note der Nachprüfung wurde '.($old_note < 0 ? 'eingetragen' : 'geändert'),
+        					"Automatische Benachrichtigung:\n\nDie Nachprüfungsnote von\n\n$victim->vorname $victim->nachname ($student_uid)\n\nwurde im Studiensemester $stsem von $culprit->vorname $culprit->nachname ($user)\n\n".
+                            ($old_note < 0
+                            ? "mit '$note' eingetragen."
+                            : "von '$old_note' auf '$note' geändert.").
+                            "\n");
+        if (!$mail->send())
+        {
+            sleep(3);
+            $mail->send();    // Desperate second attempt. 
+        }
+    }
+}
 ?>
diff --git a/cis/private/lehre/lesson.php b/cis/private/lehre/lesson.php
index 877b81dd5..420d4d3e9 100644
--- a/cis/private/lehre/lesson.php
+++ b/cis/private/lehre/lesson.php
@@ -109,15 +109,15 @@ function hideSemPlanHelp(){
 <table width="100%">
 <tr><td valign="top"><h2>&nbsp;Erstellung des Semesterplanes</h2></td><td align="right" valign="top"><a href="#" onclick="hideSemPlanHelp();">X</a>&nbsp;</td></tr>
 <tr>
-<td coslspan="2">
+<td colspan="2">
 <ol style="font-size:8pt;">
 	<li>Speichern Sie die Vorlage (html oder doc - Version) auf Ihrem Computer.</li>
 	<li>&Ouml;ffnen Sie die gespeicherte Datei mit ihrem Textverarbeitungsprogramm (MS Word, OpenOffice ...) bzw die html-Vorlage auch mit einem geeigneten Editor.</li>
 	<li>Erstellen Sie Ihren Semesterplan.</li>
-	<li>Speichern Sie das Dokument als 'Webseite' mit dem Namen 'index.html'<br>(In MS Word: 'Datei' -> 'Speichern unter' -> Dateiname: index.html, Dateityp: Webseite )</li>
+	<li>Speichern Sie das Dokument als 'Webseite' mit dem Namen 'index.html'<br>(In MS Word: 'Datei' -&gt; 'Speichern unter' -&gt; Dateiname: index.html, Dateityp: Webseite )</li>
 	<li>Laden Sie diese Datei nun &uuml;ber den 'Upload'-Knopf hoch.</li>
 	<li>Fertig!</li>
-<ol>
+</ol>
 </td>
 </tr>
 <tr><td colspan="2" align="center"><a href="#" onClick="hideSemPlanHelp();">schlie&szlig;en</a></td></tr>
@@ -189,8 +189,8 @@ function hideSemPlanHelp(){
 				}
 			}
 		}
-			
-		//Berechtigungen auf Fachbereichsebene
+
+				//Berechtigungen auf Fachbereichsebene
 	  $qry = "SELECT distinct fachbereich_kurzbz, tbl_lehrveranstaltung.studiengang_kz 
 	  		FROM lehre.tbl_lehrveranstaltung JOIN lehre.tbl_lehreinheit USING(lehrveranstaltung_id) JOIN lehre.tbl_lehrfach USING(lehrfach_id) 
 	  		WHERE lehrveranstaltung_id='$lvid'";
diff --git a/include/benutzerfunktion.class.php b/include/benutzerfunktion.class.php
index f5cc3023d..162e5cfd2 100644
--- a/include/benutzerfunktion.class.php
+++ b/include/benutzerfunktion.class.php
@@ -174,7 +174,9 @@ class benutzerfunktion extends basis_db
 	public function getBenutzerFunktionen($funktion_kurzbz, $oe_kurzbz='', $semester='')
 	{
 		$qry = "SELECT * FROM public.tbl_benutzerfunktion 
-				WHERE funktion_kurzbz='".addslashes($funktion_kurzbz)."'";
+				WHERE funktion_kurzbz='".addslashes($funktion_kurzbz)."'
+				AND (datum_bis >= now() OR datum_bis IS NULL)
+				AND (datum_von <= now() OR datum_von IS NULL)";
 
 		if($oe_kurzbz!='')
 			$qry.=" AND oe_kurzbz='".addslashes($oe_kurzbz)."'";