sminnee / silverstripe-framework

src/Forms/PrintableTransformation_TabSet.php

Doc Comments +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
 class PrintableTransformation_TabSet extends TabSet
 {
     /**
-     * @param array $tabs
+     * @param FieldList $tabs
      */
     public function __construct($tabs)
     {

src/Forms/TextField.php

Doc Comments +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
     }
 
     /**
-     * @return string
+     * @return \SilverStripe\ORM\FieldType\DBHTMLText
      */
     public function InternallyLabelledField()
     {

src/Forms/TimeField.php

Doc Comments +8 added lines, -1 removed lines

@@ -57,6 +57,10 @@ 
 
     protected $schemaDataType = FormField::SCHEMA_DATA_TYPE_TIME;
 
+    /**
+     * @param string $name
+     * @param string $title
+     */
     public function __construct($name, $title = null, $value = "")
     {
         if (!$this->locale) {
@@ -231,7 +235,7 @@ 
 
     /**
      * @param String $name Optional, returns the whole configuration array if empty
-     * @return mixed|array
+     * @return string
      */
     public function getConfig($name = null)
     {
@@ -250,6 +254,9 @@ 
         return $this->castedCopy('SilverStripe\\Forms\\TimeField_Readonly');
     }
 
+    /**
+     * @param string $class
+     */
     public function castedCopy($class)
     {
         $copy = parent::castedCopy($class);

src/Forms/TreeDropdownField.php

Doc Comments +1 added lines, -1 removed lines

@@ -238,7 +238,7 @@
 
     /**
      * @param array $properties
-     * @return string
+     * @return \SilverStripe\ORM\FieldType\DBHTMLText
      */
     public function Field($properties = array())
     {

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use SilverStripe\Core\Config\Config;
 use SilverStripe\ORM\DataObject;
 use SilverStripe\ORM\Hierarchy\Hierarchy;
-use SilverStripe\View\Requirements;
 use SilverStripe\View\ViewableData;
 use Exception;
 use InvalidArgumentException;

src/Forms/UploadField.php

Doc Comments +4 added lines, -4 removed lines

@@ -304,7 +304,7 @@ 
     /**
      * Determine if the target folder for new uploads in is visible the field UI.
      *
-     * @return boolean
+     * @return boolean|string
      */
     public function canPreviewFolder()
     {
@@ -581,7 +581,7 @@ 
      * Customises a file with additional details suitable for rendering in the
      * UploadField.ss template
      *
-     * @param ViewableData|AssetContainer $file
+     * @param AssetContainer $file
      * @return ViewableData_Customised
      */
     protected function customiseFile(AssetContainer $file)
@@ -690,7 +690,7 @@ 
     /**
      * Determine if the user has permission to upload.
      *
-     * @return boolean
+     * @return boolean|string
      */
     public function canUpload()
     {
@@ -718,7 +718,7 @@ 
      * Determine if the user has permission to attach existing files
      * By default returns true if the user has the CMS_ACCESS_AssetAdmin permission
      *
-     * @return boolean
+     * @return boolean|string
      */
     public function canAttachExisting()
     {

Unused Use Statements -2 removed lines

@@ -13,13 +13,11 @@
 use SilverStripe\ORM\SS_List;
 use SilverStripe\ORM\DataObject;
 use SilverStripe\ORM\ArrayList;
-use SilverStripe\ORM\ValidationException;
 use SilverStripe\Security\Permission;
 use SilverStripe\View\ArrayData;
 use SilverStripe\View\ViewableData;
 use SilverStripe\View\ViewableData_Customised;
 use InvalidArgumentException;
-use Exception;
 
 /**
  * Field for uploading single or multiple files of all types, including images.

src/ORM/ArrayLib.php

Doc Comments +1 added lines, -1 removed lines

@@ -178,7 +178,7 @@
      *
      * Actually only one array argument is supported.
      *
-     * @param $f callback to apply
+     * @param string $f callback to apply
      * @param $array array
      * @return array
      */

src/ORM/ArrayList.php

Doc Comments +1 added lines, -1 removed lines

@@ -745,7 +745,7 @@
     /**
      * Returns item stored in list with index $key
      *
-     * @param mixed $offset
+     * @param integer $offset
      * @return DataObject
      */
     public function offsetGet($offset)

src/ORM/Connect/Database.php

Doc Comments +1 added lines, -1 removed lines

@@ -768,7 +768,7 @@
      * @param boolean $create Flag indicating whether the database should be created
      * if it doesn't exist. If $create is false and the database doesn't exist
      * then an error will be raised
-     * @param int|boolean $errorLevel The level of error reporting to enable for the query, or false if no error
+     * @param boolean $errorLevel The level of error reporting to enable for the query, or false if no error
      * should be raised
      * @return boolean Flag indicating success
      */

src/ORM/Connect/DBSchemaManager.php

Doc Comments +3 added lines, -3 removed lines

@@ -331,7 +331,7 @@ 
      *   - array('fields' => array('A','B','C'), 'type' => 'index/unique/fulltext'): This gives you full
      *     control over the index.
      * @param boolean $hasAutoIncPK A flag indicating that the primary key on this table is an autoincrement type
-     * @param array $options Create table options (ENGINE, etc.)
+     * @param string|null $options Create table options (ENGINE, etc.)
      * @param array|bool $extensions List of extensions
      */
     public function requireTable(
@@ -529,7 +529,7 @@ 
     /**
      * Given an index spec determines the index type
      *
-     * @param array|string $spec
+     * @param string $spec
      * @return string
      */
     protected function determineIndexType($spec)
@@ -630,7 +630,7 @@ 
      *
      * @param string $table The table name.
      * @param string $field The field name.
-     * @param array|string $spec The field specification. If passed in array syntax, the specific database
+     * @param string $spec The field specification. If passed in array syntax, the specific database
      *  driver takes care of the ALTER TABLE syntax. If passed as a string, its assumed to
      *  be prepared as a direct SQL framgment ready for insertion into ALTER TABLE. In this case you'll
      *  need to take care of database abstraction in your DBField subclass.