Index: lams_tool_forum/conf/language/lams/ApplicationResources_en_AU.properties =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/conf/language/lams/ApplicationResources_en_AU.properties,v diff -u -r1.46 -r1.47 --- lams_tool_forum/conf/language/lams/ApplicationResources_en_AU.properties 24 Feb 2014 01:27:04 -0000 1.46 +++ lams_tool_forum/conf/language/lams/ApplicationResources_en_AU.properties 17 Dec 2014 00:12:30 -0000 1.47 @@ -25,6 +25,7 @@ label.authoring.advance.allow.edit =Allow learners to change their own postings label.authoring.advance.use.richeditor =Allow learners to use rich text editor label.authoring.advance.limited.input =Maximum number of characters per posting +label.authoring.advance.min.limited.input =Minimum number of characters per posting lable.topic.title.subject =Subject lable.topic.title.body =Body lable.topic.title.update =Last post @@ -229,5 +230,6 @@ js.error.validate.number =There is an error with the minimum and/or maximum set for ratings. Please check and try again. label.rateLimits.topic.reminder =You have rated {0} postings already. +label.char.required =Characters required #======= End labels: Exported 222 labels for en AU ===== Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/dbupdates/patch20141216.sql =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/dbupdates/patch20141216.sql,v diff -u --- /dev/null 1 Jan 1970 00:00:00 -0000 +++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/dbupdates/patch20141216.sql 17 Dec 2014 00:12:30 -0000 1.1 @@ -0,0 +1,17 @@ +-- Turn off autocommit, so nothing is committed if there is an error +SET AUTOCOMMIT = 0; +SET FOREIGN_KEY_CHECKS=0; +----------------------Put all sql statements below here------------------------- + +-- LDEV-3395 Implement a minimum number of characters for forum postings +ALTER TABLE tl_lafrum11_forum ADD COLUMN min_characters integer DEFAULT 0; +ALTER TABLE tl_lafrum11_forum ADD COLUMN limited_min_characters tinyint DEFAULT 0; + +UPDATE lams_tool SET tool_version='20140102' WHERE tool_signature='lafrum11'; + +----------------------Put all sql statements above here------------------------- + +-- If there were no errors, commit and restore autocommit to on +COMMIT; +SET AUTOCOMMIT = 1; +SET FOREIGN_KEY_CHECKS=1; \ No newline at end of file Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/persistence/Forum.java =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/persistence/Forum.java,v diff -u -r1.38 -r1.39 --- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/persistence/Forum.java 25 Jul 2014 13:49:34 -0000 1.38 +++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/persistence/Forum.java 17 Dec 2014 00:12:30 -0000 1.39 @@ -90,10 +90,14 @@ private ForumUser createdBy; private Set messages; + + private int minCharacters; + + private boolean limitedMinCharacters; - private int limitedChar; + private int maxCharacters; - private boolean limitedInput; + private boolean limitedMaxCharacters; private boolean reflectOnActivity; @@ -418,29 +422,53 @@ return toContent; } + + /** + * @hibernate.property column="min_characters" + * @return + */ + public int getMinCharacters() { + return minCharacters; + } + public void setMinCharacters(int minCharacters) { + this.minCharacters = minCharacters; + } + /** + * @hibernate.property column="limited_min_characters" + * @return + */ + public boolean isLimitedMinCharacters() { + return limitedMinCharacters; + } + + public void setLimitedMinCharacters(boolean limitedMinCharacters) { + this.limitedMinCharacters = limitedMinCharacters; + } + + /** * @hibernate.property column="limited_of_chars" * @return */ - public int getLimitedChar() { - return limitedChar; + public int getMaxCharacters() { + return maxCharacters; } - public void setLimitedChar(int limitedChar) { - this.limitedChar = limitedChar; + public void setMaxCharacters(int maxCharacters) { + this.maxCharacters = maxCharacters; } /** * @hibernate.property column="limited_input_flag" * @return */ - public boolean isLimitedInput() { - return limitedInput; + public boolean isLimitedMaxCharacters() { + return limitedMaxCharacters; } - public void setLimitedInput(boolean limitedInput) { - this.limitedInput = limitedInput; + public void setLimitedMaxCharacters(boolean limitedMaxCharacters) { + this.limitedMaxCharacters = limitedMaxCharacters; } /** Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/service/ForumService.java =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/service/ForumService.java,v diff -u -r1.128 -r1.129 --- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/service/ForumService.java 8 Oct 2014 15:14:25 -0000 1.128 +++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/service/ForumService.java 17 Dec 2014 00:12:30 -0000 1.129 @@ -1124,7 +1124,7 @@ toolContentObj.setDefineLater(Boolean.FALSE); toolContentObj.setInstructions(WebUtil.convertNewlines((String) importValues .get(ToolContentImport102Manager.CONTENT_BODY))); - toolContentObj.setLimitedChar(5000); // this is the default value + toolContentObj.setMaxCharacters(5000); // this is the default value toolContentObj.setReflectOnActivity(Boolean.FALSE); toolContentObj.setReflectInstructions(null); Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/util/ForumConstants.java =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/util/ForumConstants.java,v diff -u -r1.61 -r1.62 --- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/util/ForumConstants.java 19 Feb 2014 21:27:43 -0000 1.61 +++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/util/ForumConstants.java 17 Dec 2014 00:12:30 -0000 1.62 @@ -82,8 +82,10 @@ public static final String ATTR_MINIMUM_RATE = "minimumRate"; public static final String ATTR_MAXIMUM_RATE = "maximumRate"; + + public static final String ATTR_MIN_CHARACTERS = "minCharacters"; - public static final String ATTR_LIMITED_CHARS = "limitedChars"; + public static final String ATTR_MAX_CHARACTERS = "maxCharacters"; public static final String ATTR_LOCK_WHEN_FINISHED = "lockedWhenFinished"; Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/AuthoringAction.java =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/AuthoringAction.java,v diff -u -r1.82 -r1.83 --- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/AuthoringAction.java 26 Jul 2014 07:29:56 -0000 1.82 +++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/AuthoringAction.java 17 Dec 2014 00:12:30 -0000 1.83 @@ -865,8 +865,8 @@ if (StringUtils.equals(modeStr, ToolAccessMode.TEACHER.toString())) { return errors; } - if (!form.getForum().isAllowRichEditor() && form.getForum().isLimitedInput()) { - if (form.getForum().getLimitedChar() <= 0) { + if (!form.getForum().isAllowRichEditor() && form.getForum().isLimitedMaxCharacters()) { + if (form.getForum().getMaxCharacters() <= 0) { ActionMessage error = new ActionMessage("error.limit.char.less.zero"); errors.add(ActionMessages.GLOBAL_MESSAGE, error); } Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/LearningAction.java =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/LearningAction.java,v diff -u -r1.85 -r1.86 --- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/LearningAction.java 19 Feb 2014 21:27:43 -0000 1.85 +++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/LearningAction.java 17 Dec 2014 00:12:30 -0000 1.86 @@ -213,7 +213,8 @@ // Indicate don't restrict number of chars by allowNumber = 0 Long forumId = forum.getUid(); Boolean allowRichEditor = new Boolean(forum.isAllowRichEditor()); - int allowNumber = forum.isLimitedInput() || forum.isAllowRichEditor() ? forum.getLimitedChar() : 0; + int minCharacters = forum.isLimitedMinCharacters() ? forum.getMinCharacters() : 0; + int maxCharacters = forum.isLimitedMaxCharacters() || forum.isAllowRichEditor() ? forum.getMaxCharacters() : 0; sessionMap.put(AttributeNames.ATTR_MODE, mode); sessionMap.put(ForumConstants.ATTR_FORUM_ID, forumId); @@ -229,7 +230,8 @@ sessionMap.put(ForumConstants.ATTR_MAXIMUM_RATE, forum.getMaximumRate()); sessionMap.put(ForumConstants.ATTR_ALLOW_NEW_TOPICS, forum.isAllowNewTopic()); sessionMap.put(ForumConstants.ATTR_ALLOW_RICH_EDITOR, allowRichEditor); - sessionMap.put(ForumConstants.ATTR_LIMITED_CHARS, new Integer(allowNumber)); + sessionMap.put(ForumConstants.ATTR_MIN_CHARACTERS, new Integer(minCharacters)); + sessionMap.put(ForumConstants.ATTR_MAX_CHARACTERS, new Integer(maxCharacters)); sessionMap.put(AttributeNames.PARAM_TOOL_SESSION_ID, sessionId); sessionMap.put(ForumConstants.ATTR_FORUM_TITLE, forum.getTitle()); sessionMap.put(ForumConstants.ATTR_FORUM_INSTRCUTION, forum.getInstructions()); Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/forms/ForumForm.java =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/forms/ForumForm.java,v diff -u -r1.19 -r1.20 --- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/forms/ForumForm.java 17 Jan 2014 21:42:45 -0000 1.19 +++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/forms/ForumForm.java 17 Dec 2014 00:12:30 -0000 1.20 @@ -96,7 +96,7 @@ forum.setAllowEdit(false); forum.setAllowAnonym(false); forum.setAllowRichEditor(false); - forum.setLimitedInput(false); + forum.setLimitedMaxCharacters(false); forum.setLockWhenFinished(false); forum.setAllowNewTopic(false); forum.setAllowUpload(false); Index: lams_tool_forum/web/jsps/authoring/advance.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/authoring/advance.jsp,v diff -u -r1.31 -r1.32 --- lams_tool_forum/web/jsps/authoring/advance.jsp 19 Feb 2014 21:27:43 -0000 1.31 +++ lams_tool_forum/web/jsps/authoring/advance.jsp 17 Dec 2014 00:12:30 -0000 1.32 @@ -5,27 +5,21 @@

