Skip to content
Snippets Groups Projects
Commit 2bd7aca1 authored by srosse's avatar srosse
Browse files

no-jira: add audit log for course creation in REST API

parent f97698b1
No related branches found
No related tags found
No related merge requests found
...@@ -533,7 +533,7 @@ public class CoursesWebService { ...@@ -533,7 +533,7 @@ public class CoursesWebService {
OLATResource originalOres = olatResourceManager.findResourceable(src.getOlatResource()); OLATResource originalOres = olatResourceManager.findResourceable(src.getOlatResource());
boolean isAlreadyLocked = handlerFactory.getRepositoryHandler(src).isLocked(originalOres); boolean isAlreadyLocked = handlerFactory.getRepositoryHandler(src).isLocked(originalOres);
LockResult lockResult = handlerFactory.getRepositoryHandler(src).acquireLock(originalOres, ureq.getIdentity()); LockResult lockResult = handlerFactory.getRepositoryHandler(src).acquireLock(originalOres, ureq.getIdentity());
if(lockResult == null || (lockResult != null && lockResult.isSuccess()) && !isAlreadyLocked) { if(lockResult == null || (lockResult != null && lockResult.isSuccess()) && !isAlreadyLocked) {
//create new repo entry //create new repo entry
String name; String name;
...@@ -621,6 +621,8 @@ public class CoursesWebService { ...@@ -621,6 +621,8 @@ public class CoursesWebService {
repositoryManager.copyImage(src, preparedEntry); repositoryManager.copyImage(src, preparedEntry);
ICourse course = prepareCourse(preparedEntry, shortTitle, longTitle, courseConfigVO); ICourse course = prepareCourse(preparedEntry, shortTitle, longTitle, courseConfigVO);
handlerFactory.getRepositoryHandler(src).releaseLock(lockResult); handlerFactory.getRepositoryHandler(src).releaseLock(lockResult);
log.info(Tracing.M_AUDIT, "Create course {} from template {}", preparedEntry, src);
return course; return course;
} else { } else {
log.info("Course locked"); log.info("Course locked");
...@@ -687,6 +689,7 @@ public class CoursesWebService { ...@@ -687,6 +689,7 @@ public class CoursesWebService {
// create an empty course // create an empty course
CourseFactory.createCourse(addedEntry, shortTitle, longTitle, ""); CourseFactory.createCourse(addedEntry, shortTitle, longTitle, "");
log.info(Tracing.M_AUDIT, "Create course: {}", addedEntry);
return prepareCourse(addedEntry, shortTitle, longTitle, courseConfigVO); return prepareCourse(addedEntry, shortTitle, longTitle, courseConfigVO);
} catch (Exception e) { } catch (Exception e) {
throw new WebApplicationException(e); throw new WebApplicationException(e);
......
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