PHPOffice / PhpSpreadsheet

src/PhpSpreadsheet/Reader/SYLK.php

Spacing +2 added lines, -2 removed lines

@@ -219,8 +219,8 @@
         }
         $spreadsheet->setActiveSheetIndex($this->sheetIndex);
 
-        $fromFormats = ['\-',    '\ '];
-        $toFormats = ['-',    ' '];
+        $fromFormats = ['\-', '\ '];
+        $toFormats = ['-', ' '];
 
         // Loop through file
         $rowData = [];

src/PhpSpreadsheet/Reader/OOCalc.php

Spacing +4 added lines, -4 removed lines

@@ -626,10 +626,10 @@
                                     foreach ($temp as &$value) {
                                         //    Only replace in alternate array entries (i.e. non-quoted blocks)
                                         if ($tKey = !$tKey) {
-                                            $value = preg_replace('/\[([^\.]+)\.([^\.]+):\.([^\.]+)\]/Ui', '$1!$2:$3', $value);    //  Cell range reference in another sheet
-                                            $value = preg_replace('/\[([^\.]+)\.([^\.]+)\]/Ui', '$1!$2', $value);       //  Cell reference in another sheet
-                                            $value = preg_replace('/\[\.([^\.]+):\.([^\.]+)\]/Ui', '$1:$2', $value);    //  Cell range reference
-                                            $value = preg_replace('/\[\.([^\.]+)\]/Ui', '$1', $value);                  //  Simple cell reference
+                                            $value = preg_replace('/\[([^\.]+)\.([^\.]+):\.([^\.]+)\]/Ui', '$1!$2:$3', $value); //  Cell range reference in another sheet
+                                            $value = preg_replace('/\[([^\.]+)\.([^\.]+)\]/Ui', '$1!$2', $value); //  Cell reference in another sheet
+                                            $value = preg_replace('/\[\.([^\.]+):\.([^\.]+)\]/Ui', '$1:$2', $value); //  Cell range reference
+                                            $value = preg_replace('/\[\.([^\.]+)\]/Ui', '$1', $value); //  Simple cell reference
                                             $value = \PhpSpreadsheet\Calculation::translateSeparator(';', ',', $value, $inBraces);
                                         }
                                     }

src/PhpSpreadsheet/Shared/Date.php

Spacing +2 added lines, -2 removed lines

