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

src/Security/PasswordValidator.php

Doc Comments +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
      * Eg: $this->characterStrength(3, array("lowercase", "uppercase", "digits", "punctuation"))
      *
      * @param int $minScore The minimum number of character tests that must pass
-     * @param array $testNames The names of the tests to perform
+     * @param string[] $testNames The names of the tests to perform
      * @return $this
      */
     public function characterStrength($minScore, $testNames)

src/Security/Permission.php

Doc Comments +2 added lines, -2 removed lines

@@ -122,7 +122,7 @@ 
      *  will be checked for the current user
      * @param bool $strict Use "strict" checking (which means a permission
      *  will be granted if the key does not exist at all)?
-     * @return int|bool The ID of the permission record if the permission
+     * @return boolean|string The ID of the permission record if the permission
      *  exists; FALSE otherwise. If "strict" checking is
      *  disabled, TRUE will be returned if the permission does not exist at all.
      */
@@ -162,7 +162,7 @@ 
      * @param string $arg Optional argument (e.g. a permissions for a specific page)
      * @param bool $strict Use "strict" checking (which means a permission
      *  will be granted if the key does not exist at all)?
-     * @return int|bool The ID of the permission record if the permission
+     * @return boolean|string The ID of the permission record if the permission
      *  exists; FALSE otherwise. If "strict" checking is
      *  disabled, TRUE will be returned if the permission does not exist at all.
      */

src/Security/Permission_Group.php

Doc Comments +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
     /**
      * Get permissions
      *
-     * @return array Associative array of permissions in this permission
+     * @return string Associative array of permissions in this permission
      *               group. The array indicies are the permission codes as
      *               used in {@link Permission::check()}. The value is
      *               suitable for using in an interface.