silverstripe / silverstripe-framework

src/Forms/DefaultFormFactory.php

Doc Comments +1 added lines, -1 removed lines

@@ -104,7 +104,7 @@
     /**
      * Return list of mandatory context keys
      *
-     * @return mixed
+     * @return string[]
      */
     public function getRequiredContext()
     {

src/Security/Member.php

Doc Comments +6 added lines, -6 removed lines

@@ -799,7 +799,7 @@ 
     /**
      * Builds "Change / Create Password" field for this member
      *
-     * @return ConfirmedPasswordField
+     * @return \SilverStripe\Forms\FormField
      */
     public function getMemberPasswordField()
     {
@@ -879,9 +879,9 @@ 
      * });
      * </code>
      *
-     * @param Member|null|int $member Member or member ID to log in as.
+     * @param Member|null $member Member or member ID to log in as.
      * Set to null or 0 to act as a logged out user.
-     * @param $callback
+     * @param \Closure $callback
      */
     public static function actAs($member, $callback)
     {
@@ -1633,7 +1633,7 @@ 
      * This is likely to be customized for social sites etc. with a looser permission model.
      *
      * @param Member $member
-     * @return bool
+     * @return boolean|string
      */
     public function canView($member = null)
     {
@@ -1664,7 +1664,7 @@ 
      * Otherwise they'll need ADMIN or CMS_ACCESS_SecurityAdmin permissions
      *
      * @param Member $member
-     * @return bool
+     * @return boolean|string
      */
     public function canEdit($member = null)
     {
@@ -1699,7 +1699,7 @@ 
      * Otherwise they'll need ADMIN or CMS_ACCESS_SecurityAdmin permissions
      *
      * @param Member $member
-     * @return bool
+     * @return boolean|string
      */
     public function canDelete($member = null)
     {

tests/php/View/Parsers/ShortcodeParserTest.php

Doc Comments +3 added lines

@@ -208,6 +208,9 @@
         $this->assertEquals('', $this->parser->parse('[test_shortcode][test_shortcode]'));
     }
 
+    /**
+     * @param string $a
+     */
     protected function assertEqualsIgnoringWhitespace($a, $b, $message = null)
     {
         $this->assertEquals(preg_replace('/\s+/', '', $a), preg_replace('/\s+/', '', $b), $message);