symbiote / silverstripe-advancedworkflow

tests/WorkflowEmbargoExpiryTest.php

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 use SilverStripe\ORM\DataObject;
 use SilverStripe\ORM\FieldType\DBDatetime;
 use SilverStripe\Translatable\Model\Translatable;
-use SilverStripe\Security\Member;
 use SilverStripe\Subsites\Extensions\SiteTreeSubsites;
 use SilverStripe\Versioned\Versioned;
 use Symbiote\AdvancedWorkflow\DataObjects\WorkflowAction;

src/Actions/PublishItemWorkflowAction.php

Unused Use Statements -2 removed lines

@@ -3,8 +3,6 @@
 namespace Symbiote\AdvancedWorkflow\Actions;
 
 use SilverStripe\Forms\CheckboxField;
-use SilverStripe\Forms\FieldGroup;
-use SilverStripe\Forms\LabelField;
 use SilverStripe\Forms\NumericField;
 use SilverStripe\ORM\DataObject;
 use Symbiote\AdvancedWorkflow\DataObjects\WorkflowAction;

src/Controllers/FrontEndWorkflowController.php

Doc Comments +2 added lines, -2 removed lines

@@ -94,7 +94,7 @@ 
      * - required fields from the Context Object
      * - Actions from the connected WorkflowTransitions
      *
