silverstripe / silverstripe-framework

src/ORM/FieldType/DBTime.php

Doc Comments +3 added lines

@@ -26,6 +26,9 @@
      */
     const ISO_TIME = 'HH:mm:ss';
 
+    /**
+     * @param string $value
+     */
     public function setValue($value, $record = null, $markChanged = true)
     {
         $value = $this->parseTime($value);

src/ORM/FieldType/DBDate.php

Doc Comments +2 added lines, -2 removed lines

@@ -245,7 +245,7 @@ 
      * Return a date formatted as per a CMS user's settings.
      *
      * @param Member $member
-     * @return boolean | string A date formatted as per user-defined settings.
+     * @return null|string | string A date formatted as per user-defined settings.
      */
     public function FormatFromSettings($member = null)
     {
@@ -551,7 +551,7 @@ 
      * Attempt to split date string into year, month, day, and timestamp components.
      *
      * @param string $value
-     * @return array
+     * @return string[]
      */
     protected function explodeDateString($value)
     {

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/Core/Cache/CacheFactory.php

Doc Comments -2 removed lines

@@ -12,8 +12,6 @@
      * Note: While the returned object is used as a singleton (by the originating Injector->get() call),
      * this cache object shouldn't be a singleton itself - it has varying constructor args for the same service name.
      *
-     * @param string $class
-     * @param array $args
      * @return CacheInterface
      */
     public function create($service, array $params = array());