From b06f30cdde82e16eefa833497d3b7dc8d15de5bf Mon Sep 17 00:00:00 2001 From: Max Thonagel <12283268+thoniTUB@users.noreply.github.com> Date: Tue, 10 Dec 2024 17:28:29 +0100 Subject: [PATCH] adds missing parameter --- .../conquery/tasks/PermissionCleanupTask.java | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/backend/src/main/java/com/bakdata/conquery/tasks/PermissionCleanupTask.java b/backend/src/main/java/com/bakdata/conquery/tasks/PermissionCleanupTask.java index 98c00a29ad..784e8f9620 100644 --- a/backend/src/main/java/com/bakdata/conquery/tasks/PermissionCleanupTask.java +++ b/backend/src/main/java/com/bakdata/conquery/tasks/PermissionCleanupTask.java @@ -133,7 +133,7 @@ public static & Owned, ID extends Id> int del if (wpermission.getInstances().size() != 1) { log.trace("Skipping permission {} because it refers to multiple instances.", wpermission); } - ID executionId = null; + ID executionId; try { executionId = idParser.parse(wpermission.getInstances().iterator().next()); } @@ -144,14 +144,17 @@ public static & Owned, ID extends Id> int del E execution = instanceStorageExtractor.apply(executionId); if (execution == null) { - log.trace("The execution referenced in permission {} does not exist. Skipping permission"); + log.trace("The execution referenced in permission {} does not exist. Skipping permission", wpermission); continue; } if (!user.isOwner(execution)) { - log.trace("The user is not owner of the instance. Keeping the permission. User: {}, Owner: {}, Instance: {}, Permission: {}", user.getId(), execution - .getOwner(), execution - .getId(), wpermission); + log.trace("The user is not owner of the instance. Keeping the permission. User: {}, Owner: {}, Instance: {}, Permission: {}", + user.getId(), + execution.getOwner(), + execution.getId(), + wpermission + ); continue; }