Skip to content
Snippets Groups Projects
Commit 5b9f893a authored by srosse's avatar srosse
Browse files

OO-538: course wizard must hold the course from the course factory if it open a new edit session

parent 824d4123
No related branches found
No related tags found
No related merge requests found
...@@ -118,7 +118,7 @@ public class CourseExtensionHelper { ...@@ -118,7 +118,7 @@ public class CourseExtensionHelper {
* @param longTitle long title for node * @param longTitle long title for node
* @return created course node * @return created course node
*/ */
private static final CourseNode createNode(final ICourse course, final String shortTitle, final String longTitle, final String type) { private static final CourseNode createNode(ICourse course, final String shortTitle, final String longTitle, final String type) {
// create a node with default data // create a node with default data
CourseNodeConfiguration nodeConfig = CourseNodeFactory.getInstance().getCourseNodeConfiguration(type); CourseNodeConfiguration nodeConfig = CourseNodeFactory.getInstance().getCourseNodeConfiguration(type);
CourseNode node = nodeConfig.getInstance(); CourseNode node = nodeConfig.getInstance();
...@@ -126,7 +126,7 @@ public class CourseExtensionHelper { ...@@ -126,7 +126,7 @@ public class CourseExtensionHelper {
node.setLongTitle(longTitle); node.setLongTitle(longTitle);
// append node to course // append node to course
CourseFactory.openCourseEditSession(course.getResourceableId()); course = CourseFactory.openCourseEditSession(course.getResourceableId());
final CourseEditorTreeModel cetm = course.getEditorTreeModel(); final CourseEditorTreeModel cetm = course.getEditorTreeModel();
final CourseNode rootNode = cetm.getCourseNode(course.getRunStructure().getRootNode().getIdent()); final CourseNode rootNode = cetm.getCourseNode(course.getRunStructure().getRootNode().getIdent());
course.getEditorTreeModel().addCourseNode(node, rootNode); course.getEditorTreeModel().addCourseNode(node, rootNode);
...@@ -143,8 +143,8 @@ public class CourseExtensionHelper { ...@@ -143,8 +143,8 @@ public class CourseExtensionHelper {
* @param node node to be removed * @param node node to be removed
* @param course course where the node belongs to * @param course course where the node belongs to
*/ */
public static final void removeNode(final CourseNode node, final ICourse course) { public static final void removeNode(final CourseNode node, ICourse course) {
CourseFactory.openCourseEditSession(course.getResourceableId()); course = CourseFactory.openCourseEditSession(course.getResourceableId());
course.getEditorTreeModel().removeCourseNode(node); course.getEditorTreeModel().removeCourseNode(node);
CourseFactory.saveCourseEditorTreeModel(course.getResourceableId()); CourseFactory.saveCourseEditorTreeModel(course.getResourceableId());
CourseFactory.closeCourseEditSession(course.getResourceableId(), true); CourseFactory.closeCourseEditSession(course.getResourceableId(), true);
......
...@@ -302,7 +302,7 @@ public class CourseCreationHelper { ...@@ -302,7 +302,7 @@ public class CourseCreationHelper {
addedEntry = RepositoryManager.getInstance().setAccess(addedEntry, access, false); addedEntry = RepositoryManager.getInstance().setAccess(addedEntry, access, false);
} }
CourseFactory.openCourseEditSession(course.getResourceableId()); course = CourseFactory.openCourseEditSession(course.getResourceableId());
course.getRunStructure().getRootNode().setShortTitle(addedEntry.getDisplayname()); course.getRunStructure().getRootNode().setShortTitle(addedEntry.getDisplayname());
course.getRunStructure().getRootNode().setLongTitle(addedEntry.getDisplayname()); course.getRunStructure().getRootNode().setLongTitle(addedEntry.getDisplayname());
CourseFactory.saveCourse(course.getResourceableId()); CourseFactory.saveCourse(course.getResourceableId());
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment