Completed
Pull Request — develop (#147)
by
unknown
23:42
created
src/PhpSpreadsheet/Calculation/LookupRef.php 1 patch
Doc Comments   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
      *
37 37
      * @param row Row number to use in the cell reference
38 38
      * @param column Column number to use in the cell reference
39
-     * @param relativity Flag indicating the type of reference to return
39
+     * @param relativity integer indicating the type of reference to return
40 40
      *                                1 or omitted    Absolute
41 41
      *                                2                Absolute row; relative column
42 42
      *                                3                Relative row; absolute column
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
      * @param sheetText Optional Name of worksheet to use
48 48
      * @param mixed $row
49 49
      * @param mixed $column
50
-     * @param mixed $relativity
50
+     * @param integer $relativity
51 51
      * @param mixed $referenceStyle
52 52
      * @param mixed $sheetText
53 53
      *
@@ -358,7 +358,7 @@  discard block
 block discarded – undo
358 358
      *
359 359
      * @param cellAddress The reference from which you want to base the offset. Reference must refer to a cell or
360 360
      *                                range of adjacent cells; otherwise, OFFSET returns the #VALUE! error value.
361
-     * @param rows The number of rows, up or down, that you want the upper-left cell to refer to.
361
+     * @param rows integer number of rows, up or down, that you want the upper-left cell to refer to.
362 362
      *                                Using 5 as the rows argument specifies that the upper-left cell in the reference is
363 363
      *                                five rows below reference. Rows can be positive (which means below the starting reference)
364 364
      *                                or negative (which means above the starting reference).
@@ -370,8 +370,8 @@  discard block
 block discarded – undo
370 370
      * @param height The height, in number of rows, that you want the returned reference to be. Height must be a positive number.
371 371
      * @param width The width, in number of columns, that you want the returned reference to be. Width must be a positive number.
372 372
      * @param null|mixed $cellAddress
373
-     * @param mixed $rows
374
-     * @param mixed $columns
373
+     * @param integer $rows
374
+     * @param integer $columns
375 375
      * @param null|mixed $height
376 376
      * @param null|mixed $width
377 377
      * @param \PhpOffice\PhpSpreadsheet\Cell $pCell
@@ -499,7 +499,7 @@  discard block
 block discarded – undo
499 499
      *
500 500
      * @param mixed $lookupValue The value that you want to match in lookup_array
501 501
      * @param mixed $lookupArray The range of cells being searched
502
-     * @param mixed $matchType The number -1, 0, or 1. -1 means above, 0 means exact match, 1 means below. If match_type is 1 or -1, the list has to be ordered.
502
+     * @param integer $matchType The number -1, 0, or 1. -1 means above, 0 means exact match, 1 means below. If match_type is 1 or -1, the list has to be ordered.
503 503
      *
504 504
      * @return int The relative position of the found item
505 505
      */
@@ -613,8 +613,8 @@  discard block
 block discarded – undo
613 613
      * @param row_num The row in array from which to return a value. If row_num is omitted, column_num is required.
614 614
      * @param column_num The column in array from which to return a value. If column_num is omitted, row_num is required.
615 615
      * @param mixed $arrayValues
616
-     * @param mixed $rowNum
617
-     * @param mixed $columnNum
616
+     * @param integer $rowNum
617
+     * @param integer $columnNum
618 618
      *
619 619
      * @return mixed the value of a specified cell or array of cells
620 620
      */
@@ -710,11 +710,11 @@  discard block
 block discarded – undo
710 710
      *
711 711
      * @param lookup_value The value that you want to match in lookup_array
712 712
      * @param lookup_array The range of cells being searched
713
-     * @param index_number The column number in table_array from which the matching value must be returned. The first column is 1.
713
+     * @param index_number integer column number in table_array from which the matching value must be returned. The first column is 1.
714 714
      * @param not_exact_match determines if you are looking for an exact match based on lookup_value
715 715
      * @param mixed $lookup_value
716 716
      * @param mixed $lookup_array
717
-     * @param mixed $index_number
717
+     * @param integer $index_number
718 718
      * @param mixed $not_exact_match
719 719
      *
720 720
      * @return mixed The value of the found cell
Please login to merge, or discard this patch.
docs/Examples/Reading WorkBook Data/exampleWorkBookReader04.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -34,13 +34,13 @@
 block discarded – undo
34 34
 echo 'Reading the number of Worksheets in the WorkBook<br />';
35 35
 /*  Use the PhpSpreadsheet object's getSheetCount() method to get a count of the number of WorkSheets in the WorkBook  */
36 36
 $sheetCount = $spreadsheet->getSheetCount();
37
-echo 'There ',(($sheetCount == 1) ? 'is' : 'are'),' ',$sheetCount,' WorkSheet',(($sheetCount == 1) ? '' : 's'),' in the WorkBook<br /><br />';
37
+echo 'There ', (($sheetCount == 1) ? 'is' : 'are'), ' ', $sheetCount, ' WorkSheet', (($sheetCount == 1) ? '' : 's'), ' in the WorkBook<br /><br />';
38 38
 
39 39
 echo 'Reading the names of Worksheets in the WorkBook<br />';
40 40
 /*  Use the PhpSpreadsheet object's getSheetNames() method to get an array listing the names/titles of the WorkSheets in the WorkBook  */
41 41
 $sheetNames = $spreadsheet->getSheetNames();
42 42
 foreach ($sheetNames as $sheetIndex => $sheetName) {
43
-    echo 'WorkSheet #',$sheetIndex,' is named "',$sheetName,'"<br />';
43
+    echo 'WorkSheet #', $sheetIndex, ' is named "', $sheetName, '"<br />';
44 44
 }
45 45
 
46 46
 ?>
Please login to merge, or discard this patch.
src/PhpSpreadsheet/Collection/Cells.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -158,7 +158,7 @@
 block discarded – undo
158 158
     /**
159 159
      * Get a list of all cell coordinates currently held in the collection.
160 160
      *
161
-     * @return string[]
161
+     * @return integer[]
162 162
      */
163 163
     public function getCoordinates()
164 164
     {
Please login to merge, or discard this patch.
src/PhpSpreadsheet/NamedRange.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -234,7 +234,7 @@
 block discarded – undo
234 234
      * Resolve a named range to a regular cell range.
235 235
      *
236 236
      * @param string $pNamedRange Named range
237
-     * @param Worksheet|null $pSheet Scope. Use null for global scope
237
+     * @param Worksheet $pSheet Scope. Use null for global scope
238 238
      *
239 239
      * @return NamedRange
240 240
      */
Please login to merge, or discard this patch.
src/PhpSpreadsheet/Reader/Gnumeric.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -804,6 +804,9 @@
 block discarded – undo
804 804
         return $styleArray;
805 805
     }
806 806
 
807
+    /**
808
+     * @param string $is
809
+     */
807 810
     private function parseRichText($is)
808 811
     {
809 812
         $value = new \PhpOffice\PhpSpreadsheet\RichText();
Please login to merge, or discard this patch.
src/PhpSpreadsheet/Reader/Slk.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -472,7 +472,7 @@
 block discarded – undo
472 472
      *
473 473
      * @param int $pValue Sheet index
474 474
      *
475
-     * @return SYLK
475
+     * @return Slk
476 476
      */
477 477
     public function setSheetIndex($pValue)
478 478
     {
Please login to merge, or discard this patch.
src/PhpSpreadsheet/Reader/Xml.php 1 patch
Doc Comments   +9 added lines patch added patch discarded remove patch
@@ -230,6 +230,9 @@  discard block
 block discarded – undo
230 230
         return $this->loadIntoExisting($pFilename, $spreadsheet);
231 231
     }
232 232
 
233
+    /**
234
+     * @param string $styleAttributeValue
235
+     */
233 236
     protected static function identifyFixedStyleValue($styleList, &$styleAttributeValue)
234 237
     {
235 238
         $styleAttributeValue = strtolower($styleAttributeValue);
@@ -761,6 +764,9 @@  discard block
 block discarded – undo
761 764
         return $spreadsheet;
762 765
     }
763 766
 
767
+    /**
768
+     * @param string $charset
769
+     */
764 770
     protected static function convertStringEncoding($string, $charset)
765 771
     {
766 772
         if ($charset != 'UTF-8') {
@@ -770,6 +776,9 @@  discard block
 block discarded – undo
770 776
         return $string;
771 777
     }
772 778
 
779
+    /**
780
+     * @param string $is
781
+     */
773 782
     protected function parseRichText($is)
774 783
     {
775 784
         $value = new \PhpOffice\PhpSpreadsheet\RichText();
Please login to merge, or discard this patch.
src/PhpSpreadsheet/Shared/Drawing.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -31,7 +31,7 @@
 block discarded – undo
31 31
      *
32 32
      * @param int $pValue Value in pixels
33 33
      *
34
-     * @return int Value in EMU
34
+     * @return double Value in EMU
35 35
      */
36 36
     public static function pixelsToEMU($pValue)
37 37
     {
Please login to merge, or discard this patch.
src/PhpSpreadsheet/Shared/JAMA/CholeskyDecomposition.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -46,7 +46,7 @@
 block discarded – undo
46 46
      *    Class constructor - decomposes symmetric positive definite matrix
47 47
      *
48 48
      * @param mixed Matrix square symmetric positive definite matrix
49
-     * @param null|mixed $A
49
+     * @param Matrix $A
50 50
      */
51 51
     public function __construct(Matrix $A)
52 52
     {
Please login to merge, or discard this patch.