From 7228a1cabd37f7f604568e22cd222c26799b697c Mon Sep 17 00:00:00 2001
From: strentini <none@none>
Date: Wed, 29 Feb 2012 09:49:38 +0100
Subject: [PATCH] OO-131 : fix problem with long attachment-filenames in
topicAssignment ( invalidate form if filename > 100 chars, show error-msg)
--HG--
branch : 80-patch
---
.../projectbroker/ProjectEditDetailsFormController.java | 6 ++++++
.../nodes/projectbroker/_i18n/LocalStrings_de.properties | 1 +
.../nodes/projectbroker/_i18n/LocalStrings_en.properties | 2 ++
3 files changed, 9 insertions(+)
diff --git a/src/main/java/org/olat/course/nodes/projectbroker/ProjectEditDetailsFormController.java b/src/main/java/org/olat/course/nodes/projectbroker/ProjectEditDetailsFormController.java
index 8fec66f84bd..7b3b2e461d6 100644
--- a/src/main/java/org/olat/course/nodes/projectbroker/ProjectEditDetailsFormController.java
+++ b/src/main/java/org/olat/course/nodes/projectbroker/ProjectEditDetailsFormController.java
@@ -146,6 +146,12 @@ public class ProjectEditDetailsFormController extends FormBasicController {
projectTitle.setErrorKey("form.error.project.title.is.empty", null);
return false;
}
+
+ // http://jira.openolat.org/browse/OO-131 check for too long filename
+ if (attachmentFileName.getUploadFileName().length() > 99) {
+ attachmentFileName.setErrorKey("form.error.project.filenametoolong", null);
+ return false;
+ }
return true;
}
diff --git a/src/main/java/org/olat/course/nodes/projectbroker/_i18n/LocalStrings_de.properties b/src/main/java/org/olat/course/nodes/projectbroker/_i18n/LocalStrings_de.properties
index 9a0c510c6b6..06f075b0a22 100644
--- a/src/main/java/org/olat/course/nodes/projectbroker/_i18n/LocalStrings_de.properties
+++ b/src/main/java/org/olat/course/nodes/projectbroker/_i18n/LocalStrings_de.properties
@@ -147,6 +147,7 @@ handout_event=Abgabetermin
from.error.date.start.after.end=Starttermin nach Endtermin
form.error.project.title.already.exist=Es existiert bereits ein Thema mit diesem Namen.
form.error.project.title.is.empty=Der Titel darf nicht leer sein.
+form.error.project.filenametoolong=Der gewählte Dateinamen ist zu lang (max 100 Zeichen).
mail.enrolled.to.participant.subject=Wahl des Themas: $projectTitle
mail.enrolled.to.participant.body=*** Das ist eine automatisch generierte Nachricht. Bitte antworten Sie nicht auf diese Nachricht *** \n\nSie haben am $currentDate das Thema "$projectTitle" gew\u00E4hlt.
diff --git a/src/main/java/org/olat/course/nodes/projectbroker/_i18n/LocalStrings_en.properties b/src/main/java/org/olat/course/nodes/projectbroker/_i18n/LocalStrings_en.properties
index 1fcf05c512b..cc68cbcb1c2 100644
--- a/src/main/java/org/olat/course/nodes/projectbroker/_i18n/LocalStrings_en.properties
+++ b/src/main/java/org/olat/course/nodes/projectbroker/_i18n/LocalStrings_en.properties
@@ -120,6 +120,8 @@ form.accountmanagement.no.account.manager.group=No topic authors available.
form.accountmanagement.title=Administer topic authors
form.error.project.title.already.exist=A topic with this title already exists.
form.error.project.title.is.empty=This title is mandatory.
+form.error.project.filenametoolong=Filename has to many characters (max 100).
+
form.modules.dropbox=Drop box
form.modules.returnbox=Return box
form.modules.scoring=Assessment
--
GitLab