- - +

- - +

- - + @@ -68,88 +62,85 @@

- - +

- - +

- - -

+ +

+ + - +

- - +

- - +

- - +

- - +

- + + +

@@ -158,17 +149,15 @@

- + onclick="allowNewTopic()" styleId="allowNewTopic1" styleClass="noBorder"/>

- + onclick="allowNewTopic()" styleId="allowNewTopic2" styleClass="noBorder"/> @@ -192,7 +181,6 @@ 10 - @@ -213,99 +201,119 @@

Index: lams_tool_forum/web/jsps/learning/create.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/learning/create.jsp,v diff -u -r1.12 -r1.13 --- lams_tool_forum/web/jsps/learning/create.jsp 26 Mar 2014 23:27:40 -0000 1.12 +++ lams_tool_forum/web/jsps/learning/create.jsp 17 Dec 2014 00:12:30 -0000 1.13 @@ -1,9 +1,7 @@ <%@ include file="/common/taglibs.jsp"%> - + - + Index: lams_tool_forum/web/jsps/learning/edit.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/learning/edit.jsp,v diff -u -r1.8 -r1.9 --- lams_tool_forum/web/jsps/learning/edit.jsp 26 Mar 2014 23:27:40 -0000 1.8 +++ lams_tool_forum/web/jsps/learning/edit.jsp 17 Dec 2014 00:12:30 -0000 1.9 @@ -4,23 +4,21 @@ - +
- -

