PHPOffice / PhpSpreadsheet

tests/PhpSpreadsheet/Shared/CodePageTest.php

Spacing +3 added lines, -3 removed lines

@@ -14,7 +14,7 @@ 
     {
         $args = func_get_args();
         $expectedResult = array_pop($args);
-        $result = call_user_func_array(array(CodePage::class,'numberToName'), $args);
+        $result = call_user_func_array(array(CodePage::class, 'numberToName'), $args);
         $this->assertEquals($expectedResult, $result);
     }
 
@@ -27,7 +27,7 @@ 
     {
         $invalidCodePage = 12345;
         try {
-            $result = call_user_func(array(CodePage::class,'numberToName'), $invalidCodePage);
+            $result = call_user_func(array(CodePage::class, 'numberToName'), $invalidCodePage);
         } catch (Exception $e) {
             $this->assertEquals($e->getMessage(), 'Unknown codepage: 12345');
             return;
@@ -39,7 +39,7 @@ 
     {
         $unsupportedCodePage = 720;
         try {
-            $result = call_user_func(array(CodePage::class,'numberToName'), $unsupportedCodePage);
+            $result = call_user_func(array(CodePage::class, 'numberToName'), $unsupportedCodePage);
         } catch (Exception $e) {
             $this->assertEquals($e->getMessage(), 'Code page 720 not supported.');
             return;

tests/PhpSpreadsheet/Shared/StringTest.php

Spacing +13 added lines, -13 removed lines

@@ -11,18 +11,18 @@ 
         parent::setUp();
 
         // Reset Currency Code
-        call_user_func(array(StringHelper::class,'setCurrencyCode'), null);
+        call_user_func(array(StringHelper::class, 'setCurrencyCode'), null);
     }
 
     public function testGetIsMbStringEnabled()
     {
-        $result = call_user_func(array(StringHelper::class,'getIsMbstringEnabled'));
+        $result = call_user_func(array(StringHelper::class, 'getIsMbstringEnabled'));
         $this->assertTrue($result);
     }
 
     public function testGetIsIconvEnabled()
     {
-        $result = call_user_func(array(StringHelper::class,'getIsIconvEnabled'));
+        $result = call_user_func(array(StringHelper::class, 'getIsIconvEnabled'));
         $this->assertTrue($result);
     }
 
@@ -31,16 +31,16 @@ 
         $localeconv = localeconv();
 
         $expectedResult = (!empty($localeconv['decimal_point'])) ? $localeconv['decimal_point'] : ',';
-        $result = call_user_func(array(StringHelper::class,'getDecimalSeparator'));
+        $result = call_user_func(array(StringHelper::class, 'getDecimalSeparator'));
         $this->assertEquals($expectedResult, $result);
     }
 
     public function testSetDecimalSeparator()
     {
         $expectedResult = ',';
-        call_user_func(array(StringHelper::class,'setDecimalSeparator'), $expectedResult);
+        call_user_func(array(StringHelper::class, 'setDecimalSeparator'), $expectedResult);
 
-        $result = call_user_func(array(StringHelper::class,'getDecimalSeparator'));
+        $result = call_user_func(array(StringHelper::class, 'getDecimalSeparator'));
         $this->assertEquals($expectedResult, $result);
     }
 
@@ -49,33 +49,33 @@ 
         $localeconv = localeconv();
 
         $expectedResult = (!empty($localeconv['thousands_sep'])) ? $localeconv['thousands_sep'] : ',';
-        $result = call_user_func(array(StringHelper::class,'getThousandsSeparator'));
+        $result = call_user_func(array(StringHelper::class, 'getThousandsSeparator'));
         $this->assertEquals($expectedResult, $result);
     }
 
     public function testSetThousandsSeparator()
     {
         $expectedResult = ' ';
-        call_user_func(array(StringHelper::class,'setThousandsSeparator'), $expectedResult);
+        call_user_func(array(StringHelper::class, 'setThousandsSeparator'), $expectedResult);
 
-        $result = call_user_func(array(StringHelper::class,'getThousandsSeparator'));
+        $result = call_user_func(array(StringHelper::class, 'getThousandsSeparator'));
         $this->assertEquals($expectedResult, $result);
     }
 
     public function testGetCurrencyCode()
     {
         $localeconv = localeconv();
-        $expectedResult = (!empty($localeconv['currency_symbol']) ? $localeconv['currency_symbol'] : (!empty($localeconv['int_curr_symbol']) ? $localeconv['int_curr_symbol']: '$'));
-        $result = call_user_func(array(StringHelper::class,'getCurrencyCode'));
+        $expectedResult = (!empty($localeconv['currency_symbol']) ? $localeconv['currency_symbol'] : (!empty($localeconv['int_curr_symbol']) ? $localeconv['int_curr_symbol'] : '$'));
+        $result = call_user_func(array(StringHelper::class, 'getCurrencyCode'));
         $this->assertEquals($expectedResult, $result);
     }
 
     public function testSetCurrencyCode()
     {
         $expectedResult = '£';
-        call_user_func(array(StringHelper::class,'setCurrencyCode'), $expectedResult);
+        call_user_func(array(StringHelper::class, 'setCurrencyCode'), $expectedResult);
 
-        $result = call_user_func(array(StringHelper::class,'getCurrencyCode'));
+        $result = call_user_func(array(StringHelper::class, 'getCurrencyCode'));
         $this->assertEquals($expectedResult, $result);
     }
 }

