[server] Provide some more variables for mail templates

2 files changed, 45 insertions, 13 deletions

diff --git a/dozentenmodulserver/src/main/java/org/openslx/bwlp/sat/mail/MailGenerator.java b/dozentenmodulserver/src/main/java/org/openslx/bwlp/sat/mail/MailGenerator.java
index c390538b..f8de1327 100644
--- a/dozentenmodulserver/src/main/java/org/openslx/bwlp/sat/mail/MailGenerator.java
+++ b/dozentenmodulserver/src/main/java/org/openslx/bwlp/sat/mail/MailGenerator.java
@@ -49,12 +49,34 @@ public class MailGenerator {
 			throws SQLException {
 		if (!hasMailConfig())
 			return;
+		
+		String imageName;
+		try {
+			ImageDetailsRead image = DbImage.getImageDetails(null, newVersion.imageBaseId);
+			imageName = image.imageName;
+		} catch (TNotFoundException | SQLException e) {
+			LOGGER.warn("Could not get image details for image version " + newVersion.imageVersionId);
+			imageName = "(unbekannt)";
+		}
+		
+		String uploaderName;
+		try {
+			User u = DbUser.getCached(newVersion.uploaderId);
+			uploaderName = Formatter.userFullName(u.ui) + " <" + u.ui.eMail + ">";
+		} catch (TNotFoundException e) {
+			LOGGER.warn("Could not get uploading user of new image version " + newVersion.imageVersionId);
+			uploaderName = "(unbekannt)";
+		}
+		
 		for (LectureSummary lecture : lectures) {
 			List<UserInfo> relevantUsers = getUserToMail(lecture);
 
 			MailTemplate template = DbConfiguration.getMailTemplate(Template.LECTURE_UPDATED);
 			Map<String, String> templateArgs = new HashMap<>();
 			templateArgs.put("lecture", lecture.lectureName);
+			templateArgs.put("image", imageName);
+			templateArgs.put("created", Formatter.date(newVersion.createTime));
+			templateArgs.put("uploader", uploaderName);
 
 			String msg = template.format(templateArgs);
 
@@ -87,7 +109,7 @@ public class MailGenerator {
 			MailTemplate template = DbConfiguration.getMailTemplate(Template.LECTURE_FORCED_UPDATE);
 			Map<String, String> templateArgs = new HashMap<>();
 			templateArgs.put("lecture", lecture.lectureName);
-			templateArgs.put("date", Formatter.date(newVersion.createTime));
+			templateArgs.put("created", Formatter.date(newVersion.createTime));
 
 			String msg = template.format(templateArgs);
 
@@ -104,9 +126,18 @@ public class MailGenerator {
 		for (LectureSummary lecture : lectures) {
 			List<UserInfo> relevantUsers = getUserToMail(lecture);
 			
+			ImageDetailsRead image;
+			try {
+				image = DbImage.getImageDetails(null, lecture.imageBaseId);
+			} catch (TNotFoundException | SQLException e) {
+				LOGGER.warn("Could not get image details for image version " + lecture.imageVersionId);
+				return;
+			}
+			
 			MailTemplate template = DbConfiguration.getMailTemplate(Template.LECTURE_DEACTIVATED);
 			Map<String, String> templateArgs = new HashMap<>();
 			templateArgs.put("lecture", lecture.lectureName);
+			templateArgs.put("image", image.imageName);
 			
 			String msg = template.format(templateArgs);
 			
@@ -198,6 +229,7 @@ public class MailGenerator {
 		templateArgs.put("image", image.imageName);
 		templateArgs.put("remaining_days", String.valueOf(days));
 		templateArgs.put("created", Formatter.date(version.createTime));
+		templateArgs.put("expiretime", Formatter.date(version.expireTime));
 		String message = wordWrap(template.format(templateArgs));
 		relevantUsers = getUserToMail(image);
 		for (UserInfo user : relevantUsers) {
@@ -218,8 +250,8 @@ public class MailGenerator {
 			if (lecture.endTime < version.expireTime) {
 				continue;
 			}
-			template = DbConfiguration.getMailTemplate(Template.LECTURE_LINKED_VM_EXPIRING);			
-      templateArgs.put("lecture", lecture.lectureName);
+			template = DbConfiguration.getMailTemplate(Template.LECTURE_LINKED_VM_EXPIRING);
+			templateArgs.put("lecture", lecture.lectureName);
 			message = wordWrap(template.format(templateArgs));
 			relevantUsers = getUserToMail(lecture);
 			for (UserInfo user : relevantUsers) {
@@ -236,6 +268,7 @@ public class MailGenerator {
 		Map<String, String> templateArgs = new HashMap<>();
 		templateArgs.put("lecture", lecture.lectureName);
 		templateArgs.put("remaining_days", String.valueOf(days));
+		templateArgs.put("endtime", Formatter.date(lecture.endTime));
 		
 		String message = template.format(templateArgs);
 		
diff --git a/dozentenmodulserver/src/main/java/org/openslx/bwlp/sat/mail/MailTemplateConfiguration.java b/dozentenmodulserver/src/main/java/org/openslx/bwlp/sat/mail/MailTemplateConfiguration.java
index e59f9726..b06bf617 100644
--- a/dozentenmodulserver/src/main/java/org/openslx/bwlp/sat/mail/MailTemplateConfiguration.java
+++ b/dozentenmodulserver/src/main/java/org/openslx/bwlp/sat/mail/MailTemplateConfiguration.java
@@ -18,7 +18,7 @@ public class MailTemplateConfiguration {
 					Template.LECTURE_UPDATED,
 					"Wird an die Verantwortlichen einer Veranstaltung gesendet, wenn die verknüpfte VM aktualisiert wurde.",
 					"Die zur Veranstaltung '%lecture%' gehörige VM wurde aktualisiert.",
-					new String[]{},
+					new String[]{"image", "created", "uploader"},
 					new String[]{"lecture"}
 					),
 			
@@ -29,7 +29,7 @@ public class MailTemplateConfiguration {
 						+ " da die verknüpfte VM gelöscht oder beschädigt wurde. Bitte überprüfen"
 						+ " Sie die Veranstaltung und ändern Sie ggf. die Verlinkung,"
 						+ " damit die Veranstaltung wieder verwendbar ist.",
-					new String[] {},
+					new String[] {"image"},
 					new String[] {"lecture"}
 					),
 			
@@ -38,7 +38,7 @@ public class MailTemplateConfiguration {
 					"Wird versendet, wenn die aktuellste Version einer VM kurz vor dem Ablaufdatum steht.",
 					"Die aktuellste Version der VM '%image%' läuft in %remaining_days% Tag(en) ab."
 						+ " Bitte aktualisieren Sie die VM, da verknüpfte Veranstaltungen sonst deaktiviert werden.",
-					new String[]{"remaining_days"},
+					new String[]{"remaining_days", "created", "expiretime"},
 					new String[]{"image"}
 					),
 			
@@ -47,7 +47,7 @@ public class MailTemplateConfiguration {
 					"Hinweis, dass eine alte Version einer VM abläuft.",
 					"Eine alte Version der VM '%image%' läuft in %remaining_days% Tag(en) ab (Version vom %created%)."
 						+ " Eine aktuellere Version ist vorhanden, diese Nachricht dient nur der Information.",
-					new String[]{"remaining_days", "created"},
+					new String[]{"remaining_days", "created", "expiretime"},
 					new String[]{"image"}
 					),
 				
@@ -56,16 +56,15 @@ public class MailTemplateConfiguration {
 					"Hinweis, dass die zu einer Veranstaltung gehörige VM bald abläuft.",
 					"Hinweis zur Veranstaltung '%lecture%': Die verwendete VM '%image%'"
 						+ " läuft in %remaining_days% Tag(en) ab. Bitte aktualisieren oder wechseln Sie die VM.",
-					new String[]{"remaining_days"},
-					new String[]{"lecture"}
+					new String[]{"remaining_days", "expiretime"},
+					new String[]{"lecture", "image"}
 					),
 					
-					
 			new MailTemplatePlain(
 					Template.LECTURE_EXPIRING,
 					"Wird versendet, wenn eine Veranstaltung kurz vor dem Enddatum steht.",
 					"Die Veranstaltung '%lecture%' läuft in %remaining_days% Tag(en) ab. Verlängern Sie bei Bedarf das Ablaufdatum.",
-					new String[]{"remaining_days"},
+					new String[]{"remaining_days", "endtime"},
 					new String[]{"lecture"}
 					),
 			
@@ -84,7 +83,7 @@ public class MailTemplateConfiguration {
 					"Bestätigung dass eine alte Version der VM gelöscht wurde",
 					"Eine alte Version der VM '%image%' vom %old_created% wurde gelöscht\n"
 						+ "Die neueste Version ist jetzt vom %new_created% (erstellt von %uploader%)",
-					new String[]{"old_created", "new_created"},
+					new String[]{"old_created", "new_created", "uploader"},
 					new String[]{"image"}
 					),
 			
@@ -96,7 +95,7 @@ public class MailTemplateConfiguration {
 					"Die verlinkte VM zur Veranstaltung '%lecture%' wurde gelöscht oder ist beschädigt."
 						+ " Daher verweist sie jetzt auf die VM-Version vom %date%."
 						+ " Bitte überprüfen Sie ggf., ob diese VM-Version für Ihren Kurs geeignet ist.",
-					new String[]{"date"},
+					new String[]{"created"},
 					new String[]{"lecture"}
 					),