silverstripe / silverstripe-framework

src/ORM/Queries/SQLConditionalExpression.php

Doc Comments -4 removed lines

@@ -398,7 +398,6 @@ 
      * @see SQLConditionalExpression::addWhere() for syntax examples
      *
      * @param mixed $where Predicate(s) to set, as escaped SQL statements or paramaterised queries
-     * @param mixed $where,... Unlimited additional predicates
      * @return $this Self reference
      */
     public function setWhere($where)
@@ -485,7 +484,6 @@ 
      * </code>
      *
      * @param mixed $where Predicate(s) to set, as escaped SQL statements or paramaterised queries
-     * @param mixed $where,... Unlimited additional predicates
      * @return $this Self reference
      */
     public function addWhere($where)
@@ -502,7 +500,6 @@ 
      * @see SQLConditionalExpression::addWhere()
      *
      * @param mixed $filters Predicate(s) to set, as escaped SQL statements or paramaterised queries
-     * @param mixed $filters,... Unlimited additional predicates
      * @return $this Self reference
      */
     public function setWhereAny($filters)
@@ -517,7 +514,6 @@ 
      * @see SQLConditionalExpression::addWhere()
      *
      * @param mixed $filters Predicate(s) to set, as escaped SQL statements or paramaterised queries
-     * @param mixed $filters,... Unlimited additional predicates
      * @return $this Self reference
      */
     public function addWhereAny($filters)

src/ORM/Queries/SQLSelect.php

Doc Comments +2 added lines, -4 removed lines

@@ -66,7 +66,7 @@ 
     /**
      * Construct a new SQLSelect.
      *
-     * @param array|string $select An array of SELECT fields.
+     * @param string $select An array of SELECT fields.
      * @param array|string $from An array of FROM clauses. The first one should be just the table name.
      * Each should be ANSI quoted.
      * @param array $where An array of WHERE clauses.
@@ -378,7 +378,7 @@ 
      *
      * @param string $value
      * @param string $defaultDirection
-     * @return array A two element array: array($column, $direction)
+     * @return string[] A two element array: array($column, $direction)
      */
     private function getDirectionFromString($value, $defaultDirection = null)
     {
@@ -480,7 +480,6 @@ 
      * @see SQLSelect::addWhere() for syntax examples
      *
      * @param mixed $having Predicate(s) to set, as escaped SQL statements or paramaterised queries
-     * @param mixed $having,... Unlimited additional predicates
      * @return $this Self reference
      */
     public function setHaving($having)
@@ -496,7 +495,6 @@ 
      * @see SQLSelect::addWhere() for syntax examples
      *
      * @param mixed $having Predicate(s) to set, as escaped SQL statements or paramaterised queries
-     * @param mixed $having,... Unlimited additional predicates
      * @return $this Self reference
      */
     public function addHaving($having)

src/ORM/RelationList.php

Doc Comments +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
     /**
      * Any number of foreign keys to apply to this list
      *
-     * @return string|array|null
+     * @return string|null
      */
     public function getForeignID()
     {

src/ORM/UnsavedRelationList.php

Doc Comments +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
     /**
      * Create a new UnsavedRelationList
      *
-     * @param array $baseClass
+     * @param string $baseClass
      * @param string $relationName
      * @param string $dataClass The DataObject class used in the relation
      */

src/ORM/Versioning/ChangeSetItem.php

Doc Comments +2 added lines, -2 removed lines

@@ -158,7 +158,7 @@ 
      * Find version of this object in the given stage
      *
      * @param string $stage
-     * @return DataObject|Versioned
+     * @return DataObject|null
      * @throws UnexpectedDataException
      */
     protected function getObjectInStage($stage)
@@ -172,7 +172,7 @@ 
 
     /**
      * Find latest version of this object
-     * @return DataObject|Versioned
+     * @return DataObject|null
      * @throws UnexpectedDataException
      */
     protected function getObjectLatestVersion()

src/ORM/Versioning/Versioned.php

Doc Comments +6 added lines, -6 removed lines

@@ -1086,7 +1086,7 @@ 
     /**
      * Find objects in the given relationships, merging them into the given list
      *
-     * @param array $source Config property to extract relationships from
+     * @param string $source Config property to extract relationships from
      * @param bool $recursive True if recursive
      * @param ArrayList $list Optional list to add items to
      * @return ArrayList The list
@@ -1198,7 +1198,7 @@ 
     /**
      * Check if the current user can delete this record from live
      *
-     * @param null $member
+     * @param Member|null $member
      * @return mixed
      */
     public function canUnpublish($member = null)
@@ -1312,7 +1312,7 @@ 
      * Extend permissions to include additional security for objects that are not published to live.
      *
      * @param Member $member
-     * @return bool|null
+     * @return false|null
      */
     public function canView($member = null)
     {
@@ -1390,7 +1390,7 @@ 
      *
      * @param string $stage
      * @param Member $member
-     * @return bool
+     * @return boolean|string
      */
     public function canViewStage($stage = 'Live', $member = null)
     {
@@ -1459,7 +1459,7 @@ 
     /**
      * Determines if the current draft version is the same as live or rather, that there are no outstanding draft changes
      *
-     * @return bool
+     * @return string
      */
     public function latestPublished()
     {
@@ -2554,7 +2554,7 @@ 
     /**
      * Returns an array of possible stages.
      *
-     * @return array
+     * @return string[]
      */
     public function getVersionedStages()
     {

src/Security/Authenticator.php

Doc Comments +1 added lines, -1 removed lines

@@ -137,7 +137,7 @@
      * Remove a previously registered authenticator
      *
      * @param string $authenticator Name of the authenticator class to register
-     * @return bool Returns TRUE on success, FALSE otherwise.
+     * @return boolean|null Returns TRUE on success, FALSE otherwise.
      */
     public static function unregister_authenticator($authenticator)
     {

src/Security/Group.php

Doc Comments +1 added lines

@@ -409,6 +409,7 @@
      * Overloaded to ensure the code is always descent.
      *
      * @param string
+     * @param string $val
      */
     public function setCode($val)
     {

src/Security/MemberAuthenticator.php

Doc Comments +1 added lines, -1 removed lines

@@ -157,7 +157,7 @@
      * @param Form $form Optional: If passed, better error messages can be
      *                             produced by using
      *                             {@link Form::sessionMessage()}
-     * @return bool|Member Returns FALSE if authentication fails, otherwise
+     * @return Member|null Returns FALSE if authentication fails, otherwise
      *                     the member object
      * @see Security::setDefaultAdmin()
      */