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(/
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(/