PHPOffice / PhpSpreadsheet

src/PhpSpreadsheet/Writer/Excel5/Worksheet.php

Doc Comments +9 added lines, -7 removed lines

@@ -199,10 +199,12 @@ 
      * @param int        &$str_unique    Total number of unique strings
      * @param array        &$str_table        String Table
      * @param array        &$colors        Colour Table
-     * @param mixed        $parser            The formula parser created for the Workbook
+     * @param Parser        $parser            The formula parser created for the Workbook
      * @param bool    $preCalculateFormulas    Flag indicating whether formulas should be calculated or just written
      * @param string    $phpSheet        The worksheet to write
      * @param \PhpSpreadsheet\Worksheet $phpSheet
+     * @param integer $str_total
+     * @param integer $str_unique
      */
     public function __construct(&$str_total, &$str_unique, &$str_table, &$colors, $parser, $preCalculateFormulas, $phpSheet)
     {
@@ -611,7 +613,7 @@ 
      * @param int $row    Zero indexed row
      * @param int $col    Zero indexed column
      * @param float   $num    The number to write
-     * @param mixed   $xfIndex The optional XF format
+     * @param integer   $xfIndex The optional XF format
      * @return int
      */
     private function writeNumber($row, $col, $num, $xfIndex)
@@ -717,7 +719,7 @@ 
      * @param int $row    Zero indexed row
      * @param int $col    Zero indexed column
      * @param string  $str    The string to write
-     * @param mixed   $xfIndex The XF format index for the cell
+     * @param integer   $xfIndex The XF format index for the cell
      * @return int
      */
     private function writeLabelSst($row, $col, $str, $xfIndex)
@@ -783,7 +785,7 @@ 
      *
      * @param int $row    Zero indexed row
      * @param int $col    Zero indexed column
-     * @param mixed   $xfIndex The XF format index
+     * @param integer   $xfIndex The XF format index
      */
     public function writeBlank($row, $col, $xfIndex)
     {
@@ -830,7 +832,7 @@ 
      * @param int $row     Zero indexed row
      * @param int $col     Zero indexed column
      * @param string  $formula The formula text string
-     * @param mixed   $xfIndex  The XF format index
+     * @param integer   $xfIndex  The XF format index
      * @param mixed   $calculatedValue  Calculated value
      * @return int
      */
@@ -2299,8 +2301,8 @@ 
      * @param mixed   $bitmap  The bitmap filename or GD-image resource
      * @param int $x       The horizontal position (offset) of the image inside the cell.
      * @param int $y       The vertical position (offset) of the image inside the cell.
-     * @param float   $scale_x The horizontal scale
-     * @param float   $scale_y The vertical scale
+     * @param integer   $scale_x The horizontal scale
+     * @param integer   $scale_y The vertical scale
      */
     public function insertBitmap($row, $col, $bitmap, $x = 0, $y = 0, $scale_x = 1, $scale_y = 1)
     {

Indentation +8 added lines, -8 removed lines

@@ -1147,14 +1147,14 @@
 
         // Pack the main data stream
         $data = pack('vvvv', $row1, $row2, $col1, $col2) .
-                          $unknown1 .
-                          $link_type .
-                          $unknown2 .
-                          $up_count .
-                          $dir_short_len .
-                          $dir_short .
-                          $unknown3 .
-                          $stream_len; /*.
+                            $unknown1 .
+                            $link_type .
+                            $unknown2 .
+                            $up_count .
+                            $dir_short_len .
+                            $dir_short .
+                            $unknown3 .
+                            $stream_len; /*.
                           $dir_long_len .
                           $unknown4     .
                           $dir_long     .

src/PhpSpreadsheet/Writer/OpenDocument.php

Doc Comments +1 added lines, -1 removed lines

@@ -181,7 +181,7 @@
      *
      * @param  \PhpSpreadsheet\Spreadsheet $spreadsheet  PhpSpreadsheet object
      * @throws \PhpSpreadsheet\Writer\Exception
-     * @return Excel2007
+     * @return OpenDocument
      */
     public function setSpreadsheet(\PhpSpreadsheet\SpreadSheet $spreadsheet = null)
     {

tests/PhpSpreadsheetTests/Shared/StringTest.php

Spacing +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
     public function testGetCurrencyCode()
     {
         $localeconv = localeconv();
-        $expectedResult = (!empty($localeconv['currency_symbol']) ? $localeconv['currency_symbol'] : (!empty($localeconv['int_curr_symbol']) ? $localeconv['int_curr_symbol']: '$'));
+        $expectedResult = (!empty($localeconv['currency_symbol']) ? $localeconv['currency_symbol'] : (!empty($localeconv['int_curr_symbol']) ? $localeconv['int_curr_symbol'] : '$'));
         $result = call_user_func([StringHelper::class, 'getCurrencyCode']);
         $this->assertEquals($expectedResult, $result);
     }

tests/PhpSpreadsheetTests/Worksheet/ColumnIteratorTest.php

Indentation +4 added lines, -4 removed lines

@@ -18,11 +18,11 @@
             ->getMock();
 
         $this->mockWorksheet->expects($this->any())
-                 ->method('getHighestColumn')
-                 ->will($this->returnValue('E'));
+                    ->method('getHighestColumn')
+                    ->will($this->returnValue('E'));
         $this->mockWorksheet->expects($this->any())
-                 ->method('current')
-                 ->will($this->returnValue($this->mockColumn));
+                    ->method('current')
+                    ->will($this->returnValue($this->mockColumn));
     }
 
 

tests/PhpSpreadsheetTests/Worksheet/RowCellIteratorTest.php

Indentation +4 added lines, -4 removed lines

@@ -18,11 +18,11 @@
             ->getMock();
 
         $this->mockWorksheet->expects($this->any())
-                 ->method('getHighestColumn')
-                 ->will($this->returnValue('E'));
+                    ->method('getHighestColumn')
+                    ->will($this->returnValue('E'));
         $this->mockWorksheet->expects($this->any())
-                 ->method('getCellByColumnAndRow')
-                 ->will($this->returnValue($this->mockCell));
+                    ->method('getCellByColumnAndRow')
+                    ->will($this->returnValue($this->mockCell));
     }
 
 

tests/PhpSpreadsheetTests/Worksheet/ColumnCellIteratorTest.php

Indentation +4 added lines, -4 removed lines

@@ -18,11 +18,11 @@
             ->getMock();
 
         $this->mockWorksheet->expects($this->any())
-                 ->method('getHighestRow')
-                 ->will($this->returnValue(5));
+                    ->method('getHighestRow')
+                    ->will($this->returnValue(5));
         $this->mockWorksheet->expects($this->any())
-                 ->method('getCellByColumnAndRow')
-                 ->will($this->returnValue($this->mockCell));
+                    ->method('getCellByColumnAndRow')
+                    ->will($this->returnValue($this->mockCell));
     }
 
 

tests/PhpSpreadsheetTests/Worksheet/AutoFilter/ColumnTest.php

Indentation +4 added lines, -4 removed lines

@@ -113,7 +113,7 @@ 
     {
         $attributeSet = ['val' => 100,
                                 'maxVal' => 200,
-                             ];
+                                ];
 
         //    Setters return the instance to implement the fluent interface
         $result = $this->testAutoFilterColumnObject->setAttributes($attributeSet);
@@ -124,7 +124,7 @@ 
     {
         $attributeSet = ['val' => 100,
                                 'maxVal' => 200,
-                             ];
+                                ];
 
         $this->testAutoFilterColumnObject->setAttributes($attributeSet);
 
@@ -137,7 +137,7 @@ 
     {
         $attributeSet = ['val' => 100,
                                 'maxVal' => 200,
-                             ];
+                                ];
 
         foreach ($attributeSet as $attributeName => $attributeValue) {
             //    Setters return the instance to implement the fluent interface
@@ -150,7 +150,7 @@ 
     {
         $attributeSet = ['val' => 100,
                                 'maxVal' => 200,
-                             ];
+                                ];
 
         $this->testAutoFilterColumnObject->setAttributes($attributeSet);
 

tests/PhpSpreadsheetTests/Worksheet/RowIteratorTest.php

Indentation +4 added lines, -4 removed lines

@@ -18,11 +18,11 @@
             ->getMock();
 
         $this->mockWorksheet->expects($this->any())
-                 ->method('getHighestRow')
-                 ->will($this->returnValue(5));
+                    ->method('getHighestRow')
+                    ->will($this->returnValue(5));
         $this->mockWorksheet->expects($this->any())
-                 ->method('current')
-                 ->will($this->returnValue($this->mockRow));
+                    ->method('current')
+                    ->will($this->returnValue($this->mockRow));
     }
 
 

tests/PhpSpreadsheetTests/Worksheet/WorksheetRowTest.php

Indentation +2 added lines, -2 removed lines

@@ -13,8 +13,8 @@
             ->disableOriginalConstructor()
             ->getMock();
         $this->mockWorksheet->expects($this->any())
-                 ->method('getHighestColumn')
-                 ->will($this->returnValue('E'));
+                    ->method('getHighestColumn')
+                    ->will($this->returnValue('E'));
     }