Skip to content
Snippets Groups Projects
Commit 47769f4e authored by srosse's avatar srosse
Browse files

no-jira: log audit add / remove owner of question, delete question

parent 3372f99f
No related branches found
No related tags found
No related merge requests found
...@@ -29,6 +29,7 @@ import javax.persistence.EntityManager; ...@@ -29,6 +29,7 @@ import javax.persistence.EntityManager;
import javax.persistence.LockModeType; import javax.persistence.LockModeType;
import javax.persistence.TypedQuery; import javax.persistence.TypedQuery;
import org.apache.logging.log4j.Logger;
import org.olat.basesecurity.GroupRoles; import org.olat.basesecurity.GroupRoles;
import org.olat.basesecurity.SecurityGroup; import org.olat.basesecurity.SecurityGroup;
import org.olat.basesecurity.SecurityGroupMembershipImpl; import org.olat.basesecurity.SecurityGroupMembershipImpl;
...@@ -36,6 +37,7 @@ import org.olat.basesecurity.manager.SecurityGroupDAO; ...@@ -36,6 +37,7 @@ import org.olat.basesecurity.manager.SecurityGroupDAO;
import org.olat.core.commons.persistence.DB; import org.olat.core.commons.persistence.DB;
import org.olat.core.commons.services.mark.impl.MarkImpl; import org.olat.core.commons.services.mark.impl.MarkImpl;
import org.olat.core.id.Identity; import org.olat.core.id.Identity;
import org.olat.core.logging.Tracing;
import org.olat.group.BusinessGroup; import org.olat.group.BusinessGroup;
import org.olat.modules.qpool.QuestionItem; import org.olat.modules.qpool.QuestionItem;
import org.olat.modules.qpool.QuestionItem2Resource; import org.olat.modules.qpool.QuestionItem2Resource;
...@@ -60,6 +62,8 @@ import org.springframework.stereotype.Service; ...@@ -60,6 +62,8 @@ import org.springframework.stereotype.Service;
@Service("questionDao") @Service("questionDao")
public class QuestionItemDAO { public class QuestionItemDAO {
private static final Logger log = Tracing.createLoggerFor(QuestionItemDAO.class);
@Autowired @Autowired
private DB dbInstance; private DB dbInstance;
@Autowired @Autowired
...@@ -177,6 +181,8 @@ public class QuestionItemDAO { ...@@ -177,6 +181,8 @@ public class QuestionItemDAO {
for(Identity author:authors) { for(Identity author:authors) {
if(!securityGroupDao.isIdentityInSecurityGroup(author, secGroup)) { if(!securityGroupDao.isIdentityInSecurityGroup(author, secGroup)) {
securityGroupDao.addIdentityToSecurityGroup(author, secGroup); securityGroupDao.addIdentityToSecurityGroup(author, secGroup);
log.info(Tracing.M_AUDIT, "Added owner identity '{}' to item with key {} ({}, {})",
author.getKey(), item.getKey(), item.getTitle(), item.getTopic());
} }
} }
dbInstance.commit(); dbInstance.commit();
...@@ -190,6 +196,8 @@ public class QuestionItemDAO { ...@@ -190,6 +196,8 @@ public class QuestionItemDAO {
for(Identity author:authors) { for(Identity author:authors) {
if(securityGroupDao.isIdentityInSecurityGroup(author, secGroup)) { if(securityGroupDao.isIdentityInSecurityGroup(author, secGroup)) {
securityGroupDao.removeIdentityFromSecurityGroup(author, secGroup); securityGroupDao.removeIdentityFromSecurityGroup(author, secGroup);
log.info(Tracing.M_AUDIT, "Removed owner identity '{}' from item with key {} ({}, {})",
author.getKey(), item.getKey(), item.getTitle(), item.getTopic());
} }
} }
dbInstance.commit(); dbInstance.commit();
...@@ -261,6 +269,7 @@ public class QuestionItemDAO { ...@@ -261,6 +269,7 @@ public class QuestionItemDAO {
for(QuestionItemShort item:items) { for(QuestionItemShort item:items) {
QuestionItem refItem = loadLazyReferenceId(item.getKey()); QuestionItem refItem = loadLazyReferenceId(item.getKey());
if(refItem != null) { if(refItem != null) {
log.info(Tracing.M_AUDIT, "Delete question item {} ({}, {})", item.getKey(), item.getTitle(), item.getTopic());
em.remove(refItem); em.remove(refItem);
} }
} }
......
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