Skip to content

Commit 1c1bb8a

Browse files
Issue 52614: AssertionError deleting site group from its details page (#919)
* Issue 52614: AssertionError deleting site group from its details page * Switch to use Container for member variables too
1 parent ac5a183 commit 1c1bb8a

File tree

3 files changed

+10
-4
lines changed

3 files changed

+10
-4
lines changed

ehr/api-src/org/labkey/api/ehr/security/EHRSecurityEscalator.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -65,8 +65,8 @@ protected ThreadLocal<Integer> getEscalationLevelTracker() {
6565
* @return A blank new {@link SecurityEscalationAuditProvider.SecurityEscalationEvent}.
6666
*/
6767
@Override
68-
protected SecurityEscalationAuditProvider.SecurityEscalationEvent getNewSecurityEvent() {
69-
return new EHRSecurityEscalatorAuditProvider.EHRSecurityEscalationEvent();
68+
protected SecurityEscalationAuditProvider.SecurityEscalationEvent getNewSecurityEvent(Container container, String comment) {
69+
return new EHRSecurityEscalatorAuditProvider.EHRSecurityEscalationEvent(container, comment);
7070
}
7171

7272
/**

ehr/api-src/org/labkey/api/ehr/security/EHRSecurityEscalatorAuditProvider.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616
package org.labkey.api.ehr.security;
1717

1818
import org.labkey.api.audit.query.AbstractAuditDomainKind;
19+
import org.labkey.api.data.Container;
1920
import org.labkey.api.study.security.SecurityEscalationAuditProvider;
2021

2122
/**
@@ -53,6 +54,11 @@ protected AbstractAuditDomainKind getDomainKind() {
5354

5455
public static class EHRSecurityEscalationEvent extends SecurityEscalationEvent
5556
{
57+
public EHRSecurityEscalationEvent(Container container, String comment)
58+
{
59+
super(EVENT_TYPE, container, comment);
60+
}
61+
5662
@Override
5763
public String getEventType() {
5864
return EVENT_TYPE;

snd/src/org/labkey/snd/NarrativeAuditProvider.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ public <K extends AuditTypeEvent> Class<K> getEventClass()
9595

9696
public static void addAuditEntry(Container container, User user, Integer eventId, String subjectId, Date eventDate, String narrative, Integer qcState, String comment)
9797
{
98-
NarrativeAuditProvider.NarrativeAuditTypeEvent event = new NarrativeAuditProvider.NarrativeAuditTypeEvent(container.getId(), comment);
98+
NarrativeAuditProvider.NarrativeAuditTypeEvent event = new NarrativeAuditProvider.NarrativeAuditTypeEvent(container, comment);
9999
event.setNarrative(narrative);
100100
event.setEventId(eventId);
101101
event.setSubjectId(subjectId);
@@ -118,7 +118,7 @@ public NarrativeAuditTypeEvent()
118118
super();
119119
}
120120

121-
public NarrativeAuditTypeEvent(String container, String comment)
121+
public NarrativeAuditTypeEvent(Container container, String comment)
122122
{
123123
super(NARRATIVE_AUDIT_EVENT, container, comment);
124124
}

0 commit comments

Comments
 (0)