Index: lams_tool_forum/conf/language/lams/ApplicationResources.properties =================================================================== diff -u -r09963edb439b0a4ddaa251fa19cb2c188310e56c -r3bfe3be57c0b137a2ec2eb19e895492e9a3ec3aa --- lams_tool_forum/conf/language/lams/ApplicationResources.properties (.../ApplicationResources.properties) (revision 09963edb439b0a4ddaa251fa19cb2c188310e56c) +++ lams_tool_forum/conf/language/lams/ApplicationResources.properties (.../ApplicationResources.properties) (revision 3bfe3be57c0b137a2ec2eb19e895492e9a3ec3aa) @@ -221,6 +221,6 @@ label.authoring.advanced.teachers =teachers event.newposting.subject =Forum: {0} event.newposting.body ={0}:\n{1}\n\n-- email sent automatically by LAMS +label.planner.topic=Topic - #======= End labels: Exported 215 labels for en AU ===== Index: lams_tool_forum/web/jsps/authoring/pedagogicalPlannerForm.jsp =================================================================== diff -u -r8179037dc958585c054547eb7de9be14c6aed233 -r3bfe3be57c0b137a2ec2eb19e895492e9a3ec3aa --- lams_tool_forum/web/jsps/authoring/pedagogicalPlannerForm.jsp (.../pedagogicalPlannerForm.jsp) (revision 8179037dc958585c054547eb7de9be14c6aed233) +++ lams_tool_forum/web/jsps/authoring/pedagogicalPlannerForm.jsp (.../pedagogicalPlannerForm.jsp) (revision 3bfe3be57c0b137a2ec2eb19e895492e9a3ec3aa) @@ -41,7 +41,10 @@ $('#pedagogicalPlannerForm').ajaxSubmit({ url: "", success: function(responseText){ - $('#body').html(responseText.substring(responseText.search(/", success: function(responseText){ - $('#body').html(responseText.substring(responseText.search(/", success: function(responseText){ - $('#body').html(responseText.substring(responseText.search(/ newItems = new LinkedList(); - Set resourceItems = taskList.getResourceItems(); + // we need a copy for later Hibernate-bound processing + LinkedList resourceItems = new LinkedList(taskList.getResourceItems()); Iterator taskListItemIterator = resourceItems.iterator(); - // We need to reverse the order, since the items are delivered - // newest-first - LinkedList reversedResourceItems = new LinkedList(); - while (taskListItemIterator.hasNext()) { - reversedResourceItems.addFirst(taskListItemIterator.next()); - } - taskListItemIterator = reversedResourceItems.iterator(); + /* + Not the case anymore (why?): + We need to reverse the order, since the items are delivered newest-first + LinkedList reversedResourceItems = new LinkedList(); + while (taskListItemIterator.hasNext()) { + reversedResourceItems.addFirst(taskListItemIterator.next()); + } + taskListItemIterator = reversedResourceItems.iterator(); + */ do { title = plannerForm.getTitle(itemIndex); if (StringUtils.isEmpty(title)) { @@ -1307,11 +1310,12 @@ FormFile file = plannerForm.getFile(itemIndex); resourceItem.setUrl(null); IResourceService service = getResourceService(); - if (file != null) { + if (file != null && !StringUtils.isEmpty(file.getFileName())) { try { if (hasFile) { // delete the old file - service.deleteFromRepository(resourceItem.getFileUuid(), resourceItem.getFileVersionId()); + service.deleteFromRepository(resourceItem.getFileUuid(), + resourceItem.getFileVersionId()); } service.uploadResourceItemFile(resourceItem, file); } catch (Exception e) { @@ -1322,11 +1326,11 @@ plannerForm.setValid(false); return mapping.findForward(ResourceConstants.SUCCESS); } + plannerForm.setFileName(itemIndex, resourceItem.getFileName()); + plannerForm.setFileUuid(itemIndex, resourceItem.getFileUuid()); + plannerForm.setFileVersion(itemIndex, resourceItem.getFileVersionId()); + plannerForm.setFile(itemIndex, null); } - plannerForm.setFileName(itemIndex, resourceItem.getFileName()); - plannerForm.setFileUuid(itemIndex, resourceItem.getFileUuid()); - plannerForm.setFileVersion(itemIndex, resourceItem.getFileVersionId()); - plannerForm.setFile(itemIndex, null); } itemIndex++; } @@ -1340,9 +1344,9 @@ taskListItemIterator.remove(); getResourceService().deleteResourceItem(resourceItem.getUid()); } - reversedResourceItems.addAll(newItems); + resourceItems.addAll(newItems); - taskList.getResourceItems().addAll(reversedResourceItems); + taskList.getResourceItems().addAll(resourceItems); getResourceService().saveOrUpdateResource(taskList); } else { saveErrors(request, errors); Index: lams_tool_larsrc/web/pages/authoring/pedagogicalPlannerForm.jsp =================================================================== diff -u -r8179037dc958585c054547eb7de9be14c6aed233 -r3bfe3be57c0b137a2ec2eb19e895492e9a3ec3aa --- lams_tool_larsrc/web/pages/authoring/pedagogicalPlannerForm.jsp (.../pedagogicalPlannerForm.jsp) (revision 8179037dc958585c054547eb7de9be14c6aed233) +++ lams_tool_larsrc/web/pages/authoring/pedagogicalPlannerForm.jsp (.../pedagogicalPlannerForm.jsp) (revision 3bfe3be57c0b137a2ec2eb19e895492e9a3ec3aa) @@ -53,7 +53,10 @@ } }, success: function(responseText){ - $('#body').html(responseText.substring(responseText.search(/ " + title="" onclick="javascript:clearEntry(${itemIndex-1})" /> Index: lams_tool_scribe/web/pages/authoring/pedagogicalPlannerForm.jsp =================================================================== diff -u -r8179037dc958585c054547eb7de9be14c6aed233 -r3bfe3be57c0b137a2ec2eb19e895492e9a3ec3aa --- lams_tool_scribe/web/pages/authoring/pedagogicalPlannerForm.jsp (.../pedagogicalPlannerForm.jsp) (revision 8179037dc958585c054547eb7de9be14c6aed233) +++ lams_tool_scribe/web/pages/authoring/pedagogicalPlannerForm.jsp (.../pedagogicalPlannerForm.jsp) (revision 3bfe3be57c0b137a2ec2eb19e895492e9a3ec3aa) @@ -30,7 +30,10 @@ $('#pedagogicalPlannerForm').ajaxSubmit({ url: "", success: function(responseText){ - $('#body').html(responseText.substring(responseText.search(/", success: function(responseText){ - $('#body').html(responseText.substring(responseText.search(/", success: function(responseText){ - $('#body').html(responseText.substring(responseText.search(/