silverstripe / silverstripe-framework

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/DataQuery.php

Doc Comments +2 added lines, -2 removed lines

@@ -110,7 +110,7 @@ 
     /**
      * Remove a filter from the query
      *
-     * @param string|array $fieldExpression The predicate of the condition to remove
+     * @param string|null $fieldExpression The predicate of the condition to remove
      * (ignoring parameters). The expression will be considered a match if it's
      * contained within any other predicate.
      * @return DataQuery Self reference
@@ -191,7 +191,7 @@ 
     /**
      * Ensure that the query is ready to execute.
      *
-     * @param array|null $queriedColumns Any columns to filter the query by
+     * @param string[] $queriedColumns Any columns to filter the query by
      * @return SQLSelect The finalised sql query
      */
     public function getFinalisedQuery($queriedColumns = null)

src/ORM/FieldType/DBClassName.php

Doc Comments +3 added lines

@@ -171,6 +171,9 @@
         return $classNames;
     }
 
+    /**
+     * @param \SilverStripe\ORM\Tests\DBClassNameTest\ObjectSubClass $record
+     */
     public function setValue($value, $record = null, $markChanged = true)
     {
         parent::setValue($value, $record, $markChanged);

src/ORM/FieldType/DBComposite.php

Doc Comments +3 added lines

@@ -220,6 +220,9 @@
         return null;
     }
 
+    /**
+     * @param string $field
+     */
     public function hasField($field)
     {
         $fields = $this->compositeDatabaseFields();

src/ORM/FieldType/DBCurrency.php

Doc Comments +6 added lines

@@ -24,6 +24,9 @@ 
      */
     private static $currency_symbol = '$';
 
+    /**
+     * @param string $name
+     */
     public function __construct($name = null, $wholeSize = 9, $decimalSize = 2, $defaultValue = 0)
     {
         parent::__construct($name, $wholeSize, $decimalSize, $defaultValue);
@@ -56,6 +59,9 @@ 
         }
     }
 
+    /**
+     * @param string $value
+     */
     public function setValue($value, $record = null, $markChanged = true)
     {
         $matches = null;

src/ORM/FieldType/DBDecimal.php

Doc Comments +4 added lines, -4 removed lines

@@ -19,7 +19,7 @@ 
      * @param string $name
      * @param int $wholeSize
      * @param int $decimalSize
-     * @param float|int $defaultValue
+     * @param integer $defaultValue
      */
     public function __construct($name = null, $wholeSize = 9, $decimalSize = 2, $defaultValue = 0)
     {
@@ -33,7 +33,7 @@ 
     }
 
     /**
-     * @return float
+     * @return string
      */
     public function Nice()
     {
@@ -41,7 +41,7 @@ 
     }
 
     /**
-     * @return int
+     * @return double
      */
     public function Int()
     {
@@ -88,7 +88,7 @@ 
     }
 
     /**
-     * @return float
+     * @return integer
      */
     public function nullValue()
     {