Automatic DegreeSite creation upon Degree entity creation #25
Conversation
| @@ -72,4 +90,10 @@ public void delete() { | |||
| return Stream.of(categoryForSlug("announcement")).filter(Objects::nonNull).map(Category::makeWrap); | |||
| } | |||
|
|
|||
| private CMSFolder folderForPath(MenuContainer parent, String path) { | |||
There was a problem hiding this comment.
Given that folders are optional in the CMS, it does not make sense to blow up if that particular folder is not found.
One might want not to organize degree sites in folders (or even set them up in a custom, on different path), so I think it would just be better to do nothing in this case.
There was a problem hiding this comment.
Ok, like ExecutionCourseSite example, if parent folder not found, now is created a new one.
Define canPostGroup. Create the folder when it doesnt exists.
|
Squash your commits |
| setCanPostGroup(TeacherResponsibleOfExecutionCourseGroup.get(executionCourse)); | ||
|
|
||
| setPublished(true); | ||
| setBennu(Bennu.getInstance()); |
There was a problem hiding this comment.
This line must be back ported to CMS, most likely this doesn't make sense to be made on this place.
There was a problem hiding this comment.
Which line? You are referring to a whole diff block.
Should be moved where to CMS? Which entity?
There was a problem hiding this comment.
Only Line 70. Attaching the instance to Bennu should be done within the CMS. This is a bug, and should be corrected at CMS. But for now lets leave it here, because this isn't fixed yet on the other module
LRN-20