silverstripe / silverstripe-framework

tests/php/View/SSViewerCacheBlockTest.php

Doc Comments +3 added lines

@@ -31,6 +31,9 @@
         Injector::inst()->get(CacheInterface::class . '.cacheblock')->clear();
     }
 
+    /**
+     * @param string $template
+     */
     protected function _runtemplate($template, $data = null)
     {
         if ($data === null) {

src/Control/Director.php

Doc Comments +2 added lines, -2 removed lines

@@ -925,7 +925,7 @@ 
     /**
      * Returns true if the given file exists. Filename should be relative to the site root.
      *
-     * @param $file
+     * @param string $file
      *
      * @return bool
      */
@@ -1153,7 +1153,7 @@ 
      * Can also be checked with {@link Director::isDev()}, {@link Director::isTest()}, and
      * {@link Director::isLive()}.
      *
-     * @return bool
+     * @return string
      */
     public static function get_environment_type()
     {

src/Core/Manifest/Module.php

Doc Comments +4 added lines

@@ -28,6 +28,10 @@
      */
     protected $composerData = null;
 
+    /**
+     * @param string $path
+     * @param string $base
+     */
     public function __construct($path, $base)
     {
         $this->path = rtrim($path, '/\\');

src/Dev/FunctionalTest.php

Doc Comments +2 added lines, -2 removed lines

@@ -226,7 +226,7 @@ 
      * Find an attribute in a SimpleXMLElement object by name.
      * @param SimpleXMLElement $object
      * @param string $attribute Name of attribute to find
-     * @return SimpleXMLElement object of the attribute
+     * @return boolean object of the attribute
      */
     public function findAttribute($object, $attribute)
     {
@@ -260,7 +260,7 @@ 
      * Note:   characters are stripped from the content; make sure that your assertions take this into account.
      *
      * @param string $selector A basic CSS selector, e.g. 'li.jobs h3'
-     * @param array|string $expectedMatches The content of at least one of the matched tags
+     * @param string[] $expectedMatches The content of at least one of the matched tags
      * @param string $message
      * @throws PHPUnit_Framework_AssertionFailedError
      */

src/Security/PasswordEncryptor_Blowfish.php

Doc Comments +5 added lines

@@ -165,6 +165,11 @@
         return sprintf('%02d', self::$cost) . '$' . substr($generator->randomToken('sha1'), 0, 22);
     }
 
+    /**
+     * @param string|false $hash
+     * @param string $password
+     * @param string $salt
+     */
     public function check($hash, $password, $salt = null, $member = null)
     {
         if (strpos($hash, '$2y$') === 0) {

src/Forms/FormRequestHandler.php

Doc Comments +1 added lines, -1 removed lines

@@ -390,7 +390,7 @@
     /**
      * Fields can have action to, let's check if anyone of the responds to $funcname them
      *
-     * @param SS_List|array $fields
+     * @param FieldList $fields
      * @param callable $funcName
      * @return FormField
      */

src/Core/Cache/DefaultCacheFactory.php

Doc Comments +2 added lines, -2 removed lines

@@ -78,7 +78,7 @@ 
     /**
      * Determine if apcu is supported
      *
-     * @return bool
+     * @return boolean|null
      */
     protected function isAPCUSupported()
     {
@@ -92,7 +92,7 @@ 
     /**
      * Determine if PHP files is supported
      *
-     * @return bool
+     * @return boolean|null
      */
     protected function isPHPFilesSupported()
     {

src/Forms/HTMLEditor/HTMLEditorConfig.php

Doc Comments +1 added lines, -1 removed lines

@@ -151,7 +151,7 @@
      * Get the current value of an option
      *
      * @param string $key The key of the option to get
-     * @return mixed The value of the specified option
+     * @return string The value of the specified option
      */
     abstract public function getOption($key);
 

src/Forms/LiteralField.php

Doc Comments +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 
     /**
      * @param string $name
-     * @param string|FormField $content
+     * @param string $content
      */
     public function __construct($name, $content)
     {