silverstripe / silverstripe-framework

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

src/ORM/DataExtension.php

Doc Comments +6 added lines

@@ -88,10 +88,16 @@
     {
     }
 
+    /**
+     * @param \SilverStripe\Security\Member|null $member
+     */
     public function canEdit($member)
     {
     }
 
+    /**
+     * @param \SilverStripe\Security\Member|null $member
+     */
     public function canDelete($member)
     {
     }

src/ORM/DataList.php

Doc Comments +10 added lines, -10 removed lines

@@ -179,7 +179,7 @@ 
      * Returns a new DataList instance with the specified query parameter assigned
      *
      * @param string|array $keyOrArray Either the single key to set, or an array of key value pairs to set
-     * @param mixed $val If $keyOrArray is not an array, this is the value to set
+     * @param string $val If $keyOrArray is not an array, this is the value to set
      * @return static
      */
     public function setDataQueryParam($keyOrArray, $val = null)
@@ -554,7 +554,7 @@ 
      * Check if the given field specification could be interpreted as an unquoted relation name
      *
      * @param string $field
-     * @return bool
+     * @return integer
      */
     protected function isValidRelationName($field)
     {
@@ -667,7 +667,7 @@ 
      * @param int $order A numerical index to control the order that joins are added to the query; lower order values
      * will cause the query to appear first. The default is 20, and joins created automatically by the
      * ORM have a value of 10.
-     * @param array $parameters Any additional parameters if the join is a parameterised subquery
+     * @param string[] $parameters Any additional parameters if the join is a parameterised subquery
      * @return static
      */
     public function innerJoin($table, $onClause, $alias = null, $order = 20, $parameters = array())
@@ -686,7 +686,7 @@ 
      * @param int $order A numerical index to control the order that joins are added to the query; lower order values
      * will cause the query to appear first. The default is 20, and joins created automatically by the
      * ORM have a value of 10.
-     * @param array $parameters Any additional parameters if the join is a parameterised subquery
+     * @param string[] $parameters Any additional parameters if the join is a parameterised subquery
      * @return static
      */
     public function leftJoin($table, $onClause, $alias = null, $order = 20, $parameters = array())
@@ -834,7 +834,7 @@ 
      * Return the maximum value of the given field in this DataList
      *
      * @param string $fieldName
-     * @return mixed
+     * @return string
      */
     public function max($fieldName)
     {
@@ -845,7 +845,7 @@ 
      * Return the minimum value of the given field in this DataList
      *
      * @param string $fieldName
-     * @return mixed
+     * @return string
      */
     public function min($fieldName)
     {
@@ -856,7 +856,7 @@ 
      * Return the average value of the given field in this DataList
      *
      * @param string $fieldName
-     * @return mixed
+     * @return string
      */
     public function avg($fieldName)
     {
@@ -867,7 +867,7 @@ 
      * Return the sum of the values of the given field in this DataList
      *
      * @param string $fieldName
-     * @return mixed
+     * @return string
      */
     public function sum($fieldName)
     {
@@ -926,7 +926,7 @@ 
     /**
      * Restrict the columns to fetch into this DataList
      *
-     * @param array $queriedColumns
+     * @param string[] $queriedColumns
      * @return static
      */
     public function setQueriedColumns($queriedColumns)
@@ -1172,7 +1172,7 @@ 
     /**
      * Returns item stored in list with index $key
      *
-     * @param mixed $key
+     * @param integer $key
      * @return DataObject
      */
     public function offsetGet($key)