PHPOffice / PhpSpreadsheet

tests/PhpSpreadsheetTests/Writer/Xls/WorkbookTest.php

Unused Use Statements +3 added lines, -3 removed lines

@@ -8,9 +8,9 @@
 

 namespace PhpOffice\PhpSpreadsheetTests\Writer\Xls\Workbook;

 

-use PhpOffice\PhpSpreadsheet\Spreadsheet;

-use PhpOffice\PhpSpreadsheet\Writer\Xls\Parser;

-use PhpOffice\PhpSpreadsheet\Writer\Xls\Workbook;

+use PhpOffice\PhpSpreadsheet\Spreadsheet;
+use PhpOffice\PhpSpreadsheet\Writer\Xls\Parser;
+use PhpOffice\PhpSpreadsheet\Writer\Xls\Workbook;
 use PHPUnit_Framework_TestCase;

 

 class WorkbookTest extends PHPUnit_Framework_TestCase

Spacing +11 added lines, -11 removed lines

@@ -19,7 +19,7 @@ 
     private $workbook;

 

 

-    protected function setUp(){

+    protected function setUp() {

         $spreadsheet = new Spreadsheet();

         $strTotal = 0;

         $strUnique = 0;

@@ -27,7 +27,7 @@ 
         $colors = [];

         $parser = new Parser();

 

-        $this->workbook = new Workbook($spreadsheet, $strTotal, $strUnique, $str_table, $colors,$parser);

+        $this->workbook = new Workbook($spreadsheet, $strTotal, $strUnique, $str_table, $colors, $parser);

     }

 

     protected function tearDown()

@@ -40,7 +40,7 @@ 
      *

      * @param mixed $expectedResult

      */

-    public function testAddColor($testColors,$expectedResult){

+    public function testAddColor($testColors, $expectedResult) {

         $result = 0;

 

         $workbookReflection = new \ReflectionClass(Workbook::class);

@@ -49,7 +49,7 @@ 
         $methodAddColor->setAccessible(TRUE);

         $propertyPalette->setAccessible(TRUE);

 

-        foreach($testColors as $testColor){

+        foreach ($testColors as $testColor) {

             $result = $methodAddColor->invoke($this->workbook, $testColor);

         }

 

@@ -58,7 +58,7 @@ 
         $this->assertEquals($expectedResult, $palette);

     }

 

-    public function providerAddColor(){

+    public function providerAddColor() {

         $this->setUp();

 

         $workbookReflection = new \ReflectionClass(Workbook::class);

@@ -128,10 +128,10 @@ 
      * @param array $palette palette color

      * @return string rgb string

      */

-    public static function paletteToColor($palette){

-        return self::right('00'.dechex(intval($palette[0])),2)

-            .self::right('00'.dechex(intval($palette[1])),2)

-            .self::right('00'.dechex(intval($palette[2])),2);

+    public static function paletteToColor($palette) {

+        return self::right('00' . dechex(intval($palette[0])), 2)

+            .self::right('00' . dechex(intval($palette[1])), 2)

+            .self::right('00' . dechex(intval($palette[2])), 2);

     }

 

     /**

@@ -141,8 +141,8 @@ 
      * @param integer $nbchar number of char at right of string

      * @return string

      */

-    public static function right($value,$nbchar)

+    public static function right($value, $nbchar)

     {

-            return mb_substr($value,mb_strlen($value)-$nbchar, $nbchar);

+            return mb_substr($value, mb_strlen($value) - $nbchar, $nbchar);

     }

 }

src/PhpSpreadsheet/Writer/Xls/Workbook.php

Spacing +2 added lines, -2 removed lines

@@ -343,9 +343,9 @@
             if ($colorIndex) {
                 $this->colors[$rgb] = $colorIndex;
             } else {
-                if(count($this->colors) == 0){
+                if (count($this->colors) == 0) {
                     $lastColor = 7;
-                }else{
+                } else {
                     $lastColor = end($this->colors);
                 }
                 if ($lastColor < 57) {

Braces +1 added lines, -1 removed lines

@@ -345,7 +345,7 @@
             } else {
                 if(count($this->colors) == 0){
                     $lastColor = 7;
-                }else{
+                } else{
                     $lastColor = end($this->colors);
                 }
                 if ($lastColor < 57) {