Merge pull request #3636 from ingef/fix/log-message

fix: adds missing parameter
ingef · Dec 10, 2024 · 18e1f87 · 18e1f87
2 parents 13ed9dd + b06f30c
commit 18e1f87
Showing 1 changed file with 8 additions and 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
@@ -133,7 +133,7 @@ public static <E extends IdentifiableImpl<ID> & Owned, ID extends Id<E>> 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 <E extends IdentifiableImpl<ID> & Owned, ID extends Id<E>> 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;
 				}