Apply more CS rules

Author: sanmai

.php-cs-fixer.dist.php

@@ -7,7 +7,6 @@
         'array_syntax' => false,
         'concat_space' => false,
         'blank_line_after_opening_tag' => false,
-        'visibility_required' => false,
         'trailing_comma_in_multiline' => false,
         'method_argument_space' => false,
         'array_indentation' => false,
@@ -16,10 +15,8 @@
         'statement_indentation' => false,
         'binary_operator_spaces' => false,
         'single_blank_line_at_eof' => false,
-        'elseif' => false,
         'control_structure_braces' => false,
         'control_structure_continuation_position' => false,
-        'no_multiple_statements_per_line' => false,
     ])
     ->setFinder(
         PhpCsFixer\Finder::create()

Spreadsheet/Excel/Writer.php

@@ -83,7 +83,7 @@ public function send($filename)
     * @param integer $col Column for the cell to convert (0-indexed).
     * @return string The cell identifier in A1 format
     */
-    function rowcolToCell($row, $col)
+    public function rowcolToCell($row, $col)
     {
         if ($col > 255) { //maximum column value exceeded
             return new PEAR_Error("Maximum column value exceeded: $col");

Spreadsheet/Excel/Writer/Workbook.php

@@ -1414,7 +1414,7 @@ protected function _calculateSharedStringsSizes()
                             $align = 1;
                         }
                         // Split section without header => split on even boundary
-                        else if ($split_string && $space_remaining % 2 == 1) {
+                        elseif ($split_string && $space_remaining % 2 == 1) {
                             $space_remaining--;
                             $align = 1;
                         }
@@ -1587,7 +1587,7 @@ protected function _storeSharedStringsTable()
                             $align = 1;
                         }
                         // Split section without header => split on even boundary
-                        else if ($split_string && $space_remaining % 2 == 1) {
+                        elseif ($split_string && $space_remaining % 2 == 1) {
                             $space_remaining--;
                             $align = 1;
                         }

Spreadsheet/Excel/Writer/Worksheet.php

@@ -878,7 +878,8 @@ public function setColumn($firstcol, $lastcol, $width, $format = null, $hidden =
         // look for any ranges this might overlap and remove, size or split where necessary
         foreach ($this->_colinfo as $key => $colinfo)
         {
-            $existing_start = $colinfo[0]; $existing_end = $colinfo[1];
+            $existing_start = $colinfo[0];
+            $existing_end = $colinfo[1];
             // if the new range starts within another range
             if ($firstcol > $existing_start && $firstcol < $existing_end)
             { // trim the existing range to the beginning of the new range

test/Test/Spreadsheet/Excel/WriterTestCase.php

@@ -6,7 +6,7 @@
  */
 class Test_Spreadsheet_Excel_WriterTestCase extends \LegacyPHPUnit\TestCase
 {
-    const FIXTURES_PATH = 'test/fixture/';
+    public const FIXTURES_PATH = 'test/fixture/';
 
     /**
      * @param string $filename