Index: lams_tool_forum/conf/language/lams/ApplicationResources.properties =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/conf/language/lams/ApplicationResources.properties,v diff -u -r1.39 -r1.40 --- lams_tool_forum/conf/language/lams/ApplicationResources.properties 28 Oct 2010 10:05:49 -0000 1.39 +++ lams_tool_forum/conf/language/lams/ApplicationResources.properties 12 Dec 2010 14:43:29 -0000 1.40 @@ -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 =================================================================== RCS file: /usr/local/cvsroot/lams_tool_forum/web/jsps/authoring/pedagogicalPlannerForm.jsp,v diff -u -r1.8 -r1.9 --- lams_tool_forum/web/jsps/authoring/pedagogicalPlannerForm.jsp 6 Oct 2010 10:16:15 -0000 1.8 +++ lams_tool_forum/web/jsps/authoring/pedagogicalPlannerForm.jsp 12 Dec 2010 14:43:29 -0000 1.9 @@ -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 =================================================================== RCS file: /usr/local/cvsroot/lams_tool_larsrc/web/pages/authoring/pedagogicalPlannerForm.jsp,v diff -u -r1.7 -r1.8 --- lams_tool_larsrc/web/pages/authoring/pedagogicalPlannerForm.jsp 6 Oct 2010 10:17:38 -0000 1.7 +++ lams_tool_larsrc/web/pages/authoring/pedagogicalPlannerForm.jsp 12 Dec 2010 14:44:00 -0000 1.8 @@ -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 =================================================================== RCS file: /usr/local/cvsroot/lams_tool_scribe/web/pages/authoring/pedagogicalPlannerForm.jsp,v diff -u -r1.2 -r1.3 --- lams_tool_scribe/web/pages/authoring/pedagogicalPlannerForm.jsp 6 Oct 2010 10:18:04 -0000 1.2 +++ lams_tool_scribe/web/pages/authoring/pedagogicalPlannerForm.jsp 12 Dec 2010 14:43:39 -0000 1.3 @@ -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(/