Remove soft delete group member (#765)
Change-Id: I371f47e17dafd41f4d1dceaf0cae5a94491d980c
diff --git a/src/main/java/de/ids_mannheim/korap/config/FullConfiguration.java b/src/main/java/de/ids_mannheim/korap/config/FullConfiguration.java
index 0b92706..6a55ece 100644
--- a/src/main/java/de/ids_mannheim/korap/config/FullConfiguration.java
+++ b/src/main/java/de/ids_mannheim/korap/config/FullConfiguration.java
@@ -47,9 +47,6 @@
private String authenticationScheme;
- private boolean isSoftDeleteAutoGroup;
- private boolean isSoftDeleteGroupMember;
-
private EncryptionIface.Encryption secureHashAlgorithm;
private AuthenticationMethod OAuth2passwordAuthentication;
@@ -88,7 +85,6 @@
// EM: pattern for matching availability in Krill matches
setLicensePatterns(properties);
- setDeleteConfiguration(properties);
setMailConfiguration(properties);
ldapConfig = properties.getProperty("ldap.config");
@@ -168,19 +164,6 @@
}
}
- private void setDeleteConfiguration (Properties properties) {
-// setSoftDeleteGroup(
-// parseDeleteConfig(properties.getProperty("delete.group", "")));
- setSoftDeleteAutoGroup(parseDeleteConfig(
- properties.getProperty("delete.auto.group", "")));
- setSoftDeleteGroupMember(parseDeleteConfig(
- properties.getProperty("delete.group.member", "")));
- }
-
- private boolean parseDeleteConfig (String deleteConfig) {
- return deleteConfig.equals("soft") ? true : false;
- }
-
private void setLicensePatterns (Properties properties) {
setFreeLicensePattern(compilePattern(getFreeOnlyRegex()));
setPublicLicensePattern(compilePattern(
@@ -310,22 +293,6 @@
this.allOnlyRegex = allOnlyRegex;
}
- public boolean isSoftDeleteGroupMember () {
- return isSoftDeleteGroupMember;
- }
-
- public void setSoftDeleteGroupMember (boolean isSoftDeleteGroupMember) {
- this.isSoftDeleteGroupMember = isSoftDeleteGroupMember;
- }
-
- public boolean isSoftDeleteAutoGroup () {
- return isSoftDeleteAutoGroup;
- }
-
- public void setSoftDeleteAutoGroup (boolean isSoftDeleteAutoGroup) {
- this.isSoftDeleteAutoGroup = isSoftDeleteAutoGroup;
- }
-
public String getTestEmail () {
return testEmail;
}
diff --git a/src/main/java/de/ids_mannheim/korap/constant/GroupMemberStatus.java b/src/main/java/de/ids_mannheim/korap/constant/GroupMemberStatus.java
index 445e31e..87f3d04 100644
--- a/src/main/java/de/ids_mannheim/korap/constant/GroupMemberStatus.java
+++ b/src/main/java/de/ids_mannheim/korap/constant/GroupMemberStatus.java
@@ -10,8 +10,5 @@
ACTIVE,
// membership invitation was sent and has not been accepted
// or rejected yet
- PENDING,
- // either membership invitation was rejected or the member was
- // deleted by a user-group admin
- DELETED;
+ PENDING;
}
diff --git a/src/main/java/de/ids_mannheim/korap/dao/UserGroupDao.java b/src/main/java/de/ids_mannheim/korap/dao/UserGroupDao.java
index 38ab977..3d99612 100644
--- a/src/main/java/de/ids_mannheim/korap/dao/UserGroupDao.java
+++ b/src/main/java/de/ids_mannheim/korap/dao/UserGroupDao.java
@@ -195,9 +195,7 @@
criteriaBuilder.equal(root.get(UserGroup_.status),
UserGroupStatus.ACTIVE),
criteriaBuilder.equal(members.get(UserGroupMember_.userId),
- userId),
- criteriaBuilder.notEqual(members.get(UserGroupMember_.status),
- GroupMemberStatus.DELETED));
+ userId));
// criteriaBuilder.equal(members.get(UserGroupMember_.status),
// GroupMemberStatus.ACTIVE));
diff --git a/src/main/java/de/ids_mannheim/korap/dao/UserGroupMemberDao.java b/src/main/java/de/ids_mannheim/korap/dao/UserGroupMemberDao.java
index d7b57bd..4909019 100644
--- a/src/main/java/de/ids_mannheim/korap/dao/UserGroupMemberDao.java
+++ b/src/main/java/de/ids_mannheim/korap/dao/UserGroupMemberDao.java
@@ -53,8 +53,8 @@
entityManager.merge(member);
}
- public void deleteMember (UserGroupMember member, String deletedBy,
- boolean isSoftDelete) throws KustvaktException {
+ public void deleteMember (UserGroupMember member, String deletedBy)
+ throws KustvaktException {
ParameterChecker.checkObjectValue(member, "UserGroupMember");
ParameterChecker.checkStringValue(deletedBy, "deletedBy");
@@ -62,16 +62,8 @@
member = entityManager.merge(member);
}
- if (isSoftDelete) {
- member.setStatus(GroupMemberStatus.DELETED);
- member.setDeletedBy(deletedBy);
- member.setRoles(new HashSet<>());
- entityManager.persist(member);
- }
- else {
- member.setRoles(new HashSet<>());
- entityManager.remove(member);
- }
+ member.setRoles(new HashSet<>());
+ entityManager.remove(member);
}
public UserGroupMember retrieveMemberById (String userId, int groupId)
@@ -138,14 +130,9 @@
}
}
+ @SuppressWarnings("unchecked")
public List<UserGroupMember> retrieveMemberByGroupId (int groupId)
throws KustvaktException {
- return retrieveMemberByGroupId(groupId, false);
- }
-
- @SuppressWarnings("unchecked")
- public List<UserGroupMember> retrieveMemberByGroupId (int groupId,
- boolean isAdmin) throws KustvaktException {
CriteriaBuilder criteriaBuilder = entityManager.getCriteriaBuilder();
CriteriaQuery<UserGroupMember> query = criteriaBuilder
.createQuery(UserGroupMember.class);
@@ -155,12 +142,6 @@
Predicate predicate = criteriaBuilder.and(criteriaBuilder
.equal(root.get(UserGroupMember_.group), groupId));
- if (!isAdmin) {
- predicate = criteriaBuilder.and(predicate,
- criteriaBuilder.notEqual(root.get(UserGroupMember_.status),
- GroupMemberStatus.DELETED));
- }
-
query.select(root);
query.where(predicate);
Query q = entityManager.createQuery(query);
diff --git a/src/main/java/de/ids_mannheim/korap/service/UserGroupService.java b/src/main/java/de/ids_mannheim/korap/service/UserGroupService.java
index ccb65ad..7456ee4 100644
--- a/src/main/java/de/ids_mannheim/korap/service/UserGroupService.java
+++ b/src/main/java/de/ids_mannheim/korap/service/UserGroupService.java
@@ -162,8 +162,7 @@
List<UserGroupMember> members;
UserGroupDto groupDto;
for (UserGroup group : userGroups) {
- members = groupMemberDao.retrieveMemberByGroupId(group.getId(),
- true);
+ members = groupMemberDao.retrieveMemberByGroupId(group.getId());
groupDto = converter.createUserGroupDto(group, members, null, null);
dtos.add(groupDto);
}
@@ -370,11 +369,6 @@
|| existingStatus.equals(status)) {
return existingStatus;
}
- else if (existingStatus.equals(GroupMemberStatus.DELETED)) {
- // hard delete, not customizable
- doDeleteMember(username, groupId, "system", false);
- }
-
return null;
}
@@ -445,13 +439,7 @@
UserGroupMember member = groupMemberDao.retrieveMemberById(username,
userGroup.getId());
GroupMemberStatus status = member.getStatus();
- if (status.equals(GroupMemberStatus.DELETED)) {
- throw new KustvaktException(StatusCodes.GROUP_MEMBER_DELETED,
- username + " has already been deleted from the group "
- + userGroup.getName(),
- username, userGroup.getName());
- }
- else if (member.getStatus().equals(GroupMemberStatus.ACTIVE)) {
+ if (member.getStatus().equals(GroupMemberStatus.ACTIVE)) {
throw new KustvaktException(StatusCodes.GROUP_MEMBER_EXISTS,
"Username " + username + " with status " + status
+ " exists in the user-group "
@@ -506,9 +494,9 @@
else if (memberId.equals(deletedBy)
|| isUserGroupAdmin(deletedBy, userGroup)
|| adminDao.isAdmin(deletedBy)) {
- // soft delete
- doDeleteMember(memberId, userGroup.getId(), deletedBy,
- config.isSoftDeleteGroupMember());
+ UserGroupMember member = groupMemberDao.retrieveMemberById(memberId,
+ userGroup.getId());
+ groupMemberDao.deleteMember(member, deletedBy);
}
else {
throw new KustvaktException(StatusCodes.AUTHORIZATION_FAILED,
@@ -516,39 +504,6 @@
}
}
- /**
- * Updates the {@link GroupMemberStatus} of a member to
- * {@link GroupMemberStatus#DELETED}
- *
- * @param userId
- * user to be deleted
- * @param groupId
- * user-group id
- * @param deletedBy
- * user that issue the delete
- * @param isSoftDelete
- * true if database entry is to be deleted
- * permanently, false otherwise
- * @throws KustvaktException
- */
- private void doDeleteMember (String username, int groupId, String deletedBy,
- boolean isSoftDelete) throws KustvaktException {
-
- UserGroup group = userGroupDao.retrieveGroupById(groupId);
-
- UserGroupMember member = groupMemberDao.retrieveMemberById(username,
- groupId);
- GroupMemberStatus status = member.getStatus();
- if (isSoftDelete && status.equals(GroupMemberStatus.DELETED)) {
- throw new KustvaktException(StatusCodes.GROUP_MEMBER_DELETED,
- username + " has already been deleted from the group "
- + group.getName(),
- username, group.getName());
- }
-
- groupMemberDao.deleteMember(member, deletedBy, isSoftDelete);
- }
-
public UserGroupDto searchByName (String groupName)
throws KustvaktException {
UserGroup userGroup = userGroupDao.retrieveGroupByName(groupName, true);