- minor code improvement

attila.meszaros · attila.meszaros · commit e1dd5d27451b · 2020-07-12T17:12:32.000+02:00
diff --git a/operator-framework/src/main/java/com/github/containersolutions/operator/processing/EventDispatcher.java b/operator-framework/src/main/java/com/github/containersolutions/operator/processing/EventDispatcher.java
@@ -43,10 +43,10 @@ public void handleEvent(CustomResourceEvent event) {
             log.error("Received error for resource: {}", resource.getMetadata().getName());
             return;
         }
-        Context context = new DefaultContext(new RetryInfo(event.getRetryCount(), event.getRetryExecution().isLastExecution()));
         if (markedForDeletion(resource) && !ControllerUtils.hasDefaultFinalizer(resource, resourceDefaultFinalizer)) {
             return;
         }
+        Context context = new DefaultContext(new RetryInfo(event.getRetryCount(), event.getRetryExecution().isLastExecution()));
         if (markedForDeletion(resource)) {
             boolean removeFinalizer = controller.deleteResource(resource, context);
             if (removeFinalizer && ControllerUtils.hasDefaultFinalizer(resource, resourceDefaultFinalizer)) {

Original file line number	Diff line number	Diff line change
`@@ -43,10 +43,10 @@ public void handleEvent(CustomResourceEvent event) {`
`43`	`43`	`log.error("Received error for resource: {}", resource.getMetadata().getName());`
`44`	`44`	`return;`
`45`	`45`	`}`
`46`		`- Context context = new DefaultContext(new RetryInfo(event.getRetryCount(), event.getRetryExecution().isLastExecution()));`
`47`	`46`	`if (markedForDeletion(resource) && !ControllerUtils.hasDefaultFinalizer(resource, resourceDefaultFinalizer)) {`
`48`	`47`	`return;`
`49`	`48`	`}`
	`49`	`+ Context context = new DefaultContext(new RetryInfo(event.getRetryCount(), event.getRetryExecution().isLastExecution()));`
`50`	`50`	`if (markedForDeletion(resource)) {`
`51`	`51`	`boolean removeFinalizer = controller.deleteResource(resource, context);`
`52`	`52`	`if (removeFinalizer && ControllerUtils.hasDefaultFinalizer(resource, resourceDefaultFinalizer)) {`