tests/PhpSpreadsheet/Shared/TimeZoneTest.php

Spacing +2 added lines, -2 removed lines

@@ -17,7 +17,7 @@ 
         );
 
         foreach ($timezoneValues as $timezoneValue) {
-            $result = call_user_func(array(TimeZone::class,'setTimezone'), $timezoneValue);
+            $result = call_user_func(array(TimeZone::class, 'setTimezone'), $timezoneValue);
             $this->assertTrue($result);
         }
     }
@@ -25,7 +25,7 @@ 
     public function testSetTimezoneWithInvalidValue()
     {
         $unsupportedTimezone = 'Etc/GMT+10';
-        $result = call_user_func(array(TimeZone::class,'setTimezone'), $unsupportedTimezone);
+        $result = call_user_func(array(TimeZone::class, 'setTimezone'), $unsupportedTimezone);
         $this->assertFalse($result);
     }
 }

tests/PhpSpreadsheet/CellTest.php

Spacing +20 added lines, -20 removed lines

@@ -14,7 +14,7 @@ 
     {
         $args = func_get_args();
         $expectedResult = array_pop($args);
-        $result = call_user_func_array(array(Cell::class,'columnIndexFromString'), $args);
+        $result = call_user_func_array(array(Cell::class, 'columnIndexFromString'), $args);
         $this->assertEquals($expectedResult, $result);
     }
 
