ENH Add generic types

src/Actions/AssignUsersToWorkflowAction.php

@@ -20,6 +20,9 @@
  * @license    BSD License (http://silverstripe.org/bsd-license/)
  * @package    advancedworkflow
  * @subpackage actions
+ *
+ * @method \SilverStripe\ORM\ManyManyList<Member> Users()
+ * @method \SilverStripe\ORM\ManyManyList<Group> Groups()
  */
 class AssignUsersToWorkflowAction extends WorkflowAction
 {
@@ -87,7 +90,7 @@ public function fieldLabels($relations = true)
     /**
      * Returns a set of all Members that are assigned to this WorkflowAction subclass, either directly or via a group.
      *
-     * @return ArrayList
+     * @return ArrayList<Member>
      */
     public function getAssignedMembers()
     {

src/Admin/AdvancedWorkflowAdmin.php

@@ -17,12 +17,14 @@
 use SilverStripe\Forms\GridField\GridFieldImportButton;
 use SilverStripe\Forms\GridField\GridFieldPaginator;
 use SilverStripe\ORM\ArrayList;
+use SilverStripe\ORM\DataList;
 use SilverStripe\ORM\DataObject;
 use SilverStripe\Security\Member;
 use SilverStripe\Security\Permission;
 use SilverStripe\Security\Security;
 use SilverStripe\View\Requirements;
 use Symbiote\AdvancedWorkflow\DataObjects\WorkflowDefinition;
+use Symbiote\AdvancedWorkflow\DataObjects\WorkflowInstance;
 use Symbiote\AdvancedWorkflow\Dev\WorkflowBulkLoader;
 use Symbiote\AdvancedWorkflow\Forms\GridField\GridFieldExportAction;
 use Symbiote\AdvancedWorkflow\Forms\GridField\GridFieldWorkflowRestrictedEditButton;
@@ -195,7 +197,7 @@ public function getEditForm($id = null, $fields = null)
         }
 
         $this->extend('updateEditFormAfter', $form);
-        
+
         return $form;
     }
 
@@ -260,7 +262,7 @@ public function setFieldFormatting(&$config)
      *
      * @param Member $member
      * @param string $fieldName The name of the gridfield that determines which dataset to return
-     * @return DataList
+     * @return ArrayList<WorkflowInstance>
      * @todo Add the ability to see embargo/expiry dates in report-gridfields at-a-glance if QueuedJobs module installed
      */
     public function userObjects(Member $user, $fieldName)
@@ -293,11 +295,12 @@ public function userObjects(Member $user, $fieldName)
         return $list;
     }
 
