Issue #3513033 by spiderman: Too few arguments to function WorkflowEntityHooks::userDelete()

src/Entity/WorkflowTargetEntity.php

@@ -7,7 +7,7 @@ use Drupal\Core\Entity\EntityInterface;
 /**
  * Provides a wrapper/ decorator for the $transition->getTargetEntity().
  *
- * @todo Change all sttaic functions to non static.
+ * @todo Change all static functions to non static.
  * But a decorator requires duplciating many functions...
  */
 class WorkflowTargetEntity {

src/Hook/WorkflowEntityHooks.php

@@ -115,7 +115,11 @@ class WorkflowEntityHooks {
         }
         break;
 
-      case !WorkflowTargetEntity::isWorkflowEntityType($entity->getEntityTypeId()):
+      case WorkflowTargetEntity::isWorkflowEntityType($entity->getEntityTypeId()):
+        // A Workflow entity.
+        break;
+
+      default:
         // A 'normal' entity is deleted.
         foreach ($fields = _workflow_info_fields($entity) as $field_id => $field_storage) {
           $field_name = $field_storage->getName();
@@ -124,10 +128,6 @@ class WorkflowEntityHooks {
           $this->deleteTransitionsOfEntity($entity, 'workflow_transition', $field_name);
         }
         break;
-
-      default:
-        // A Workflow entity.
-        break;
     }
   }
 
@@ -222,7 +222,7 @@ class WorkflowEntityHooks {
    * @todo Hook hook_user_delete does not exist. hook_ENTITY_TYPE_delete?
    */
   #[Hook('user_delete')]
-  public function userDelete($edit, AccountInterface $account, $method) {
+  public function userDelete($account) {
     $this->userCancel([], $account, 'user_cancel_delete');
   }