Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/LearningAction.java =================================================================== diff -u -r71cf49f904dd88f033e5db3f20d784cfbe6d59eb -r5f90da972465070c83d4344fed95952f1d94fe8e --- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/LearningAction.java (.../LearningAction.java) (revision 71cf49f904dd88f033e5db3f20d784cfbe6d59eb) +++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/LearningAction.java (.../LearningAction.java) (revision 5f90da972465070c83d4344fed95952f1d94fe8e) @@ -92,8 +92,6 @@ private static final boolean MODE_OPTIONAL = false; - private static final int UPLOAD_MAX_FILE_SIZE = Configuration.getAsInt(ConfigurationKeys.UPLOAD_FILE_MAX_SIZE); - private IForumService forumService; @Override @@ -623,11 +621,12 @@ boolean hideReflection = WebUtil.readBooleanParam(request, ForumConstants.ATTR_HIDE_REFLECTION, false); sessionMap.put(ForumConstants.ATTR_HIDE_REFLECTION, hideReflection); - if (sessionMap.get(ForumConstants.ATTR_UPLOAD_MAX_FILE_SIZE) == null && LearningAction.UPLOAD_MAX_FILE_SIZE > 0) { - sessionMap.put(ForumConstants.ATTR_UPLOAD_MAX_FILE_SIZE, - FileValidatorUtil.formatSize(LearningAction.UPLOAD_MAX_FILE_SIZE)); + int uploadMaxFileSize = Configuration.getAsInt(ConfigurationKeys.UPLOAD_FILE_MAX_SIZE); + // it defaults to -1 if property was not found + if (uploadMaxFileSize > 0) { + sessionMap.put(ForumConstants.ATTR_UPLOAD_MAX_FILE_SIZE, FileValidatorUtil.formatSize(uploadMaxFileSize)); } - + return mapping.findForward("success"); }