From dd791d96dec918158c0d4dfebd49d3f68a022bc1 Mon Sep 17 00:00:00 2001
From: srosse <none@none>
Date: Tue, 15 Jan 2013 08:34:47 +0100
Subject: [PATCH] OO-500: the subject element is not initialized with some
 configurations

---
 .../ui/wizard/BGMailTemplateController.java    | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/src/main/java/org/olat/group/ui/wizard/BGMailTemplateController.java b/src/main/java/org/olat/group/ui/wizard/BGMailTemplateController.java
index 4fa0e33bf85..d2ad6b3e5f2 100644
--- a/src/main/java/org/olat/group/ui/wizard/BGMailTemplateController.java
+++ b/src/main/java/org/olat/group/ui/wizard/BGMailTemplateController.java
@@ -127,14 +127,16 @@ public class BGMailTemplateController extends FormBasicController {
 		if(defaultTemplate.isSelected(0)) {
 			allOk &= true;
 		} else if (mandatoryEmail || sendMail.isSelected(0)) {
-			subjectElem.clearError();
-			if (subjectElem.getValue().trim().length() == 0) {
-				subjectElem.setErrorKey("mailtemplateform.error.emptyfield", null);
-				allOk &= false;
-			}
-			if (subjectElem.getValue().indexOf("#") != -1) {
-				subjectElem.setErrorKey("mailtemplateform.error.velocity", null);
-				allOk &= false;
+			if(subjectElem != null) {
+				subjectElem.clearError();
+				if (subjectElem.getValue().trim().length() == 0) {
+					subjectElem.setErrorKey("mailtemplateform.error.emptyfield", null);
+					allOk &= false;
+				}
+				if (subjectElem.getValue().indexOf("#") != -1) {
+					subjectElem.setErrorKey("mailtemplateform.error.velocity", null);
+					allOk &= false;
+				}
 			}
 			
 			bodyElem.clearError();
-- 
GitLab