Skip to content

Commit 30ca14d

Browse files
committed
invalid logic
1 parent 226f7d0 commit 30ca14d

1 file changed

Lines changed: 5 additions & 13 deletions

File tree

sormas-backend/src/main/java/de/symeda/sormas/backend/user/UserService.java

Lines changed: 5 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -241,7 +241,7 @@ public List<UserReference> getUserReferences(
241241
CriteriaBuilderHelper.or(
242242
cb,
243243
cb.in(regionJoin.get(AbstractDomainObject.UUID)).value(regionUuids),
244-
createCurrentUserHasNationJurisdictionFilter(cb, userRoot)));
244+
createUserHasNationJurisdictionFilter(cb, userRoot)));
245245
userEntityJoinUsed = true;
246246
}
247247
if (CollectionUtils.isNotEmpty(districtUuids)) {
@@ -258,7 +258,7 @@ public List<UserReference> getUserReferences(
258258
cb.equal(root.get(UserReference.JURISDICTION_LEVEL), JurisdictionLevel.REGION)));
259259

260260
filter = CriteriaBuilderHelper
261-
.and(cb, filter, CriteriaBuilderHelper.or(cb, districtFilter, createCurrentUserHasNationJurisdictionFilter(cb, userRoot)));
261+
.and(cb, filter, CriteriaBuilderHelper.or(cb, districtFilter, createUserHasNationJurisdictionFilter(cb, userRoot)));
262262
userEntityJoinUsed = true;
263263
}
264264
if (!hasRight(UserRight.SEE_PERSONAL_DATA_OUTSIDE_JURISDICTION)) {
@@ -267,8 +267,7 @@ public List<UserReference> getUserReferences(
267267
filter = CriteriaBuilderHelper.and(
268268
cb,
269269
filter,
270-
CriteriaBuilderHelper
271-
.or(cb, createCurrentUserJurisdictionFilter(cb, userRoot), createCurrentUserHasNationJurisdictionFilter(cb, userRoot)));
270+
CriteriaBuilderHelper.or(cb, createCurrentUserJurisdictionFilter(cb, userRoot), createUserHasNationJurisdictionFilter(cb, userRoot)));
272271
userEntityJoinUsed = true;
273272
}
274273

@@ -306,7 +305,7 @@ public List<UserReference> getUserReferences(
306305
CriteriaBuilderHelper.or(
307306
cb,
308307
cb.in(communityJoin.get(AbstractDomainObject.UUID)).value(communityUuids),
309-
createCurrentUserHasNationJurisdictionFilter(cb, userRoot)));
308+
createUserHasNationJurisdictionFilter(cb, userRoot)));
310309
}
311310

312311
if (filter != null) {
@@ -728,14 +727,7 @@ public Predicate createCurrentUserJurisdictionFilter(CriteriaBuilder cb, From<?,
728727
}
729728
}
730729

731-
public Predicate createCurrentUserHasNationJurisdictionFilter(CriteriaBuilder cb, From<?, User> from) {
732-
User currentUser = getCurrentUser();
733-
734-
if (currentUser.getJurisdictionLevel() == JurisdictionLevel.NATION) {
735-
logger.error("getJurisdictionLevel = NATION");
736-
return cb.equal(from.get(User.JURISDICTION_LEVEL), JurisdictionLevel.NATION);
737-
}
738-
730+
public Predicate createUserHasNationJurisdictionFilter(CriteriaBuilder cb, From<?, User> from) {
739731
return cb.equal(from.get(User.JURISDICTION_LEVEL), JurisdictionLevel.NATION);
740732
}
741733

0 commit comments

Comments
 (0)