Completed
Pull Request — master (#6641)
by Damian
09:35 queued 27s
created
admin/code/LeftAndMain.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -1002,7 +1002,7 @@  discard block
 block discarded – undo
1002 1002
      * {@link SSViewer::get_templates_by_class()}
1003 1003
      *
1004 1004
      * @param string $suffix
1005
-     * @return array
1005
+     * @return string
1006 1006
      */
1007 1007
     public function getTemplatesWithSuffix($suffix)
1008 1008
     {
@@ -2154,7 +2154,7 @@  discard block
 block discarded – undo
2154 2154
     }
2155 2155
 
2156 2156
     /**
2157
-     * @return String
2157
+     * @return DBField
2158 2158
      */
2159 2159
     public function Locale()
2160 2160
     {
Please login to merge, or discard this patch.
src/Control/Director.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -925,7 +925,7 @@  discard block
 block discarded – undo
925 925
     /**
926 926
      * Returns true if the given file exists. Filename should be relative to the site root.
927 927
      *
928
-     * @param $file
928
+     * @param string $file
929 929
      *
930 930
      * @return bool
931 931
      */
@@ -1153,7 +1153,7 @@  discard block
 block discarded – undo
1153 1153
      * Can also be checked with {@link Director::isDev()}, {@link Director::isTest()}, and
1154 1154
      * {@link Director::isLive()}.
1155 1155
      *
1156
-     * @return bool
1156
+     * @return string
1157 1157
      */
1158 1158
     public static function get_environment_type()
1159 1159
     {
Please login to merge, or discard this patch.
src/Core/Manifest/Module.php 1 patch
Doc Comments   +4 added lines patch added patch discarded remove patch
@@ -28,6 +28,10 @@
 block discarded – undo
28 28
      */
29 29
     protected $composerData = null;
30 30
 
31
+    /**
32
+     * @param string $path
33
+     * @param string $base
34
+     */
31 35
     public function __construct($path, $base)
32 36
     {
33 37
         $this->path = rtrim($path, '/\\');
Please login to merge, or discard this patch.
src/Dev/SapphireTest.php 1 patch
Doc Comments   +7 added lines, -4 removed lines patch added patch discarded remove patch
@@ -184,6 +184,9 @@  discard block
 block discarded – undo
184 184
         return self::$is_running_test;
185 185
     }
186 186
 
187
+    /**
188
+     * @param boolean $bool
189
+     */
187 190
     public static function set_is_running_test($bool)
188 191
     {
189 192
         self::$is_running_test = $bool;
@@ -658,10 +661,10 @@  discard block
 block discarded – undo
658 661
     /**
659 662
      * Assert that the matching email was sent since the last call to clearEmails()
660 663
      * All of the parameters can either be a string, or, if they start with "/", a PREG-compatible regular expression.
661
-     * @param $to
664
+     * @param string $to
662 665
      * @param $from
663
-     * @param $subject
664
-     * @param $content
666
+     * @param string $subject
667
+     * @param string $content
665 668
      * @return array Contains the keys: 'type', 'to', 'from', 'subject', 'content', 'plainContent', 'attachedFiles',
666 669
      *               'customHeaders', 'htmlContent', inlineImages'
667 670
      */
@@ -894,7 +897,7 @@  discard block
 block discarded – undo
894 897
      * @param string $expectedSQL
895 898
      * @param string $actualSQL
896 899
      * @param string $message
897
-     * @param float|int $delta
900
+     * @param integer $delta
898 901
      * @param integer $maxDepth
899 902
      * @param boolean $canonicalize
900 903
      * @param boolean $ignoreCase
Please login to merge, or discard this patch.