-     * @return Form
+     * @return null|FrontendWorkflowForm
      */
     public function Form()
     {
@@ -150,7 +150,7 @@ 
      *
      * @param array $data
      * @param Form $form
-     * @param SS_HTTPRequest $request
+     * @param HTTPRequest $request
      * @throws Exception
      */
     public function doFrontEndAction(array $data, Form $form, HTTPRequest $request)

src/DataObjects/WorkflowAction.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use SilverStripe\Forms\CheckboxField;
 use SilverStripe\Forms\DropdownField;
 use SilverStripe\Forms\FieldList;
-use SilverStripe\Forms\ReadonlyField;
 use SilverStripe\Forms\RequiredFields;
 use SilverStripe\Forms\TabSet;
 use SilverStripe\Forms\TextField;

Doc Comments +6 added lines, -6 removed lines

@@ -75,7 +75,7 @@ 
      * Admin level users can always edit.
      *
      * @param  DataObject $target
-     * @return bool
+     * @return boolean|null
      */
     public function canEditTarget(DataObject $target)
     {
@@ -91,7 +91,7 @@ 
      * Does this action restrict viewing of the document?
      *
      * @param  DataObject $target
-     * @return bool
+     * @return boolean|null
      */
     public function canViewTarget(DataObject $target)
     {
@@ -102,7 +102,7 @@ 
      * Does this action restrict the publishing of a document?
      *
      * @param  DataObject $target
-     * @return bool
+     * @return boolean|null
      */
     public function canPublishTarget(DataObject $target)
     {
@@ -114,7 +114,7 @@ 
      *
      * @param Member $member
      * @param array $context
-     * @return bool
+     * @return boolean|string
      */
     public function canCreate($member = null, $context = array())
     {
@@ -123,7 +123,7 @@ 
 
     /**
      * @param  Member $member
-     * @return bool
+     * @return boolean|string
      */
     public function canEdit($member = null)
     {
@@ -132,7 +132,7 @@ 
 
     /**
      * @param  Member $member
-     * @return bool
+     * @return boolean|string
      */
     public function canDelete($member = null)
     {

src/DataObjects/WorkflowActionInstance.php

Doc Comments +7 added lines

@@ -57,6 +57,7 @@ 
 
     /**
      * Gets fields for when this is part of an active workflow
+     * @param \SilverStripe\Forms\FieldList $fields
      */
     public function updateWorkflowFields($fields)
     {
@@ -74,6 +75,9 @@ 
         }
     }
 
+    /**
+     * @param \SilverStripe\Forms\FieldList $fields
+     */
     public function updateFrontendWorkflowFields($fields)
     {
         if ($this->BaseAction()->AllowCommenting) {
@@ -108,6 +112,9 @@ 
         return $obj;
     }
 
+    /**
+     * @param \SilverStripe\Forms\FieldList $actions
+     */
     public function updateFrontEndWorkflowActions($actions)
     {
         $ba = $this->BaseAction();

src/DataObjects/WorkflowDefinition.php

Unused Use Statements -2 removed lines

@@ -5,7 +5,6 @@ 
 use SilverStripe\Control\Controller;
 use SilverStripe\Forms\CheckboxSetField;
 use SilverStripe\Forms\DropdownField;
-use SilverStripe\Forms\FieldGroup;
 use SilverStripe\Forms\FieldList;
 use SilverStripe\Forms\FormAction;
 use SilverStripe\Forms\GridField\GridFieldConfig_RecordEditor;
@@ -16,7 +15,6 @@ 
 use SilverStripe\Forms\GridField\GridFieldViewButton;
 use SilverStripe\Forms\GridField\GridFieldDetailForm;
 use SilverStripe\Forms\GridField\GridFieldConfig_Base;
-use SilverStripe\Forms\LabelField;
 use SilverStripe\Forms\LiteralField;
 use SilverStripe\Forms\NumericField;
 use SilverStripe\Forms\ReadonlyField;

Doc Comments +6 added lines, -6 removed lines

@@ -479,7 +479,7 @@ 
      *
      * @param $member
      * @param $target
-     * @return Boolean
+     * @return boolean|string
      */
     public function canWorkflowPublish($member, $target)
     {
@@ -494,7 +494,7 @@ 
      *
      * @param Member $member
      * @param array $context
-     * @return bool
+     * @return boolean|string
      */
     public function canCreate($member = null, $context = [])
     {
@@ -510,7 +510,7 @@ 
     /**
      *
      * @param Member $member
-     * @return boolean
+     * @return boolean|null
      */
     public function canView($member = null)
     {
@@ -520,7 +520,7 @@ 
     /**
      *
      * @param Member $member
-     * @return boolean
+     * @return boolean|string
      */
     public function canEdit($member = null)
     {
@@ -530,7 +530,7 @@ 
     /**
      *
      * @param Member $member
-     * @return boolean
+     * @return boolean|string
      * @see {@link $this->onBeforeDelete()}
      */
     public function canDelete($member = null)
@@ -558,7 +558,7 @@ 
      * Checks whether the passed user is able to view this ModelAdmin
      *
      * @param Member $member
-     * @return bool
+     * @return boolean|null
      */
     protected function userHasAccess($member)
     {

src/DataObjects/WorkflowInstance.php

Doc Comments +5 added lines, -5 removed lines

@@ -603,7 +603,7 @@ 
     /**
      * Does this action restrict the publishing of a document?
      *
-     * @return boolean
+     * @return boolean|null
      */
     public function canPublishTarget()
     {
@@ -616,7 +616,7 @@ 
      * Get the current set of transitions that are valid for the current workflow state,
      * and are available to the current user.
      *
-     * @return array
+     * @return ArrayList
      */
     public function validTransitions()
     {
@@ -750,9 +750,9 @@ 
      * to view WorkflowInstances within GridFields
      * @see {@link $this->userHasAccess()}
      *
-     * @param number $recordID
+     * @param integer $recordID
      * @param number $userID
-     * @param number $wasPublished
+     * @param integer $wasPublished
      * @return boolean
      */
     public function getVersionedConnection($recordID, $userID, $wasPublished = 0)
@@ -789,7 +789,7 @@ 
     /**
      * Tells us if $member has had permissions over some part of the current WorkflowInstance.
      *
-     * @param $member
+     * @param Member $member
      * @return WorkflowAction|boolean
      */
     public function getMostRecentActionForUser($member = null)

src/DataObjects/WorkflowTransition.php

Doc Comments +3 added lines, -3 removed lines

@@ -217,7 +217,7 @@ 
      *
      * @param  Member $member
      * @param array $context
-     * @return bool
+     * @return boolean|string
      */
     public function canCreate($member = null, $context = array())
     {
@@ -226,7 +226,7 @@ 
 
     /**
      * @param  Member $member
-     * @return bool
+     * @return boolean|string
      */
     public function canEdit($member = null)
     {
@@ -235,7 +235,7 @@ 
 
     /**
      * @param  Member $member
-     * @return bool
+     * @return boolean|string
      */
     public function canDelete($member = null)
     {

src/Extensions/WorkflowEmbargoExpiryExtension.php

Doc Comments +1 added lines, -1 removed lines

@@ -535,7 +535,7 @@
      * disabled.
      *
      * @param Member $member
-     * @return bool|null
+     * @return false|null
      */
     public function canEdit($member)
     {

Unused Use Statements -4 removed lines

@@ -4,16 +4,12 @@
 
 use SilverStripe\Control\Controller;
 use SilverStripe\Core\Injector\Injector;
-use SilverStripe\Core\Manifest\ModuleLoader;
 use SilverStripe\Forms\FieldList;
 use SilverStripe\Forms\DatetimeField;
 use SilverStripe\Forms\HeaderField;
 use SilverStripe\Forms\LiteralField;
-use SilverStripe\ORM\ArrayList;
 use SilverStripe\ORM\DataExtension;
-use SilverStripe\ORM\DataQuery;
 use SilverStripe\ORM\FieldType\DBDatetime;
-use SilverStripe\ORM\Queries\SQLSelect;
 use SilverStripe\Security\Member;
 use SilverStripe\Security\Permission;
 use SilverStripe\View\Requirements;