silverstripe / silverstripe-asset-admin

code/Forms/UploadField.php

Doc Comments +1 added lines, -1 removed lines

@@ -137,7 +137,7 @@
     /**
      * Set upload type to multi / single
      *
-     * @param $multi
+     * @param boolean $multi
      * @return $this
      */
     public function setIsMultiUpload($multi)

code/Controller/AssetAdmin.php

Doc Comments +1 added lines, -1 removed lines

@@ -902,7 +902,7 @@
      *
      * @param array $data
      * @param Form $form
-     * @return DBHTMLText|HTTPResponse
+     * @return null|HTTPResponse
      */
     public function addtocampaign($data, $form)
     {

code/GraphQL/FolderTypeCreator.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use GraphQL\Type\Definition\ResolveInfo;
 use GraphQL\Type\Definition\Type;
 use SilverStripe\AssetAdmin\Controller\AssetAdminFile;
-use SilverStripe\AssetAdmin\Controller\AssetAdminFolder;
 use SilverStripe\Assets\File;
 use SilverStripe\Assets\Folder;
 use SilverStripe\ORM\Filterable;

code/Controller/AssetAdminFile.php

Doc Comments +1 added lines, -1 removed lines

@@ -208,7 +208,7 @@
      *
      * @param int $parentID
      * @param int $maxDepth Hard limit of max depth
-     * @return array List of parent IDs, including $parentID
+     * @return integer[] List of parent IDs, including $parentID
      */
     public static function nestedFolderIDs($parentID, $maxDepth = 5)
     {

code/GraphQL/ReadFileQueryCreator.php

Doc Comments +3 added lines

@@ -51,6 +51,9 @@
             ->setConnectionResolver(array($this, 'resolveConnection'));
     }
 
+    /**
+     * @param \GraphQL\Type\Definition\ResolveInfo $info
+     */
     public function resolveConnection($object, array $args, $context, $info)
     {
         $filter = (!empty($args['filter'])) ? $args['filter'] : [];

code/Forms/RemoteFileFormFactory.php

Doc Comments +8 added lines

@@ -66,6 +66,10 @@ 
         return ['type'];
     }
     
+    /**
+     * @param Controller $controller
+     * @param string $name
+     */
     protected function getFormFields($controller, $name, $context)
     {
         $fields = [];
@@ -130,6 +134,10 @@ 
         return FieldList::create($fields);
     }
     
+    /**
+     * @param Controller $controller
+     * @param string $name
+     */
     protected function getFormActions($controller, $name, $context)
     {
         $actions = [];

Unused Use Statements -3 removed lines

@@ -14,13 +14,10 @@
 use SilverStripe\Forms\Form;
 use SilverStripe\Forms\FormAction;
 use SilverStripe\Forms\FormFactory;
-use SilverStripe\Forms\HeaderField;
 use SilverStripe\Forms\HiddenField;
 use SilverStripe\Forms\HTMLEditor\HTMLEditorField_Embed;
-use SilverStripe\Forms\LabelField;
 use SilverStripe\Forms\LiteralField;
 use SilverStripe\Forms\OptionsetField;
-use SilverStripe\Forms\ReadonlyField;
 use SilverStripe\Forms\RequiredFields;
 use SilverStripe\Forms\TextField;
 

code/Forms/AssetFormFactory.php

Doc Comments +6 added lines, -3 removed lines

@@ -75,7 +75,7 @@ 
      * Get the validator for the form to be built
      *
      * @param $controller
-     * @param $name
+     * @param string $name
      * @param $context
      * @return RequiredFields
      */
@@ -139,6 +139,9 @@ 
         return null;
     }
 
+    /**
+     * @param string $name
+     */
     protected function getFormActions(Controller $controller, $name, $context = [])
     {
         $record = isset($context['Record']) ? $context['Record'] : null;
@@ -213,8 +216,8 @@ 
     /**
      * Get actions that go into the Popover menu
      *
-     * @param $record
-     * @return array
+     * @param File $record
+     * @return string|null
      */
     protected function getPopoverActions($record)
     {