Index: lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/service/MonitoringService.java =================================================================== diff -u -rb7cec35daf3ceab433021f88c904e707b4512061 -r97bcad4dce01d8533b99820e75b612bd185500ed --- lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/service/MonitoringService.java (.../MonitoringService.java) (revision b7cec35daf3ceab433021f88c904e707b4512061) +++ lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/service/MonitoringService.java (.../MonitoringService.java) (revision 97bcad4dce01d8533b99820e75b612bd185500ed) @@ -436,7 +436,11 @@ authoringService.saveLearningDesign(copiedLearningDesign); - Lesson lesson = createNewLesson(lessonName,lessonDescription,user,learnerExportAvailable,copiedLearningDesign); + // Make all efforts to make sure it has a title + String title = lessonName != null ? lessonName : copiedLearningDesign.getTitle(); + title = title != null ? title : "Unknown Lesson"; + + Lesson lesson = createNewLesson(title,lessonDescription,user,learnerExportAvailable,copiedLearningDesign); auditAction(AUDIT_LESSON_CREATED_KEY, new Object[] {lessonName, copiedLearningDesign.getTitle(), learnerExportAvailable}); return lesson; } @@ -453,16 +457,10 @@ // parse WDDX values String title = WDDXProcessor.convertToString("lessonName", table.get("lessonName")); - if ( title == null ) title = "lesson"; // TODO Use getMessage - String desc = WDDXProcessor.convertToString("lessonDescription", table.get("lessonDescription")); - if ( desc == null ) desc = "description"; // TODO Use getMessage - int copyType = WDDXProcessor.convertToInt("copyType", table.get("copyType")); - Integer organisationId = WDDXProcessor.convertToInteger("organisationID", table.get("organisationID")); long ldId = WDDXProcessor.convertToLong(AttributeNames.PARAM_LEARNINGDESIGN_ID, table.get(AttributeNames.PARAM_LEARNINGDESIGN_ID)); - boolean learnerExportAvailable = WDDXProcessor.convertToBoolean("learnerExportPortfolio", table.get("learnerExportPortfolio")); // initialize lesson