From b3c044bf86fe257910c72803b3b69ec9b6898662 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Andreas=20=C3=96sterreicher?= <oesi@technikum-wien.at>
Date: Thu, 22 Nov 2018 17:25:56 +0100
Subject: [PATCH] =?UTF-8?q?Im=20Infomail=20f=C3=BCr=20die=20Notenfreigabe?=
 =?UTF-8?q?=20ist=20per=20Default=20nur=20noch=20die=20Anzahl=20der=20ge?=
 =?UTF-8?q?=C3=A4nderten=20Noten=20enthalten.=20Es=20sind=20keine=20Studie?=
 =?UTF-8?q?rendendaten=20mehr=20im=20Mail=20vorhanden?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../benotungstool/lvgesamtnoteverwalten.php   | 55 ++++++++++---------
 1 file changed, 30 insertions(+), 25 deletions(-)

diff --git a/cis/private/lehre/benotungstool/lvgesamtnoteverwalten.php b/cis/private/lehre/benotungstool/lvgesamtnoteverwalten.php
index 2a1a615e8..d3919d25e 100644
--- a/cis/private/lehre/benotungstool/lvgesamtnoteverwalten.php
+++ b/cis/private/lehre/benotungstool/lvgesamtnoteverwalten.php
@@ -803,19 +803,19 @@ if (isset($_REQUEST["freigabe"]) && ($_REQUEST["freigabe"] == 1))
 	{
 		$jetzt = date("Y-m-d H:i:s");
 		$neuenoten = 0;
+
 		$studlist = "<table border='1'>
-			<tr>
-				<td><b>" . $p->t('global/personenkz') . "</b></td>
-				<td><b>" . $p->t('global/nachname') . "</b></td>
-				<td><b>" . $p->t('global/vorname') . "</b></td>";
-		if (defined('CIS_GESAMTNOTE_FREIGABEMAIL_NOTE') && CIS_GESAMTNOTE_FREIGABEMAIL_NOTE)
+		<tr>
+			<td><b>" . $p->t('global/personenkz') . "</b></td>
+			<td><b>" . $p->t('global/nachname') . "</b></td>
+			<td><b>" . $p->t('global/vorname') . "</b></td>";
+
+		if (defined('CIS_GESAMTNOTE_PUNKTE') && CIS_GESAMTNOTE_PUNKTE)
 		{
-			if (defined('CIS_GESAMTNOTE_PUNKTE') && CIS_GESAMTNOTE_PUNKTE)
-			{
-				$studlist .= "<td><b>" . $p->t('benotungstool/punkte') . "</b></td>\n";
-			}
-			$studlist .= "<td><b>" . $p->t('benotungstool/note') . "</b></td>\n";
+			$studlist .= "<td><b>" . $p->t('benotungstool/punkte') . "</b></td>\n";
 		}
+		$studlist .= "<td><b>" . $p->t('benotungstool/note') . "</b></td>\n";
+
 		$studlist .= "<td><b>" . $p->t('benotungstool/bearbeitetvon') . "</b></td></tr>\n";
 
 		// studentenquery
@@ -841,24 +841,25 @@ if (isset($_REQUEST["freigabe"]) && ($_REQUEST["freigabe"] == 1))
 						$lvgesamtnote->freigabedatum = $jetzt;
 						$lvgesamtnote->freigabevon_uid = $user;
 						$lvgesamtnote->save();
+
 						$studlist .= "<tr><td>" . trim($row_stud->matrikelnr) . "</td>";
 						$studlist .= "<td>" . trim($row_stud->nachname) . "</td>";
 						$studlist .= "<td>" . trim($row_stud->vorname) . "</td>";
-						if (defined('CIS_GESAMTNOTE_FREIGABEMAIL_NOTE') && CIS_GESAMTNOTE_FREIGABEMAIL_NOTE)
+
+						if (defined('CIS_GESAMTNOTE_PUNKTE') && CIS_GESAMTNOTE_PUNKTE)
 						{
-							if (defined('CIS_GESAMTNOTE_PUNKTE') && CIS_GESAMTNOTE_PUNKTE)
-							{
-								$studlist .= "<td>";
-								if ($lvgesamtnote->punkte != '')
-									$studlist .= trim(number_format($lvgesamtnote->punkte, 2));
-								$studlist .= "</td>\n";
-							}
-							$studlist .= "<td>" . $noten_array[trim($lvgesamtnote->note)]['bezeichnung_mehrsprachig'][$sprache] . "</td>";
+							$studlist .= "<td>";
+							if ($lvgesamtnote->punkte != '')
+								$studlist .= trim(number_format($lvgesamtnote->punkte, 2));
+							$studlist .= "</td>\n";
 						}
+						$studlist .= "<td>" . $noten_array[trim($lvgesamtnote->note)]['bezeichnung_mehrsprachig'][$sprache] . "</td>";
+
 						$studlist .= "<td>" . $lvgesamtnote->mitarbeiter_uid;
 						if ($lvgesamtnote->updatevon != '')
 							$studlist .= " (" . $lvgesamtnote->updatevon . ")";
 						$studlist .= "</td></tr>\n";
+
 						$neuenoten ++;
 					}
 				}
@@ -883,20 +884,24 @@ if (isset($_REQUEST["freigabe"]) && ($_REQUEST["freigabe"] == 1))
 
 			$mit = new mitarbeiter();
 			$mit->load($user);
+			$name = $mit->anrede.' '.$mit->vorname.' '.$mit->nachname.' ('.$mit->kurzbz.')';
 
-			$freigeber = "<b>" . mb_strtoupper($user) . "</b>";
 			$betreff = 'Notenfreigabe ' . $lv->bezeichnung . ' ' . $lv->orgform_kurzbz . ' - ' . $studienplan_bezeichnung;
 			$mail = new mail($adressen, 'vilesci@' . DOMAIN, $betreff, '');
 			$htmlcontent = "<html>
 				<body>
+					$name hat neue Noten für die Lehrveranstaltung\n\n<br>
 					<b>" . $sg->kuerzel . ' ' . $lv->semester . '.Semester
 					' . $lv->bezeichnung . " " . $lv->orgform_kurzbz . " - " . $stsem . "</b>
-					(" . $lv->semester . ". Sem.)
-					<br><br>" . $p->t('global/benutzer') . " " . $freigeber . " (" . $mit->kurzbz . ")
-					" . $p->t('benotungstool/hatDieLvNotenFuerFolgendeStudenten') . ":
-					<br><br>\n" . $studlist . "
+					<br>eingetragen.\n<br><br>
+					Die Noten können jetzt ins Zeugnis übernommen werden.\n";
+
+			if (defined('CIS_GESAMTNOTE_FREIGABEMAIL_NOTE') && CIS_GESAMTNOTE_FREIGABEMAIL_NOTE)
+				$htmlcontent.= $studlist;
+
+			$htmlcontent.= "
 					<br>Anzahl der Noten:" . $neuenoten . "
-					<br>" . $p->t('abgabetool/mailVerschicktAn') . ": " . $adressen . "
+					<br><br>" . $p->t('abgabetool/mailVerschicktAn') . ": " . $adressen . "
 				</body></html>";
 			$mail->setHTMLContent($htmlcontent);
 			$mail->setReplyTo($lektor_adresse);