- -

- - - + +

+ +

+

- + - - <%@ include file="/jsps/learning/message/topiceditform.jsp"%> - + + <%@ include file="/jsps/learning/message/topiceditform.jsp"%> +
Index: lams_tool_forum/web/jsps/learning/viewtopic.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/learning/viewtopic.jsp,v diff -u -r1.25 -r1.26 --- lams_tool_forum/web/jsps/learning/viewtopic.jsp 26 Mar 2014 23:27:40 -0000 1.25 +++ lams_tool_forum/web/jsps/learning/viewtopic.jsp 17 Dec 2014 00:12:30 -0000 1.26 @@ -201,16 +201,3 @@ - - - - - - - - - - - - - Index: lams_tool_forum/web/jsps/learning/message/bodyarea.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/learning/message/bodyarea.jsp,v diff -u -r1.14 -r1.15 --- lams_tool_forum/web/jsps/learning/message/bodyarea.jsp 12 Jul 2013 14:17:07 -0000 1.14 +++ lams_tool_forum/web/jsps/learning/message/bodyarea.jsp 17 Dec 2014 00:12:30 -0000 1.15 @@ -8,40 +8,79 @@ toolbarSet="DefaultLearner"> + - <%-- Does not user general tag becuase this field need keep compatible with FCKEditor's content --%> + + <%-- Does not user general tag becuase this field need keep compatible with CKEditor's content --%>
+ <%-- If limitChars == 0, then we don't want to limit the characters at all. --%> - - : - - + + : + +
+ +
+ + + : + + + + var count = (limit - bodyTxt.value.length) > 0 ? limit - bodyTxt.value.length : 0; + limitCount.value = count; + showDiv.innerHTML = count; + } + calculateRequired(); + + bodyTxt.onkeyup = calculateRequired; + }); +
Index: lams_tool_forum/web/jsps/learning/message/topiceditform.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/learning/message/topiceditform.jsp,v diff -u -r1.18 -r1.19 --- lams_tool_forum/web/jsps/learning/message/topiceditform.jsp 21 Mar 2012 17:57:21 -0000 1.18 +++ lams_tool_forum/web/jsps/learning/message/topiceditform.jsp 17 Dec 2014 00:12:30 -0000 1.19 @@ -1,45 +1,48 @@ <%@ include file="/common/taglibs.jsp"%> +
+ +
- -
- -
- - - - - -
- * -
- - <%@include file="bodyarea.jsp"%> - - - -
- -
- - - -
- <%@ include file="/jsps/learning/message/msgattachment.jsp"%> -
- -
-
-
- - - - - - - - - - + + + + +
+ + *
+ +<%@include file="bodyarea.jsp"%> + + + +
+ +
+ + + +
+ <%@ include file="/jsps/learning/message/msgattachment.jsp"%> +
+ +
+
+
+ + + + + + + + + + +
Index: lams_tool_forum/web/jsps/learning/message/topicform.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/learning/message/topicform.jsp,v diff -u -r1.17 -r1.18 --- lams_tool_forum/web/jsps/learning/message/topicform.jsp 25 Dec 2013 00:01:32 -0000 1.17 +++ lams_tool_forum/web/jsps/learning/message/topicform.jsp 17 Dec 2014 00:12:30 -0000 1.18 @@ -25,21 +25,19 @@ -
- + - - - - - - - - - + + + + + + + + + -
Index: lams_tool_forum/web/jsps/learning/message/topicreplyform.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/learning/message/topicreplyform.jsp,v diff -u -r1.18 -r1.19 --- lams_tool_forum/web/jsps/learning/message/topicreplyform.jsp 25 Dec 2013 00:01:32 -0000 1.18 +++ lams_tool_forum/web/jsps/learning/message/topicreplyform.jsp 17 Dec 2014 00:12:30 -0000 1.19 @@ -1,48 +1,47 @@ <%@ include file="/common/taglibs.jsp"%> +
+ +
-
- -
- - - - - -
- - * -
- <%@include file="bodyarea.jsp"%> - - - -
- -
- - -
- - ${sessionMap.uploadMaxFileSize} - -
- - -
-
-
- - - - - - - - - -
-
+ + + +
+ + * +
+<%@include file="bodyarea.jsp"%> + + + +
+ +
+ + +
+ + ${sessionMap.uploadMaxFileSize} + +
+ + +
+
+
+ + + + + + + + + +
+
Index: lams_tool_forum/web/jsps/learning/mobile/message/bodyarea.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/learning/mobile/message/bodyarea.jsp,v diff -u -r1.2 -r1.3 --- lams_tool_forum/web/jsps/learning/mobile/message/bodyarea.jsp 21 Mar 2012 17:57:21 -0000 1.2 +++ lams_tool_forum/web/jsps/learning/mobile/message/bodyarea.jsp 17 Dec 2014 00:12:30 -0000 1.3 @@ -1,18 +1,18 @@ <%@ include file="/common/taglibs.jsp"%>
- <%-- Does not user general tag because this field need keep compatible with FCKEditor's content --%> + <%-- Does not user general tag because this field need keep compatible with CKEditor's content --%>
<%-- If limitChars == 0, then we don't want to limit the characters at all. --%> - + : + Index: lams_tool_forum/web/jsps/learning/mobile/message/topiceditform.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/learning/mobile/message/topiceditform.jsp,v diff -u -r1.2 -r1.3 --- lams_tool_forum/web/jsps/learning/mobile/message/topiceditform.jsp 21 Mar 2012 17:57:21 -0000 1.2 +++ lams_tool_forum/web/jsps/learning/mobile/message/topiceditform.jsp 17 Dec 2014 00:12:30 -0000 1.3 @@ -14,7 +14,7 @@
- Index: lams_tool_forum/web/jsps/learning/mobile/message/topicform.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/learning/mobile/message/topicform.jsp,v diff -u -r1.2 -r1.3 --- lams_tool_forum/web/jsps/learning/mobile/message/topicform.jsp 21 Mar 2012 17:57:21 -0000 1.2 +++ lams_tool_forum/web/jsps/learning/mobile/message/topicform.jsp 17 Dec 2014 00:12:30 -0000 1.3 @@ -19,7 +19,7 @@
- Index: lams_tool_forum/web/jsps/learning/mobile/message/topicreplyform.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/learning/mobile/message/topicreplyform.jsp,v diff -u -r1.2 -r1.3 --- lams_tool_forum/web/jsps/learning/mobile/message/topicreplyform.jsp 21 Mar 2012 17:57:21 -0000 1.2 +++ lams_tool_forum/web/jsps/learning/mobile/message/topicreplyform.jsp 17 Dec 2014 00:12:30 -0000 1.3 @@ -15,7 +15,7 @@
- Index: lams_tool_forum/web/jsps/monitoring/summary.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/monitoring/summary.jsp,v diff -u -r1.31 -r1.32 --- lams_tool_forum/web/jsps/monitoring/summary.jsp 28 Jul 2014 17:08:44 -0000 1.31 +++ lams_tool_forum/web/jsps/monitoring/summary.jsp 17 Dec 2014 00:12:30 -0000 1.32 @@ -328,13 +328,30 @@ + + + + + + + , ${forum.minCharacters} + + + + + + + + + + - - , ${forum.limitedChar} + + , ${forum.maxCharacters}