-    /*
+    /**
      * Return content-object data depending on which gridfeld is calling for it
      *
      * @param Member $user
      * @param string $fieldName
+     * @return DataList<WorkflowInstance>
      */
     public function getFieldDependentData(Member $user, $fieldName)
     {

src/DataObjects/ImportedWorkflowTemplate.php

@@ -11,6 +11,8 @@
  * @author  [email protected]
  * @license BSD License (http://silverstripe.org/bsd-license/)
  * @package advancedworkflow
+ *
+ * @method WorkflowDefinition Definition()
  */
 class ImportedWorkflowTemplate extends DataObject
 {

src/DataObjects/WorkflowAction.php

@@ -25,6 +25,10 @@
  * @author  [email protected]
  * @license BSD License (http://silverstripe.org/bsd-license/)
  * @package advancedworkflow
+ *
+ * @method WorkflowDefinition WorkflowDef()
+ * @method Member Member()
+ * @method \SilverStripe\ORM\HasManyList<WorkflowTransition> Transitions()
  */
 class WorkflowAction extends DataObject
 {

src/DataObjects/WorkflowActionInstance.php

@@ -22,6 +22,10 @@
  *
  * @license BSD License (http://silverstripe.org/bsd-license/)
  * @package advancedworkflow
+ *
+ * @method WorkFlowInstance Workflow()
+ * @method WorkflowAction Baseaction()
+ * @method Member Member()
  */
 class WorkflowActionInstance extends DataObject
 {
@@ -181,7 +185,7 @@ public function getTitle()
      * If this action returns only one valid transition it will be immediately
      * followed; otherwise the user will decide which transition to follow.
      *
-     * @return ArrayList
+     * @return ArrayList<WorkflowTransition>
      */
     public function getValidTransitions()
     {

src/DataObjects/WorkflowDefinition.php

@@ -49,6 +49,11 @@
  * @author  [email protected]
  * @license BSD License (http://silverstripe.org/bsd-license/)
  * @package advancedworkflow
+ *
+ * @method \SilverStripe\ORM\HasManyList<WorkflowAction> Actions()
+ * @method \SilverStripe\ORM\HasManyList<WorkflowInstance> Instances()
+ * @method \SilverStripe\ORM\ManyManyList<Member> Users()
+ * @method \SilverStripe\ORM\ManyManyList<Group> Groups()
  */
 class WorkflowDefinition extends DataObject
 {

src/DataObjects/WorkflowInstance.php

@@ -41,6 +41,9 @@
  * @method WorkflowDefinition Definition()
  * @method WorkflowActionInstance CurrentAction()
  * @method Member Initiator()
+ * @method \SilverStripe\ORM\HasManyList<WorkflowActionInstance> Actions()
+ * @method \SilverStripe\ORM\ManyManyList<Member> Users()
+ * @method \SilverStripe\ORM\ManyManyList<Group> Groups()
  *
  * @author  [email protected]
  * @license BSD License (http://silverstripe.org/bsd-license/)
@@ -287,7 +290,7 @@ public function Target($getLive = false)
     /**
      * Returns the field differences between the older version and current version of Target
      *
-     * @return ArrayList
+     * @return ArrayList<ArrayData>
      */
     public function getTargetDiff()
     {
@@ -481,7 +484,7 @@ public function performTransition(WorkflowTransition $transition)
      * Returns a list of all Members that are assigned to this instance, either directly or via a group.
      *
      * @todo   This could be made more efficient.
-     * @return ArrayList
+     * @return ArrayList<Member>
      */
     public function getAssignedMembers()
     {

src/DataObjects/WorkflowTransition.php

@@ -27,11 +27,14 @@
  * Therefore, any logic around whether the workflow can proceed should be
  * managed within this method.
  *
- * @method WorkflowAction Action()
- * @method WorkflowAction NextAction()
  * @author  [email protected]
  * @license BSD License (http://silverstripe.org/bsd-license/)
  * @package advancedworkflow
+ *
+ * @method WorkflowAction Action()
+ * @method WorkflowAction NextAction()
+ * @method \SilverStripe\ORM\ManyManyList<Member> Users()
+ * @method \SilverStripe\ORM\ManyManyList<Group> Groups()
  */
 class WorkflowTransition extends DataObject
 {
@@ -251,7 +254,7 @@ public function canDelete($member = null)
     /**
      * Returns a set of all Members that are assigned to this transition, either directly or via a group.
      *
-     * @return ArrayList
+     * @return ArrayList<Member>
      */
     public function getAssignedMembers()
     {

src/Extensions/AdvancedWorkflowExtension.php

@@ -4,6 +4,7 @@
 
 use SilverStripe\Control\Controller;
 use SilverStripe\Control\HTTPRequest;
+use SilverStripe\Control\RequestHandler;
 use SilverStripe\Core\Extension;
 use SilverStripe\Core\Manifest\ModuleLoader;
 use SilverStripe\Forms\Form;
@@ -21,6 +22,8 @@
  * @author  [email protected]
  * @license BSD License (http://silverstripe.org/bsd-license/)
  * @package advancedworkflow
+ *
+ * @extends Extension<RequestHandler>
  */
 class AdvancedWorkflowExtension extends Extension
 {
@@ -62,7 +65,6 @@ public function startworkflow($data, $form, $request)
     public function updateEditForm(Form $form)
     {
         Requirements::javascript('symbiote/silverstripe-advancedworkflow:client/dist/js/advancedworkflow.js');
-        /** @var WorkflowService $service */
         $service = singleton(WorkflowService::class);
         /** @var DataObject|WorkflowApplicable $record */
         $record = $form->getRecord();
@@ -127,7 +129,6 @@ public function updateItemEditForm($form)
      */
     public function updateworkflow($data, Form $form, $request)
     {
-        /** @var WorkflowService $service */
         $service = singleton(WorkflowService::class);
         /** @var DataObject $record */
         $record = $form->getRecord();

src/Extensions/WorkflowApplicable.php

@@ -19,6 +19,7 @@
 use SilverStripe\ORM\CMSPreviewable;
 use SilverStripe\ORM\DataExtension;
 use SilverStripe\ORM\DataList;
+use SilverStripe\ORM\DataObject;
 use SilverStripe\Security\Permission;
 use SilverStripe\Security\Security;
 use Symbiote\AdvancedWorkflow\DataObjects\WorkflowActionInstance;
@@ -35,6 +36,11 @@
  * @author  [email protected]
  * @license BSD License (http://silverstripe.org/bsd-license/)
  * @package advancedworkflow
+ *
+ * @method WorkflowDefinition WorkflowDefinition()
+ * @method \SilverStripe\ORM\ManyManyList<WorkflowDefinition> AdditionalWorkflowDefinitions()
+ *
+ * @extends DataExtension<DataObject&static>
  */
 class WorkflowApplicable extends DataExtension
 {
@@ -353,7 +359,7 @@ public function getWorkflowInstance()
     /**
      * Gets the history of a workflow instance
      *
-     * @return DataList
+     * @return DataList<WorkflowActionInstance>|void
      */
     public function getWorkflowHistory($limit = null)
     {

src/Extensions/WorkflowEmbargoExpiryExtension.php

@@ -35,6 +35,11 @@
  *
  * @author [email protected]
  * @license BSD License http://silverstripe.org/bsd-license/
+ *
+ * @method QueuedJobDescriptor PublishJob()
+ * @method QueuedJobDescriptor UnPublishJob()
+ *
+ * @extends DataExtension<DataObject&static>
  */
 class WorkflowEmbargoExpiryExtension extends DataExtension
 {
@@ -293,7 +298,7 @@ public function onBeforeWrite()
         if (!$this->owner->ID) {
             return;
         }
-        
+
         if ($this->owner->hasMethod('isPublishJobRunning') && $this->owner->isPublishJobRunning()) {
             return;
         }

src/Services/WorkflowService.php

@@ -15,6 +15,7 @@
 use Symbiote\AdvancedWorkflow\Extensions\FileWorkflowApplicable;
 use Symbiote\AdvancedWorkflow\Extensions\WorkflowApplicable;
 use Symbiote\AdvancedWorkflow\DataObjects\WorkflowAction;
+use Symbiote\AdvancedWorkflow\DataObjects\WorkflowActionInstance;
 use Symbiote\AdvancedWorkflow\DataObjects\WorkflowDefinition;
 use Symbiote\AdvancedWorkflow\DataObjects\WorkflowInstance;
 use Symbiote\AdvancedWorkflow\DataObjects\WorkflowTransition;
@@ -199,7 +200,7 @@ public function getWorkflowFor($item, $includeComplete = false)
     /**
      * Get all the workflow action instances for an item
      *
-     * @return DataList|null
+     * @return DataList<WorkflowActionInstance>|void
      */
     public function getWorkflowHistoryFor($item, $limit = null)
     {
@@ -212,7 +213,7 @@ public function getWorkflowHistoryFor($item, $limit = null)
     /**
      * Get all the available workflow definitions
      *
-     * @return DataList
+     * @return DataList<WorkflowDefinition>
      */
     public function getDefinitions()
     {
@@ -297,11 +298,10 @@ public function startWorkflow(DataObject $object, $workflowID = null)
      * Get all the workflows that this user is responsible for
      *
      * @param Member $user The user to get workflows for
-     * @return ArrayList The list of workflow instances this user owns
+     * @return ArrayList<WorkflowInstance> The list of workflow instances this user owns
      */
     public function usersWorkflows(Member $user)
     {
-
         $groupIds = $user->Groups()->column('ID');
 
         $groupInstances = null;
@@ -339,7 +339,7 @@ public function usersWorkflows(Member $user)
      * Get items that the passed-in user has awaiting for them to action
      *
      * @param Member $member
-     * @return DataList
+     * @return DataList<WorkflowInstance>
      */
     public function userPendingItems(Member $user)
     {
@@ -367,7 +367,7 @@ public function userPendingItems(Member $user)
      * Get items that the passed-in user has submitted for workflow review
      *
      * @param Member $member
-     * @return DataList
+     * @return DataList<WorkflowInstance>
      */
     public function userSubmittedItems(Member $user)
     {