fix: Some users are wrongly flagged as primary owners

Closes gravitee-io/issues#3273
gravitee-io · Mar 5, 2020 · 3603584 · 3603584
1 parent d0547b3
commit 3603584
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/...api-service/src/main/java/io/gravitee/management/service/impl/ApplicationServiceImpl.java b/...api-service/src/main/java/io/gravitee/management/service/impl/ApplicationServiceImpl.java
@@ -452,6 +452,8 @@ public void archive(String applicationId) {
             applicationRepository.update(application);
             // remove notifications
             genericNotificationConfigService.deleteReference(NotificationReferenceType.APPLICATION, applicationId);
+            // remove memberships
+            membershipRepository.deleteMembers(MembershipReferenceType.APPLICATION, applicationId);
             // Audit
             auditService.createApplicationAuditLog(
                     application.getId(),

diff --git a/...-service/src/test/java/io/gravitee/management/service/ApplicationService_ArchiveTest.java b/...-service/src/test/java/io/gravitee/management/service/ApplicationService_ArchiveTest.java
@@ -21,6 +21,7 @@
 import io.gravitee.management.service.impl.ApplicationServiceImpl;
 import io.gravitee.repository.exceptions.TechnicalException;
 import io.gravitee.repository.management.api.ApplicationRepository;
+import io.gravitee.repository.management.api.MembershipRepository;
 import io.gravitee.repository.management.model.Application;
 import io.gravitee.repository.management.model.ApplicationStatus;
 import org.junit.Assert;
@@ -70,6 +71,8 @@ public class ApplicationService_ArchiveTest {
     private AuditService auditService;
     @Mock
     private GenericNotificationConfigService genericNotificationConfigService;
+    @Mock
+    private MembershipRepository membershipRepository;
 
     @Test
     public void shouldArchive() throws TechnicalException {