@@ -27,7 +27,7 @@ 
     {
         $cellAddress = 'ABCD';
         try {
-            $result = call_user_func(array(Cell::class,'columnIndexFromString'), $cellAddress);
+            $result = call_user_func(array(Cell::class, 'columnIndexFromString'), $cellAddress);
         } catch (\Exception $e) {
             $this->assertInstanceOf(Exception::class, $e);
             $this->assertEquals($e->getMessage(), 'Column string index can not be longer than 3 characters');
@@ -40,7 +40,7 @@ 
     {
         $cellAddress = '';
         try {
-            $result = call_user_func(array(Cell::class,'columnIndexFromString'), $cellAddress);
+            $result = call_user_func(array(Cell::class, 'columnIndexFromString'), $cellAddress);
         } catch (\Exception $e) {
             $this->assertInstanceOf(Exception::class, $e);
             $this->assertEquals($e->getMessage(), 'Column string index can not be empty');
@@ -56,7 +56,7 @@ 
     {
         $args = func_get_args();
         $expectedResult = array_pop($args);
-        $result = call_user_func_array(array(Cell::class,'stringFromColumnIndex'), $args);
+        $result = call_user_func_array(array(Cell::class, 'stringFromColumnIndex'), $args);
         $this->assertEquals($expectedResult, $result);
     }
 
@@ -72,7 +72,7 @@ 
     {
         $args = func_get_args();
         $expectedResult = array_pop($args);
-        $result = call_user_func_array(array(Cell::class,'coordinateFromString'), $args);
+        $result = call_user_func_array(array(Cell::class, 'coordinateFromString'), $args);
         $this->assertEquals($expectedResult, $result);
     }
 
@@ -85,7 +85,7 @@ 
     {
         $cellAddress = 'A1:AI2012';
         try {
-            $result = call_user_func(array(Cell::class,'coordinateFromString'), $cellAddress);
+            $result = call_user_func(array(Cell::class, 'coordinateFromString'), $cellAddress);
         } catch (\Exception $e) {
             $this->assertInstanceOf(Exception::class, $e);
             $this->assertEquals($e->getMessage(), 'Cell coordinate string can not be a range of cells');
@@ -98,7 +98,7 @@ 
     {
         $cellAddress = '';
         try {
-            $result = call_user_func(array(Cell::class,'coordinateFromString'), $cellAddress);
+            $result = call_user_func(array(Cell::class, 'coordinateFromString'), $cellAddress);
         } catch (\Exception $e) {
             $this->assertInstanceOf(Exception::class, $e);
             $this->assertEquals($e->getMessage(), 'Cell coordinate can not be zero-length string');
@@ -111,10 +111,10 @@ 
     {
         $cellAddress = 'AI';
         try {
-            $result = call_user_func(array(Cell::class,'coordinateFromString'), $cellAddress);
+            $result = call_user_func(array(Cell::class, 'coordinateFromString'), $cellAddress);
         } catch (\Exception $e) {
             $this->assertInstanceOf(Exception::class, $e);
-            $this->assertEquals($e->getMessage(), 'Invalid cell coordinate '.$cellAddress);
+            $this->assertEquals($e->getMessage(), 'Invalid cell coordinate ' . $cellAddress);
             return;
         }
         $this->fail('An expected exception has not been raised.');
@@ -127,7 +127,7 @@ 
     {
         $args = func_get_args();
         $expectedResult = array_pop($args);
-        $result = call_user_func_array(array(Cell::class,'absoluteCoordinate'), $args);
+        $result = call_user_func_array(array(Cell::class, 'absoluteCoordinate'), $args);
         $this->assertEquals($expectedResult, $result);
     }
 
@@ -140,7 +140,7 @@ 
     {
         $cellAddress = 'A1:AI2012';
         try {
-            $result = call_user_func(array(Cell::class,'absoluteCoordinate'), $cellAddress);
+            $result = call_user_func(array(Cell::class, 'absoluteCoordinate'), $cellAddress);
         } catch (\Exception $e) {
             $this->assertInstanceOf(Exception::class, $e);
             $this->assertEquals($e->getMessage(), 'Cell coordinate string can not be a range of cells');
@@ -156,7 +156,7 @@ 
     {
         $args = func_get_args();
         $expectedResult = array_pop($args);
-        $result = call_user_func_array(array(Cell::class,'absoluteReference'), $args);
+        $result = call_user_func_array(array(Cell::class, 'absoluteReference'), $args);
         $this->assertEquals($expectedResult, $result);
     }
 
@@ -169,7 +169,7 @@ 
     {
         $cellAddress = 'A1:AI2012';
         try {
-            $result = call_user_func(array(Cell::class,'absoluteReference'), $cellAddress);
+            $result = call_user_func(array(Cell::class, 'absoluteReference'), $cellAddress);
         } catch (\Exception $e) {
             $this->assertInstanceOf(Exception::class, $e);
             $this->assertEquals($e->getMessage(), 'Cell coordinate string can not be a range of cells');
@@ -185,7 +185,7 @@ 
     {
         $args = func_get_args();
         $expectedResult = array_pop($args);
-        $result = call_user_func_array(array(Cell::class,'splitRange'), $args);
+        $result = call_user_func_array(array(Cell::class, 'splitRange'), $args);
         foreach ($result as $key => $split) {
             if (!is_array($expectedResult[$key])) {
                 $this->assertEquals($expectedResult[$key], $split[0]);
@@ -207,7 +207,7 @@ 
     {
         $args = func_get_args();
         $expectedResult = array_pop($args);
-        $result = call_user_func_array(array(Cell::class,'buildRange'), $args);
+        $result = call_user_func_array(array(Cell::class, 'buildRange'), $args);
         $this->assertEquals($expectedResult, $result);
     }
 
@@ -220,7 +220,7 @@ 
     {
         $cellRange = '';
         try {
-            $result = call_user_func(array(Cell::class,'buildRange'), $cellRange);
+            $result = call_user_func(array(Cell::class, 'buildRange'), $cellRange);
         } catch (\Exception $e) {
             $this->assertInstanceOf(Exception::class, $e);
             $this->assertEquals($e->getMessage(), 'Range does not contain any information');
@@ -236,7 +236,7 @@ 
     {
         $args = func_get_args();
         $expectedResult = array_pop($args);
-        $result = call_user_func_array(array(Cell::class,'rangeBoundaries'), $args);
+        $result = call_user_func_array(array(Cell::class, 'rangeBoundaries'), $args);
         $this->assertEquals($expectedResult, $result);
     }
 
@@ -252,7 +252,7 @@ 
     {
         $args = func_get_args();
         $expectedResult = array_pop($args);
-        $result = call_user_func_array(array(Cell::class,'rangeDimension'), $args);
+        $result = call_user_func_array(array(Cell::class, 'rangeDimension'), $args);
         $this->assertEquals($expectedResult, $result);
     }
 
@@ -268,7 +268,7 @@ 
     {
         $args = func_get_args();
         $expectedResult = array_pop($args);
-        $result = call_user_func_array(array(Cell::class,'getRangeBoundaries'), $args);
+        $result = call_user_func_array(array(Cell::class, 'getRangeBoundaries'), $args);
         $this->assertEquals($expectedResult, $result);
     }
 
@@ -284,7 +284,7 @@ 
     {
         $args = func_get_args();
         $expectedResult = array_pop($args);
-        $result = call_user_func_array(array(Cell::class,'extractAllCellReferencesInRange'), $args);
+        $result = call_user_func_array(array(Cell::class, 'extractAllCellReferencesInRange'), $args);
         $this->assertEquals($expectedResult, $result);
     }
 

tests/PhpSpreadsheet/Worksheet/ColumnCellIteratorTest.php

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
         $iterator->seek(4);
         $this->assertEquals($columnIndexResult, $iterator->key());
 
-        for ($i = 1; $i < $columnIndexResult-1; $i++) {
+        for ($i = 1; $i < $columnIndexResult - 1; $i++) {
             $iterator->prev();
             $this->assertEquals($columnIndexResult - $i, $iterator->key());
         }

tests/PhpSpreadsheet/Worksheet/AutoFilter/ColumnTest.php

Spacing +4 added lines, -4 removed lines

@@ -109,7 +109,7 @@ 
 
     public function testSetAttributes()
     {
-        $attributeSet = array(    'val' => 100,
+        $attributeSet = array('val' => 100,
                                 'maxVal' => 200
                              );
 
@@ -120,7 +120,7 @@ 
 
     public function testGetAttributes()
     {
-        $attributeSet = array(    'val' => 100,
+        $attributeSet = array('val' => 100,
                                 'maxVal' => 200
                              );
 
@@ -133,7 +133,7 @@ 
 
     public function testSetAttribute()
     {
-        $attributeSet = array(    'val' => 100,
+        $attributeSet = array('val' => 100,
                                 'maxVal' => 200
                              );
 
@@ -146,7 +146,7 @@ 
 
     public function testGetAttribute()
     {
-        $attributeSet = array(    'val' => 100,
+        $attributeSet = array('val' => 100,
                                 'maxVal' => 200
                              );
 

tests/PhpSpreadsheet/Worksheet/RowIteratorTest.php

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
         $iterator->seek(4);
         $this->assertEquals($columnIndexResult, $iterator->key());
 
-        for ($i = 1; $i < $columnIndexResult-1; $i++) {
+        for ($i = 1; $i < $columnIndexResult - 1; $i++) {
             $iterator->prev();
             $this->assertEquals($columnIndexResult - $i, $iterator->key());
         }

tests/PhpSpreadsheet/Worksheet/AutoFilterTest.php

Spacing +6 added lines, -6 removed lines

@@ -203,7 +203,7 @@ 
 
     public function testGetColumns()
     {
-        $columnIndexes = ['L','M'];
+        $columnIndexes = ['L', 'M'];
 
         foreach ($columnIndexes as $columnIndex) {
             $this->testAutoFilterObject->setColumn($columnIndex);
@@ -222,7 +222,7 @@ 
 
     public function testGetColumn()
     {
-        $columnIndexes = array('L','M');
+        $columnIndexes = array('L', 'M');
 
         foreach ($columnIndexes as $columnIndex) {
             $this->testAutoFilterObject->setColumn($columnIndex);
@@ -275,7 +275,7 @@ 
     {
         $expectedResult = '';
 
-        $columnIndexes = ['L','M','N'];
+        $columnIndexes = ['L', 'M', 'N'];
         foreach ($columnIndexes as $columnIndex) {
             $this->testAutoFilterObject->setColumn($columnIndex);
         }
@@ -299,12 +299,12 @@ 
         $expectedResult = 'G1:J512';
 
         //  These columns should be retained
-        $columnIndexes1 = ['I','J'];
+        $columnIndexes1 = ['I', 'J'];
         foreach ($columnIndexes1 as $columnIndex) {
             $this->testAutoFilterObject->setColumn($columnIndex);
         }
         //  These columns should be discarded
-        $columnIndexes2 = ['K','L','M'];
+        $columnIndexes2 = ['K', 'L', 'M'];
         foreach ($columnIndexes2 as $columnIndex) {
             $this->testAutoFilterObject->setColumn($columnIndex);
         }
@@ -326,7 +326,7 @@ 
 
     public function testClone()
     {
-        $columnIndexes = ['L','M'];
+        $columnIndexes = ['L', 'M'];
 
         foreach ($columnIndexes as $columnIndex) {
             $this->testAutoFilterObject->setColumn($columnIndex);

tests/PhpSpreadsheet/Cell/DefaultValueBinderTest.php

Spacing +2 added lines, -2 removed lines

@@ -66,7 +66,7 @@ 
     public function testDataTypeForValue()
     {
         list($args, $expectedResult) = func_get_args();
-        $result = call_user_func_array(array(DefaultValueBinder::class,'dataTypeForValue'), $args);
+        $result = call_user_func_array(array(DefaultValueBinder::class, 'dataTypeForValue'), $args);
         $this->assertEquals($expectedResult, $result);
     }
 
@@ -81,7 +81,7 @@ 
         $objRichText->createText('Hello World');
 
         $expectedResult = DataType::TYPE_INLINE;
-        $result = call_user_func(array(DefaultValueBinder::class,'dataTypeForValue'), $objRichText);
+        $result = call_user_func(array(DefaultValueBinder::class, 'dataTypeForValue'), $objRichText);
         $this->assertEquals($expectedResult, $result);
     }
 }