symball / report

Service/ReportBuilder.php

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 
 namespace Symball\ReportBundle\Service;
 
-use Symball\ReportBundle\Service\ReportPattern;
 use Symball\ReportBundle\Service\ReportStyle;
 use Symball\ReportBundle\Service\Meta;
 use Symball\ReportBundle\Interfaces\NavInterface;

Tests/Unit/MetaTest.php

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 
 /* The base PHPUnit test class */
 use PHPUnit\Framework\TestCase;
-
 use Symball\ReportBundle\Service\Meta;
 
 /* Extend the default PHPUnit test case */

Tests/Unit/ReportPatternTest.php

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 
 /* The base PHPUnit test class */
 use PHPUnit\Framework\TestCase;
-
 use Symball\ReportBundle\Service\ReportPattern;
 use Symball\ReportBundle\Service\ReportBuilder;
 use Symball\ReportBundle\Interfaces\PatternInterface;

Spacing +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
         $mockPattern->method('run')->willReturn(true);
         $reportPattern->addPattern($mockPattern, 'an_alias');
 
-        $mockReportBuilder= $this->createMock(ReportBuilder::class);
+        $mockReportBuilder = $this->createMock(ReportBuilder::class);
 
         $result = $reportPattern->run('an_alias', $mockReportBuilder);
 
@@ -54,7 +54,7 @@ 
         $reportPattern = new ReportPattern();
 
         try {
-            $mockReportBuilder= $this->createMock(ReportBuilder::class);
+            $mockReportBuilder = $this->createMock(ReportBuilder::class);
             $result = $reportPattern->run('an_alias', $mockReportBuilder);
         } catch (\Exception $ex) {
             // General exception

Tests/Unit/ReportStyleTest.php

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 
 /* The base PHPUnit test class */
 use PHPUnit\Framework\TestCase;
-
 use Symball\ReportBundle\Service\Meta;
 use Symball\ReportBundle\Service\ReportStyle;
 use Symball\ReportBundle\Service\ReportBuilder;

Spacing +2 added lines, -2 removed lines

@@ -47,7 +47,7 @@ 
         $mockMeta = $this->createMock(Meta::class);
         $mockMeta->method('getOptions')->willReturn([]);
 
-        $mockReportBuilder= $this->createMock(ReportBuilder::class);
+        $mockReportBuilder = $this->createMock(ReportBuilder::class);
         $mockReportBuilder->method('meta')->will($this->returnValue($mockMeta));
 
         $result = $reportStyle->run('an_alias', $mockReportBuilder, 'A1');
@@ -61,7 +61,7 @@ 
         $mockMeta = $this->createMock(Meta::class);
         $mockMeta->method('getOptions')->willReturn([]);
 
-        $mockReportBuilder= $this->createMock(ReportBuilder::class);
+        $mockReportBuilder = $this->createMock(ReportBuilder::class);
         $mockReportBuilder->method('meta')->will($this->returnValue($mockMeta));
 
         try {

DependencyInjection/Configuration.php

Indentation +8 added lines, -8 removed lines

@@ -30,18 +30,18 @@
 
         $rootNode
             ->children()
-              ->scalarNode('excel_factory_namespace')
+                ->scalarNode('excel_factory_namespace')
                 ->defaultValue('\PHPExcel_IOFactory')
-              ->end()
-              ->scalarNode('layout_style')
+                ->end()
+                ->scalarNode('layout_style')
                 ->defaultValue('horizontal')
-              ->end()
-              ->scalarNode('output_format')
+                ->end()
+                ->scalarNode('output_format')
                 ->defaultValue('Excel2007')
-              ->end()
-              ->scalarNode('default_report_path')
+                ->end()
+                ->scalarNode('default_report_path')
                 ->isRequired()
-              ->end()
+                ->end()
             ->end()
         ;
 

Styles/Width.php

Indentation +5 added lines, -5 removed lines

@@ -28,13 +28,13 @@
         switch ($options['type']) {
         case 'auto':
           $context->sheet()
-          ->getColumnDimension($coordString)
-          ->setAutoSize(true);
-          break;
+            ->getColumnDimension($coordString)
+            ->setAutoSize(true);
+            break;
 
         default:
           throw new \Exception('Unrecognised style width option: '.$options['type']);
-          break;
-      }
+            break;
+        }
     }
 }

Switch Indentation +8 added lines, -8 removed lines

@@ -26,15 +26,15 @@
     public function run(ReportBuilder &$context, $coordString, $options) {
         
         switch ($options['type']) {
-        case 'auto':
-          $context->sheet()
-          ->getColumnDimension($coordString)
-          ->setAutoSize(true);
-          break;
+            case 'auto':
+              $context->sheet()
+              ->getColumnDimension($coordString)
+              ->setAutoSize(true);
+              break;
 
-        default:
-          throw new \Exception('Unrecognised style width option: '.$options['type']);
-          break;
+            default:
+              throw new \Exception('Unrecognised style width option: '.$options['type']);
+              break;
       }
     }
 }

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
           break;
 
         default:
-          throw new \Exception('Unrecognised style width option: '.$options['type']);
+          throw new \Exception('Unrecognised style width option: ' . $options['type']);
           break;
       }
     }

Service/ReportQuery.php

Doc Comments -1 removed lines

@@ -40,7 +40,6 @@
      * A front-end for running pattern operations that are loaded on to the stack
      *
      * @param string        $alias
-     * @param ReportBuilder $context
      * @return boolean Success condition
      * @throws \Exception when pattern not available
      */

Tests/Unit/ReportQueryTest.php

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 
 /* The base PHPUnit test class */
 use PHPUnit\Framework\TestCase;
-
 use Symball\ReportBundle\Service\ReportQuery;
 use Symball\ReportBundle\Interfaces\QueryInterface;
 

Tests/Extension/ConfigurationTest.php

Indentation +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
     use ConfigurationTestCaseTrait;
     
     protected function getContainerExtension() {
-       return new SymballReportExtension();
+        return new SymballReportExtension();
     }
     
     protected function getConfiguration()