Remove redundant capability checks

Just a minor code improvement.
Evolveum · Nov 7, 2023 · ade50c9 · ade50c9
1 parent d08ef80
commit ade50c9
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 17 deletions.
diff --git a/...om/evolveum/midpoint/provisioning/impl/resourceobjects/ResourceObjectDeleteOperation.java b/...om/evolveum/midpoint/provisioning/impl/resourceobjects/ResourceObjectDeleteOperation.java
@@ -95,11 +95,6 @@ private AsynchronousOperationResult doExecute(OperationResult result)
                         SchemaDebugUtil.debugDump(identifiers));
             }
 
-            if (!ResourceTypeUtil.isDeleteCapabilityEnabled(ctx.getResource())) {
-                throw new UnsupportedOperationException(
-                        String.format("Resource does not support 'delete' operation (%s)", ctx.getExceptionDescription()));
-            }
-
             connectorAsyncOpRet =
                     connector.deleteObject(
                             ctx.getObjectDefinitionRequired(),

diff --git a/...evolveum/midpoint/provisioning/impl/resourceobjects/ResourceObjectUcfModifyOperation.java b/...evolveum/midpoint/provisioning/impl/resourceobjects/ResourceObjectUcfModifyOperation.java
@@ -179,18 +179,6 @@ private AsynchronousOperationReturnValue<Collection<PropertyModificationOperatio
                         primaryIdentification.debugDump(1), SchemaDebugUtil.debugDump(operations, 1));
             }
 
-            // TODO wasn't this checked above?
-            if (!ResourceTypeUtil.isUpdateCapabilityEnabled(ctx.getResource())) {
-                if (operations.isEmpty()) {
-                    LOGGER.debug("No modifications for connector object specified (after filtering). Skipping processing.");
-                    result.recordSuccess();
-                    return null;
-                }
-                UnsupportedOperationException e = new UnsupportedOperationException("Resource does not support 'update' operation");
-                result.recordFatalError(e);
-                throw e;
-            }
-
             // because identifiers can be modified e.g. on rename operation (TODO: is this really needed?)
             ResourceObjectIdentification identificationClone = primaryIdentification.clone();
             List<Collection<Operation>> operationsWaves = sortOperationsIntoWaves(operations, objectDefinition);