PHPOffice / PhpSpreadsheet

src/PhpSpreadsheet/Shared/OLERead.php

Doc Comments +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
     /**
      * Read the file.
      *
-     * @param $pFilename string Filename
+     * @param string $pFilename string Filename
      *
      * @throws ReaderException
      */

src/PhpSpreadsheet/Writer/Html.php

Doc Comments +1 added lines, -1 removed lines

@@ -1188,7 +1188,7 @@
      * @param Worksheet $pSheet \PhpOffice\PhpSpreadsheet\Worksheet
      * @param array $pValues Array containing cells in a row
      * @param int $pRow Row number (0-based)
-     * @param mixed $cellType eg: 'td'
+     * @param string $cellType eg: 'td'
      *
      * @throws WriterException
      *

src/PhpSpreadsheet/Chart/Layout.php

Doc Comments +4 added lines, -4 removed lines

@@ -236,7 +236,7 @@ 
     /**
      * Get X-Position.
      *
-     * @return number
+     * @return string
      */
     public function getXPosition()
     {
@@ -260,7 +260,7 @@ 
     /**
      * Get Y-Position.
      *
-     * @return number
+     * @return string
      */
     public function getYPosition()
     {
@@ -284,7 +284,7 @@ 
     /**
      * Get Width.
      *
-     * @return number
+     * @return string
      */
     public function getWidth()
     {
@@ -308,7 +308,7 @@ 
     /**
      * Get Height.
      *
-     * @return number
+     * @return string
      */
     public function getHeight()
     {

src/PhpSpreadsheet/Cell.php

Doc Comments +1 added lines, -1 removed lines

@@ -961,7 +961,7 @@
      *
      *    [ 'A1:A3' => 'x', 'A4' => 'y' ]
      *
-     * @param array $pCoordCollection associative array mapping coordinates to values
+     * @param Cell\DataValidation[] $pCoordCollection associative array mapping coordinates to values
      *
      * @return array associative array mapping coordinate ranges to valuea
      */

src/PhpSpreadsheet/Calculation/Engineering.php

Doc Comments +5 added lines, -3 removed lines

@@ -1855,7 +1855,7 @@ 
      *
      * @param string $complexNumber the complex number for which you want the real coefficient
      *
-     * @return float
+     * @return string
      */
     public static function IMREAL($complexNumber)
     {
@@ -2294,6 +2294,7 @@ 
      *        IMSUM(complexNumber[,complexNumber[,...]])
      *
      * @param string ...$complexNumbers Series of complex numbers to add
+     * @param string[] $complexNumbers
      *
      * @return string
      */
@@ -2334,6 +2335,7 @@ 
      *        IMPRODUCT(complexNumber[,complexNumber[,...]])
      *
      * @param string ...$complexNumbers Series of complex numbers to multiply
+     * @param string[] $complexNumbers
      *
      * @return string
      */
@@ -2377,7 +2379,7 @@ 
      *        DELTA(a[,b])
      *
      * @param float $a the first number
-     * @param float $b The second number. If omitted, b is assumed to be zero.
+     * @param integer $b The second number. If omitted, b is assumed to be zero.
      *
      * @return int
      */
@@ -2400,7 +2402,7 @@ 
      * functions you calculate the count of values that exceed a threshold.
      *
      * @param float $number the value to test against step
-     * @param float $step The threshold value.
+     * @param integer $step The threshold value.
      *                                    If you omit a value for step, GESTEP uses zero.
      *
      * @return int

src/PhpSpreadsheet/Calculation/MathTrig.php

Doc Comments +11 added lines, -11 removed lines

@@ -56,6 +56,10 @@ 
         return [(int) $value];
     }
 
+    /**
+     * @param integer $num
+     * @param integer $n
+     */
     private static function romanCut($num, $n)
     {
         return ($num - ($num % $n)) / $n;
@@ -508,7 +512,7 @@ 
      * @category Mathematical and Trigonometric Functions
      *
      * @param float $number The positive real number for which you want the logarithm
-     * @param float $base The base of the logarithm. If base is omitted, it is assumed to be 10.
+     * @param integer $base The base of the logarithm. If base is omitted, it is assumed to be 10.
      *
      * @return float
      */
@@ -826,8 +830,8 @@ 
      *
      * Computes x raised to the power y.
      *
-     * @param float $x
-     * @param float $y
+     * @param integer $x
+     * @param integer $y
      *
      * @return float
      */
@@ -901,7 +905,7 @@ 
      *
      * @param mixed ...$args Data values
      *
-     * @return float
+     * @return integer
      */
     public static function QUOTIENT(...$args)
     {
@@ -1039,9 +1043,6 @@ 
      *
      * Returns the sum of a power series
      *
-     * @param float $x Input value to the power series
-     * @param float $n Initial power to which you want to raise $x
-     * @param float $m Step by which to increase $n for each term in the series
      * @param array of mixed Data Series
      *
      * @return float
@@ -1219,7 +1220,7 @@ 
      * @param mixed $aArgs
      * @param mixed $sumArgs
      *
-     * @return float
+     * @return integer
      */
     public static function SUMIF($aArgs, $condition, $sumArgs = [])
     {
@@ -1259,9 +1260,8 @@ 
      *    @category Mathematical and Trigonometric Functions
      *
      * @param mixed $args Data values
-     * @param string $condition the criteria that defines which cells will be summed
      *
-     * @return float
+     * @return integer
      */
     public static function SUMIFS(...$args)
     {
@@ -1450,7 +1450,7 @@ 
      *
      * Truncates value to the number of fractional digits by number_digits.
      *
-     * @param float $value
+     * @param integer $value
      * @param int $digits
      *
      * @return float Truncated value

src/PhpSpreadsheet/Calculation.php

Spacing +1 added lines, -1 removed lines

@@ -3849,7 +3849,7 @@
      *
      * @return bool
      */
-    private function executeBinaryComparisonOperation($cellID, $operand1, $operand2, $operation, Stack &$stack, $recursingArrays = false)
+    private function executeBinaryComparisonOperation($cellID, $operand1, $operand2, $operation, Stack & $stack, $recursingArrays = false)
     {
         //    If we're dealing with matrix operations, we want a matrix result
         if ((is_array($operand1)) || (is_array($operand2))) {

Doc Comments +2 added lines, -2 removed lines

@@ -2664,7 +2664,7 @@ 
 
     /**
      * @param string $cellReference
-     * @param mixed $cellValue
+     * @param boolean $cellValue
      */
     public function saveValueToCache($cellReference, $cellValue)
     {
@@ -2789,7 +2789,7 @@ 
      *
      * @param mixed &$matrix matrix operand
      *
-     * @return int[] An array comprising the number of rows, and number of columns
+     * @return integer[] An array comprising the number of rows, and number of columns
      */
     private static function getMatrixDimensions(&$matrix)
     {

src/PhpSpreadsheet/NamedRange.php

Doc Comments +1 added lines, -1 removed lines

@@ -212,7 +212,7 @@
      * Resolve a named range to a regular cell range.
      *
      * @param string $pNamedRange Named range
-     * @param null|Worksheet $pSheet Scope. Use null for global scope
+     * @param Worksheet $pSheet Scope. Use null for global scope
      *
      * @return NamedRange
      */

src/PhpSpreadsheet/Shared/Font.php

Doc Comments +1 added lines, -1 removed lines

@@ -217,7 +217,7 @@
      * Calculate an (approximate) OpenXML column width, based on font size and text contained.
      *
      * @param \PhpOffice\PhpSpreadsheet\Style\Font $font Font object
-     * @param RichText|string $cellText Text to calculate width
+     * @param string $cellText Text to calculate width
      * @param int $rotation Rotation angle
      * @param null|\PhpOffice\PhpSpreadsheet\Style\Font $defaultFont Font object
      *