silverstripe / silverstripe-framework

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()
     {

src/i18n/i18n.php

Doc Comments +3 added lines, -1 removed lines

@@ -2171,6 +2171,7 @@ 
 
     /**
      * @param String
+     * @param string $name
      * @return Zend_Translate
      */
     public static function get_translator($name)
@@ -2212,6 +2213,7 @@ 
 
     /**
      * @param String
+     * @param string $name
      */
     public static function unregister_translator($name)
     {
@@ -2323,7 +2325,7 @@ 
      *
      * @see get_locale_name()
      *
-     * @param mixed $code Language code
+     * @param string $code Language code
      * @param boolean $native If true, the native name will be returned
      * @return string Name of the language
      */

src/i18n/i18nTextCollector_Parser.php

Doc Comments +1 added lines

@@ -52,6 +52,7 @@
 
     /**
      * Parses a template and returns any translatable entities
+     * @param string $template
      */
     public static function GetTranslatables($template)
     {

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.

src/ORM/Connect/MySQLSchemaManager.php

Doc Comments +3 added lines

@@ -129,6 +129,9 @@
         $this->query("ALTER TABLE \"$tableName\" $alterations");
     }
 
+    /**
+     * @param string $tableName
+     */
     public function isView($tableName)
     {
         $info = $this->query("SHOW /*!50002 FULL*/ TABLES LIKE '$tableName'")->record();

src/ORM/DatabaseAdmin.php

Doc Comments +2 added lines, -2 removed lines

@@ -143,7 +143,7 @@ 
     /**
      * Gets the url to return to after build
      *
-     * @return string|null
+     * @return string
      */
     protected function getReturnURL()
     {
@@ -175,7 +175,7 @@ 
     /**
      * Returns the timestamp of the time that the database was last built
      *
-     * @return string Returns the timestamp of the time that the database was
+     * @return integer|null Returns the timestamp of the time that the database was
      *                last built
      */
     public static function lastBuilt()