@@ -29,8 +29,8 @@
 class Date
 {
     /** constants */
-    const CALENDAR_WINDOWS_1900 = 1900;        //    Base date of 1st Jan 1900 = 1.0
-    const CALENDAR_MAC_1904 = 1904;            //    Base date of 2nd Jan 1904 = 1.0
+    const CALENDAR_WINDOWS_1900 = 1900; //    Base date of 1st Jan 1900 = 1.0
+    const CALENDAR_MAC_1904 = 1904; //    Base date of 2nd Jan 1904 = 1.0
 
     /*
      * Names of the months of the year, indexed by shortname

Doc Comments +5 added lines, -5 removed lines

@@ -164,7 +164,7 @@ 
     /**
      * Convert a MS serialized datetime value from Excel to a PHP Date/Time object
      *
-     * @param     int|float                      $excelTimestamp      MS Excel serialized date/time value
+     * @param     integer                      $excelTimestamp      MS Excel serialized date/time value
      * @param     \DateTimeZone|string|null          $timeZone            The timezone to assume for the Excel timestamp,
      *                                                                        if you don't want to treat it as a UTC value
      *                                                                    Use the default (UST) unless you absolutely need a conversion
@@ -203,7 +203,7 @@ 
     /**
      * Convert a MS serialized datetime value from Excel to a unix timestamp
      *
-     * @param     int|float                      $excelTimestamp        MS Excel serialized date/time value
+     * @param     integer                      $excelTimestamp        MS Excel serialized date/time value
      * @param     \DateTimeZone|string|null          $timeZone            The timezone to assume for the Excel timestamp,
      *                                                                        if you don't want to treat it as a UTC value
      *                                                                    Use the default (UST) unless you absolutely need a conversion
@@ -219,7 +219,7 @@ 
     /**
      *    Convert a date from PHP to an MS Excel serialized date/time value
      *
-     *    @param    mixed            $dateValue            Unix Timestamp or PHP DateTime object or a string
+     *    @param    integer            $dateValue            Unix Timestamp or PHP DateTime object or a string
      *    @return   float|bool    Excel date/time value
      *                                  or boolean FALSE on failure
      */
@@ -257,7 +257,7 @@ 
     /**
      *    Convert a Unix timestamp to an MS Excel serialized date/time value
      *
-     *    @param    \DateTimeInterface    $dateValue       Unix Timestamp
+     *    @param    integer    $dateValue       Unix Timestamp
      *    @return   float                 MS Excel serialized date/time value
      */
     public static function timestampToExcel($dateValue = 0)
@@ -278,7 +278,7 @@ 
      * @param    int    $hours
      * @param    int    $minutes
      * @param    int    $seconds
-     * @return   int    Excel date/time value
+     * @return   double    Excel date/time value
      */
     public static function formattedPHPToExcel($year, $month, $day, $hours = 0, $minutes = 0, $seconds = 0)
     {

src/PhpSpreadsheet/Shared/Excel5.php

Spacing +2 added lines, -2 removed lines

@@ -233,8 +233,8 @@
         $y1 = $offsetY;
 
         // Initialise end cell to the same as the start cell
-        $col_end = $col_start;  // Col containing lower right corner of object
-        $row_end = $row_start;  // Row containing bottom right corner of object
+        $col_end = $col_start; // Col containing lower right corner of object
+        $row_end = $row_start; // Row containing bottom right corner of object
 
         // Zero the specified offset if greater than the cell dimensions
         if ($x1 >= self::sizeCol($sheet, \PhpSpreadsheet\Cell::stringFromColumnIndex($col_start))) {

src/PhpSpreadsheet/Shared/PasswordHasher.php

Spacing +4 added lines, -4 removed lines

@@ -41,14 +41,14 @@
     public static function hashPassword($pPassword = '')
     {
         $password = 0x0000;
-        $charPos = 1;       // char position
+        $charPos = 1; // char position
 
         // split the plain text password in its component characters
         $chars = preg_split('//', $pPassword, -1, PREG_SPLIT_NO_EMPTY);
         foreach ($chars as $char) {
-            $value = ord($char) << $charPos++;    // shifted ASCII value
-            $rotated_bits = $value >> 15;                // rotated bits beyond bit 15
-            $value &= 0x7fff;                    // first 15 bits
+            $value = ord($char) << $charPos++; // shifted ASCII value
+            $rotated_bits = $value >> 15; // rotated bits beyond bit 15
+            $value &= 0x7fff; // first 15 bits
             $password ^= ($value | $rotated_bits);
         }
 

src/PhpSpreadsheet/Shared/OLE/PPS.php

Indentation +15 added lines, -15 removed lines

@@ -170,21 +170,21 @@
         $ret = str_pad($this->Name, 64, "\x00");
 
         $ret .= pack('v', strlen($this->Name) + 2)  // 66
-              . pack('c', $this->Type)              // 67
-              . pack('c', 0x00) //UK                // 68
-              . pack('V', $this->PrevPps) //Prev    // 72
-              . pack('V', $this->NextPps) //Next    // 76
-              . pack('V', $this->DirPps)  //Dir     // 80
-              . "\x00\x09\x02\x00"                  // 84
-              . "\x00\x00\x00\x00"                  // 88
-              . "\xc0\x00\x00\x00"                  // 92
-              . "\x00\x00\x00\x46"                  // 96 // Seems to be ok only for Root
-              . "\x00\x00\x00\x00"                  // 100
-              . \PhpSpreadsheet\Shared\OLE::localDateToOLE($this->Time1st)          // 108
-              . \PhpSpreadsheet\Shared\OLE::localDateToOLE($this->Time2nd)          // 116
-              . pack('V', isset($this->startBlock) ? $this->startBlock : 0)  // 120
-              . pack('V', $this->Size)               // 124
-              . pack('V', 0);                        // 128
+                . pack('c', $this->Type)              // 67
+                . pack('c', 0x00) //UK                // 68
+                . pack('V', $this->PrevPps) //Prev    // 72
+                . pack('V', $this->NextPps) //Next    // 76
+                . pack('V', $this->DirPps)  //Dir     // 80
+                . "\x00\x09\x02\x00"                  // 84
+                . "\x00\x00\x00\x00"                  // 88
+                . "\xc0\x00\x00\x00"                  // 92
+                . "\x00\x00\x00\x46"                  // 96 // Seems to be ok only for Root
+                . "\x00\x00\x00\x00"                  // 100
+                . \PhpSpreadsheet\Shared\OLE::localDateToOLE($this->Time1st)          // 108
+                . \PhpSpreadsheet\Shared\OLE::localDateToOLE($this->Time2nd)          // 116
+                . pack('V', isset($this->startBlock) ? $this->startBlock : 0)  // 120
+                . pack('V', $this->Size)               // 124
+                . pack('V', 0);                        // 128
         return $ret;
     }
 

Spacing +1 added lines, -1 removed lines

@@ -184,7 +184,7 @@
               . \PhpSpreadsheet\Shared\OLE::localDateToOLE($this->Time2nd)          // 116
               . pack('V', isset($this->startBlock) ? $this->startBlock : 0)  // 120
               . pack('V', $this->Size)               // 124
-              . pack('V', 0);                        // 128
+              . pack('V', 0); // 128
         return $ret;
     }
 

src/PhpSpreadsheet/Worksheet/BaseDrawing.php

Spacing +1 added lines, -2 removed lines

@@ -137,8 +137,7 @@
         $this->rotation = 0;
         $this->shadow = new Drawing\Shadow();
 
-        // Set image index
-        ++self::$imageCounter;
+        // Set image index++self::$imageCounter;
         $this->imageIndex = self::$imageCounter;
     }
 

tests/PhpSpreadsheet/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/PhpSpreadsheet/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);