Index: lams_tool_images/conf/language/lams/ApplicationResources.properties =================================================================== diff -u -r0c0c05bbcad9cb7918b6c3309244ae99d5a5b207 -rf8b98e45847fcee40188af4c9db5d13e1bd61e25 --- lams_tool_images/conf/language/lams/ApplicationResources.properties (.../ApplicationResources.properties) (revision 0c0c05bbcad9cb7918b6c3309244ae99d5a5b207) +++ lams_tool_images/conf/language/lams/ApplicationResources.properties (.../ApplicationResources.properties) (revision f8b98e45847fcee40188af4c9db5d13e1bd61e25) @@ -169,5 +169,6 @@ label.image.options=Image Options label.activity.completion=End of Activity label.posted.by ={0} on {1} +errors.maxfilesize = Image exceeds the max file size #======= End labels: Exported 161 labels for en AU ===== Index: lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/LearningAction.java =================================================================== diff -u -r3399163940c61c9132223c758d274486e57ff9b7 -rf8b98e45847fcee40188af4c9db5d13e1bd61e25 --- lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/LearningAction.java (.../LearningAction.java) (revision 3399163940c61c9132223c758d274486e57ff9b7) +++ lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/LearningAction.java (.../LearningAction.java) (revision f8b98e45847fcee40188af4c9db5d13e1bd61e25) @@ -814,21 +814,21 @@ private List createFileListFromMultipleForm(MultipleImagesForm multipleForm) { List fileList = new ArrayList(); - if (!StringUtils.isEmpty(multipleForm.getFile1().getFileName())) { + if (multipleForm.getFile1() != null && !StringUtils.isEmpty(multipleForm.getFile1().getFileName())) { fileList.add(multipleForm.getFile1()); } - if (!StringUtils.isEmpty(multipleForm.getFile2().getFileName())) { + if (multipleForm.getFile2() != null && !StringUtils.isEmpty(multipleForm.getFile2().getFileName())) { fileList.add(multipleForm.getFile2()); } - if (!StringUtils.isEmpty(multipleForm.getFile3().getFileName())) { + if (multipleForm.getFile3() != null && !StringUtils.isEmpty(multipleForm.getFile3().getFileName())) { fileList.add(multipleForm.getFile3()); } - if (!StringUtils.isEmpty(multipleForm.getFile4().getFileName())) { + if (multipleForm.getFile4() != null && !StringUtils.isEmpty(multipleForm.getFile4().getFileName())) { fileList.add(multipleForm.getFile4()); } - if (!StringUtils.isEmpty(multipleForm.getFile5().getFileName())) { + if (multipleForm.getFile5() != null && !StringUtils.isEmpty(multipleForm.getFile5().getFileName())) { fileList.add(multipleForm.getFile5()); - } + } return fileList; } Index: lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/MonitoringAction.java =================================================================== diff -u -r3399163940c61c9132223c758d274486e57ff9b7 -rf8b98e45847fcee40188af4c9db5d13e1bd61e25 --- lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/MonitoringAction.java (.../MonitoringAction.java) (revision 3399163940c61c9132223c758d274486e57ff9b7) +++ lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/MonitoringAction.java (.../MonitoringAction.java) (revision f8b98e45847fcee40188af4c9db5d13e1bd61e25) @@ -580,21 +580,21 @@ private List createFileListFromMultipleForm(MultipleImagesForm multipleForm) { List fileList = new ArrayList(); - if (!StringUtils.isEmpty(multipleForm.getFile1().getFileName())) { + if (multipleForm.getFile1() != null && !StringUtils.isEmpty(multipleForm.getFile1().getFileName())) { fileList.add(multipleForm.getFile1()); } - if (!StringUtils.isEmpty(multipleForm.getFile2().getFileName())) { + if (multipleForm.getFile2() != null && !StringUtils.isEmpty(multipleForm.getFile2().getFileName())) { fileList.add(multipleForm.getFile2()); } - if (!StringUtils.isEmpty(multipleForm.getFile3().getFileName())) { + if (multipleForm.getFile3() != null && !StringUtils.isEmpty(multipleForm.getFile3().getFileName())) { fileList.add(multipleForm.getFile3()); } - if (!StringUtils.isEmpty(multipleForm.getFile4().getFileName())) { + if (multipleForm.getFile4() != null && !StringUtils.isEmpty(multipleForm.getFile4().getFileName())) { fileList.add(multipleForm.getFile4()); } - if (!StringUtils.isEmpty(multipleForm.getFile5().getFileName())) { + if (multipleForm.getFile5() != null && !StringUtils.isEmpty(multipleForm.getFile5().getFileName())) { fileList.add(multipleForm.getFile5()); - } + } return fileList; } Index: lams_tool_images/web/includes/javascript/imageGallerycommon.js =================================================================== diff -u -rf4d9fb8cc51c25fecdd0eac1dcf5f5efc916cdf8 -rf8b98e45847fcee40188af4c9db5d13e1bd61e25 --- lams_tool_images/web/includes/javascript/imageGallerycommon.js (.../imageGallerycommon.js) (revision f4d9fb8cc51c25fecdd0eac1dcf5f5efc916cdf8) +++ lams_tool_images/web/includes/javascript/imageGallerycommon.js (.../imageGallerycommon.js) (revision f8b98e45847fcee40188af4c9db5d13e1bd61e25) @@ -6,6 +6,7 @@ $("#new-image-input-area").load(url, function() { $(this).show(); $("#saveCancelButtons").hide(); + $('#file-title').focus(); }); } Index: lams_tool_images/web/includes/javascript/imageGalleryitem.js =================================================================== diff -u -rf4d9fb8cc51c25fecdd0eac1dcf5f5efc916cdf8 -rf8b98e45847fcee40188af4c9db5d13e1bd61e25 --- lams_tool_images/web/includes/javascript/imageGalleryitem.js (.../imageGalleryitem.js) (revision f4d9fb8cc51c25fecdd0eac1dcf5f5efc916cdf8) +++ lams_tool_images/web/includes/javascript/imageGalleryitem.js (.../imageGalleryitem.js) (revision f8b98e45847fcee40188af4c9db5d13e1bd61e25) @@ -54,18 +54,19 @@ if (file.name.length < 1) { alert("file.name.length < 1"); return; - } else if (file.size > UPLOAD_FILE_LARGE_MAX_SIZE) { - alert(LABEL_MAX_FILE_SIZE); - return; } else if (file.type != 'image/png' && file.type != 'image/jpg' && file.type != 'image/gif' && file.type != 'image/jpeg' ) { alert(LABEL_NOT_ALLOWED_FORMAT); return; + } else if (file.size > UPLOAD_FILE_LARGE_MAX_SIZE) { + alert(LABEL_MAX_FILE_SIZE); + return; } // Add the file to the request. formData.append('file', file, file.name); $('#uploadButtons').hide(); $('#itemAttachmentArea_Busy').show(); + } $.ajax({ @@ -75,9 +76,11 @@ processData: false, contentType: false, success: function(data) { - $('#new-image-input-area').html(data); + $('#new-image-input-area').html(data); }, error: function(jqXHR, textStatus, errorMessage) { + $('#uploadButtons').show(); + $('#itemAttachmentArea_Busy').hide(); alert(errorMessage); } }); @@ -152,7 +155,9 @@ $('#new-image-input-area').html(data); }, error: function(jqXHR, textStatus, errorMessage) { - alert(errorMessage); + $('#uploadButtons').show(); + $('#itemAttachmentArea_Busy').hide(); + alert(errorMessage); } }); } Index: lams_tool_images/web/pages/authoring/parts/addimage.jsp =================================================================== diff -u -rf4d9fb8cc51c25fecdd0eac1dcf5f5efc916cdf8 -rf8b98e45847fcee40188af4c9db5d13e1bd61e25 --- lams_tool_images/web/pages/authoring/parts/addimage.jsp (.../addimage.jsp) (revision f4d9fb8cc51c25fecdd0eac1dcf5f5efc916cdf8) +++ lams_tool_images/web/pages/authoring/parts/addimage.jsp (.../addimage.jsp) (revision f8b98e45847fcee40188af4c9db5d13e1bd61e25) @@ -17,6 +17,7 @@ +
<%@ include file="/common/messages.jsp"%> Index: lams_tool_images/web/pages/authoring/parts/imagefile.jsp =================================================================== diff -u -r5f7cdba78b6fde3914f078d77c1131bf742c0111 -rf8b98e45847fcee40188af4c9db5d13e1bd61e25 --- lams_tool_images/web/pages/authoring/parts/imagefile.jsp (.../imagefile.jsp) (revision 5f7cdba78b6fde3914f078d77c1131bf742c0111) +++ lams_tool_images/web/pages/authoring/parts/imagefile.jsp (.../imagefile.jsp) (revision f8b98e45847fcee40188af4c9db5d13e1bd61e25) @@ -38,7 +38,7 @@ - +
Index: lams_tool_images/web/pages/learning/learning.jsp =================================================================== diff -u -r0ac411e19ec849c9586c506feede55d178f84d56 -rf8b98e45847fcee40188af4c9db5d13e1bd61e25 --- lams_tool_images/web/pages/learning/learning.jsp (.../learning.jsp) (revision 0ac411e19ec849c9586c506feede55d178f84d56) +++ lams_tool_images/web/pages/learning/learning.jsp (.../learning.jsp) (revision f8b98e45847fcee40188af4c9db5d13e1bd61e25) @@ -163,7 +163,8 @@ sessionMapID: "${sessionMapID}" } ); - } + } + Index: lams_tool_images/web/pages/learning/parts/addimage.jsp =================================================================== diff -u -r61ebeba7cc58c61cfb547ecfa0d61cd091bf08dc -rf8b98e45847fcee40188af4c9db5d13e1bd61e25 --- lams_tool_images/web/pages/learning/parts/addimage.jsp (.../addimage.jsp) (revision 61ebeba7cc58c61cfb547ecfa0d61cd091bf08dc) +++ lams_tool_images/web/pages/learning/parts/addimage.jsp (.../addimage.jsp) (revision f8b98e45847fcee40188af4c9db5d13e1bd61e25) @@ -1,17 +1,85 @@ - <%@ include file="/common/taglibs.jsp"%> +<%@ page import="org.lamsfoundation.lams.util.Configuration" %> +<%@ page import="org.lamsfoundation.lams.util.ConfigurationKeys" %> +<%=Configuration.get(ConfigurationKeys.UPLOAD_FILE_MAX_SIZE)%> + <fmt:message key="label.learning.title" /> <%@ include file="/common/header.jsp"%> - + @@ -20,6 +88,11 @@
+ + + +
@@ -34,36 +107,53 @@ - +
- +
-
- - +
+ + + + +
- - - - + + + + + + -
- - - - + Index: lams_tool_images/web/pages/learning/parts/addmultipleimages.jsp =================================================================== diff -u -r61ebeba7cc58c61cfb547ecfa0d61cd091bf08dc -rf8b98e45847fcee40188af4c9db5d13e1bd61e25 --- lams_tool_images/web/pages/learning/parts/addmultipleimages.jsp (.../addmultipleimages.jsp) (revision 61ebeba7cc58c61cfb547ecfa0d61cd091bf08dc) +++ lams_tool_images/web/pages/learning/parts/addmultipleimages.jsp (.../addmultipleimages.jsp) (revision f8b98e45847fcee40188af4c9db5d13e1bd61e25) @@ -1,12 +1,92 @@ - <%@ include file="/common/taglibs.jsp"%> - +<%@ page import="org.lamsfoundation.lams.util.Configuration" %> +<%@ page import="org.lamsfoundation.lams.util.ConfigurationKeys" %> +<%=Configuration.get(ConfigurationKeys.UPLOAD_FILE_MAX_SIZE)%> + <fmt:message key="label.learning.title" /> <%@ include file="/common/header.jsp"%> + @@ -15,6 +95,10 @@ @@ -23,45 +107,141 @@ <%@ include file="/common/messages.jsp"%> - +
- -
-
- -
-
- -
-
- -
-
- -
+
+ + + + + +
+ + + +
+ + + + + +
+ + + +
+ + + + + +
+ + + +
+ + + + + +
+ + + +
+ + + + + +
+ +
- - - -
+ + - +
+ +
Index: lams_tool_images/web/pages/learning/parts/commentsarea.jsp =================================================================== diff -u -r61ebeba7cc58c61cfb547ecfa0d61cd091bf08dc -rf8b98e45847fcee40188af4c9db5d13e1bd61e25 --- lams_tool_images/web/pages/learning/parts/commentsarea.jsp (.../commentsarea.jsp) (revision 61ebeba7cc58c61cfb547ecfa0d61cd091bf08dc) +++ lams_tool_images/web/pages/learning/parts/commentsarea.jsp (.../commentsarea.jsp) (revision f8b98e45847fcee40188af4c9db5d13e1bd61e25) @@ -96,27 +96,25 @@ <%--"Check for new", "Add new image" and "Delete" buttons---------------%> - - - - - - - -
- - - +
+ + + + + + + + + + + + - - -
- - - -
- - +
Index: lams_tool_images/web/pages/monitoring/imagesummary.jsp =================================================================== diff -u -raaf77d979eda13d8e6947b82857bbeda7c62be3f -rf8b98e45847fcee40188af4c9db5d13e1bd61e25 --- lams_tool_images/web/pages/monitoring/imagesummary.jsp (.../imagesummary.jsp) (revision aaf77d979eda13d8e6947b82857bbeda7c62be3f) +++ lams_tool_images/web/pages/monitoring/imagesummary.jsp (.../imagesummary.jsp) (revision f8b98e45847fcee40188af4c9db5d13e1bd61e25) @@ -134,12 +134,12 @@
Index: lams_tool_images/web/pages/monitoring/monitoring.jsp =================================================================== diff -u -r61ebeba7cc58c61cfb547ecfa0d61cd091bf08dc -rf8b98e45847fcee40188af4c9db5d13e1bd61e25 --- lams_tool_images/web/pages/monitoring/monitoring.jsp (.../monitoring.jsp) (revision 61ebeba7cc58c61cfb547ecfa0d61cd091bf08dc) +++ lams_tool_images/web/pages/monitoring/monitoring.jsp (.../monitoring.jsp) (revision f8b98e45847fcee40188af4c9db5d13e1bd61e25) @@ -75,6 +75,12 @@ launchPopup(myUrl,"MonitoringReview"); } + function checkNew(){ + var monitorURL = window.location.href; + document.location.href = monitorURL; + return false; + } + $(document).ready(function(){ $(".tablesorter").tablesorter({ Index: lams_tool_images/web/pages/monitoring/parts/addimage.jsp =================================================================== diff -u -r61ebeba7cc58c61cfb547ecfa0d61cd091bf08dc -rf8b98e45847fcee40188af4c9db5d13e1bd61e25 --- lams_tool_images/web/pages/monitoring/parts/addimage.jsp (.../addimage.jsp) (revision 61ebeba7cc58c61cfb547ecfa0d61cd091bf08dc) +++ lams_tool_images/web/pages/monitoring/parts/addimage.jsp (.../addimage.jsp) (revision f8b98e45847fcee40188af4c9db5d13e1bd61e25) @@ -1,65 +1,161 @@ +<%@ page import="org.lamsfoundation.lams.util.Configuration" %> +<%@ page import="org.lamsfoundation.lams.util.ConfigurationKeys" %> <%@ include file="/common/taglibs.jsp"%> +<%=Configuration.get(ConfigurationKeys.UPLOAD_FILE_LARGE_MAX_SIZE)%> <fmt:message key="label.learning.title" /> <%@ include file="/common/header.jsp"%> + + + +
+ + + +
<%@ include file="/common/messages.jsp"%> - - -
- +
- +
-
- - +
+ + + + +
+ + - - - + + + - Index: lams_tool_images/web/pages/monitoring/parts/addmultipleimages.jsp =================================================================== diff -u -r61ebeba7cc58c61cfb547ecfa0d61cd091bf08dc -rf8b98e45847fcee40188af4c9db5d13e1bd61e25 --- lams_tool_images/web/pages/monitoring/parts/addmultipleimages.jsp (.../addmultipleimages.jsp) (revision 61ebeba7cc58c61cfb547ecfa0d61cd091bf08dc) +++ lams_tool_images/web/pages/monitoring/parts/addmultipleimages.jsp (.../addmultipleimages.jsp) (revision f8b98e45847fcee40188af4c9db5d13e1bd61e25) @@ -1,12 +1,95 @@ - +<%@ page import="org.lamsfoundation.lams.util.Configuration" %> +<%@ page import="org.lamsfoundation.lams.util.ConfigurationKeys" %> <%@ include file="/common/taglibs.jsp"%> + +<%=Configuration.get(ConfigurationKeys.UPLOAD_FILE_LARGE_MAX_SIZE)%> + <fmt:message key="label.learning.title" /> <%@ include file="/common/header.jsp"%> + @@ -15,6 +98,10 @@ @@ -23,45 +110,138 @@ <%@ include file="/common/messages.jsp"%> - - +
- -
-
- -
-
- -
-
- -
-
- -
-
+
+ + + + + +
+ + - - - - +
+ + + + + +
+ + + +
+ + + + + +
+ + + +
+ + + + + +
+ + + +
+ + + + + +
+ + +
+
-
- - - + + +