sminnee / silverstripe-framework

src/Dev/Backtrace.php

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php
 
 use SilverStripe\Core\Convert;
-use SilverStripe\Core\Config\Config;
 use SilverStripe\Dev\SapphireTest;
 use SilverStripe\View\Parsers\URLSegmentFilter;
 

Doc Comments +1 added lines, -1 removed lines

@@ -132,7 +132,7 @@
      * @param mixed $returnVal
      * @param bool $ignoreAjax
      * @param array $ignoredFunctions
-     * @return mixed
+     * @return string|null
      */
     public static function backtrace($returnVal = false, $ignoreAjax = false, $ignoredFunctions = null)
     {

src/Dev/CSVParser.php

Unused Use Statements -1 removed lines

@@ -5,7 +5,6 @@
 use SilverStripe\Filesystem\ImageManipulation;
 use SilverStripe\Filesystem\Storage\AssetContainer;
 use SilverStripe\Filesystem\Storage\AssetStore;
-
 use Injector;
 use AssetField;
 use File;

src/Forms/ConfirmedPasswordField.php

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use SilverStripe\ORM\SS_List;
 use SilverStripe\ORM\ArrayList;
 use SilverStripe\ORM\DataObjectInterface;
-use SilverStripe\View\Requirements;
 use InvalidArgumentException;
 
 /**

src/Forms/DatetimeField.php

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use SilverStripe\ORM\SS_List;
 use SilverStripe\ORM\ArrayList;
 use SilverStripe\ORM\DataObjectInterface;
-use SilverStripe\View\Requirements;
 use InvalidArgumentException;
 
 /**

Doc Comments +20 added lines, -4 removed lines

@@ -70,6 +70,10 @@ 
      */
     protected $config;
 
+    /**
+     * @param string $name
+     * @param string $title
+     */
     public function __construct($name, $title = null, $value = "")
     {
         $this->config = $this->config()->default_config;
@@ -81,6 +85,9 @@ 
         parent::__construct($name, $title, $value);
     }
 
+    /**
+     * @param Form $form
+     */
     public function setForm($form)
     {
         parent::setForm($form);
@@ -92,6 +99,9 @@ 
         return $this;
     }
 
+    /**
+     * @param string $name
+     */
     public function setName($name)
     {
         parent::setName($name);
@@ -105,7 +115,7 @@ 
 
     /**
      * @param array $properties
-     * @return string
+     * @return \SilverStripe\ORM\FieldType\DBHTMLText
      */
     public function FieldHolder($properties = array())
     {
@@ -120,7 +130,7 @@ 
 
     /**
      * @param array $properties
-     * @return string
+     * @return \SilverStripe\ORM\FieldType\DBHTMLText
      */
     public function Field($properties = array())
     {
@@ -254,6 +264,9 @@ 
         return $this;
     }
 
+    /**
+     * @param boolean $bool
+     */
     public function setReadonly($bool)
     {
         parent::setReadonly($bool);
@@ -354,7 +367,7 @@ 
      * to set field-specific config options.
      *
      * @param string $name
-     * @param mixed $val
+     * @param string $val
      * @return $this
      */
     public function setConfig($name, $val)
@@ -374,7 +387,7 @@ 
      * to get field-specific config options.
      *
      * @param String $name Optional, returns the whole configuration array if empty
-     * @return mixed
+     * @return string
      */
     public function getConfig($name = null)
     {
@@ -385,6 +398,9 @@ 
         }
     }
 
+    /**
+     * @param RequiredFields $validator
+     */
     public function validate($validator)
     {
         $dateValid = $this->dateField->validate($validator);

src/Forms/GridField/GridField.php

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use SilverStripe\ORM\SS_List;
 use SilverStripe\ORM\ArrayList;
 use SilverStripe\ORM\DataObjectInterface;
-use SilverStripe\View\Requirements;
 use InvalidArgumentException;
 
 /**

Doc Comments +1 added lines, -1 removed lines

@@ -228,7 +228,7 @@
      *
      * @todo refactor this into GridFieldComponent
      *
-     * @param mixed $value
+     * @param string $value
      * @param string|array $castingDefinition
      *
      * @return mixed

src/Forms/HTMLEditor/HTMLEditorSanitiser.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use DOMAttr;
 use DOMElement;
-use DOMNode;
 use SilverStripe\Core\Injector\Injectable;
 use SilverStripe\View\Parsers\HTMLValue;
 use stdClass;

Doc Comments +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
     /**
      * Given a TinyMCE pattern (close to unix glob style), create a regex that does the match
      *
-     * @param $str - The TinyMCE pattern
+     * @param string|null $str - The TinyMCE pattern
      * @return string - The equivalent regex
      */
     protected function patternToRegex($str)

src/Forms/InlineFormAction.php

Unused Use Statements -2 removed lines

@@ -2,8 +2,6 @@
 
 namespace SilverStripe\Forms;
 
-use SilverStripe\View\Requirements;
-
 /**
  * Render a button that will submit the form its contained in through ajax.
  *

src/Forms/MemberDatetimeOptionsetField.php

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use SilverStripe\ORM\SS_List;
 use SilverStripe\ORM\ArrayList;
 use SilverStripe\ORM\DataObjectInterface;
-use SilverStripe\View\Requirements;
 use InvalidArgumentException;
 
 /**

src/Forms/TabSet.php

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use SilverStripe\ORM\SS_List;
 use SilverStripe\ORM\ArrayList;
 use SilverStripe\ORM\DataObjectInterface;
-use SilverStripe\View\Requirements;
 use InvalidArgumentException;
 
 /**