MNT: Minor code cleanup

dnadesign · Aug 12, 2024 · fe70175 · fe70175
1 parent aeaf9cf
commit fe70175
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/Admin/AdvancedWorkflowAdmin.php b/src/Admin/AdvancedWorkflowAdmin.php
@@ -20,6 +20,8 @@
 use SilverStripe\ORM\ArrayList;
 use SilverStripe\ORM\DataList;
 use SilverStripe\ORM\DataObject;
+use SilverStripe\ORM\Filterable;
+use SilverStripe\ORM\SS_List;
 use SilverStripe\Security\Member;
 use SilverStripe\Security\Permission;
 use SilverStripe\Security\Security;
@@ -180,15 +182,15 @@ public function getFieldDependentData(Member $user, string $fieldName): ?DataLis
         $list = null;
 
         if ($fieldName === 'PendingObjects') {
-            $list =  $this->getWorkflowService()->userPendingItems($user);
+            $list = $this->getWorkflowService()->userPendingItems($user);
         }
 
         if ($fieldName === 'SubmittedObjects') {
             $list = $this->getWorkflowService()->userSubmittedItems($user);
         }
 
         // If the list is null, then the user has entered a bad input
-        if (!$list instanceof DataList && !$list instanceof ArrayList) {
+        if (!$list instanceof Filterable) {
             throw new InvalidArgumentException('$fieldName must be one of ("PendingObjects", "SubmittedObjects")');
         }