CrossKnowledge / DataTableBundle

DataTable/Table/Element/Column/Column.php

Doc Comments +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
         return $this;
     }
     /**
-     * @return mixed
+     * @return string
      */
     public function getOptions()
     {

Spacing +4 added lines, -4 removed lines

@@ -14,7 +14,7 @@ 
      * @var array
      */
     public static $clientSideColumnOptions = [
-        'cellType','className','contentPadding', 'createdCell', 'data', 'defaultContent', 'name', 'orderable',
+        'cellType', 'className', 'contentPadding', 'createdCell', 'data', 'defaultContent', 'name', 'orderable',
         'orderData', 'orderDataType', 'render', 'searchable', 'title', 'type', 'visible', 'width'
     ];
 
@@ -33,7 +33,7 @@ 
      */
     protected $formatValueCallback;
 
-    public function __construct($title='', $options=[])
+    public function __construct($title = '', $options = [])
     {
         $this->optionsResolver = new OptionsResolver();
         $this->configureOptions($this->optionsResolver);
@@ -92,7 +92,7 @@ 
     {
         if (is_callable($this->formatValueCallback)) {
             return call_user_func_array($this->formatValueCallback, [$value, $rowData]);
-        } else{
+        } else {
             return $value;
         }
     }
@@ -106,7 +106,7 @@ 
     /**
      * @param \Closure $callback
      */
-    public function setFormatValueCallback(\Closure $callback=null)
+    public function setFormatValueCallback(\Closure $callback = null)
     {
         $this->formatValueCallback = $callback;
         return $this;

Tests/UsesContainerTrait.php

Unused Use Statements -2 removed lines

@@ -5,9 +5,7 @@
 use CrossKnowledge\DataTableBundle\DataTable\Table\AbstractTable;
 use CrossKnowledge\DataTableBundle\DependencyInjection\Compiler\DatatablePass;
 use CrossKnowledge\DataTableBundle\DependencyInjection\CrossKnowledgeDataTableExtension;
-use Symfony\Bundle\FrameworkBundle\Controller\Controller;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
-use Symfony\Component\DependencyInjection\Definition;
 
 trait UsesContainerTrait
 {

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 
 trait UsesContainerTrait
 {
-    protected function compileContainer(ContainerBuilder $container, $loadExt=true)
+    protected function compileContainer(ContainerBuilder $container, $loadExt = true)
     {
         if ($loadExt) {
             $extension = new CrossKnowledgeDataTableExtension();

DataTable/Renderer/JsonRenderer.php

Spacing +2 added lines, -2 removed lines

@@ -12,13 +12,13 @@
         $viewVars = $table->buildView();
         $jsonVars = [];
         $jsonVars['recordsTotal'] = $viewVars['unfilteredRowsCount'];
-        if ($viewVars['filteredRowsCount']!==false) {
+        if ($viewVars['filteredRowsCount'] !== false) {
             $jsonVars['recordsFiltered'] = $viewVars['filteredRowsCount'];
         } else {
             $jsonVars['recordsFiltered'] = $viewVars['unfilteredRowsCount'];
         }
         $jsonVars['data'] = array_map(
-            function ($item) {
+            function($item) {
                 $t = [];
                 foreach ($item as $k => $v) {
                     $t[] = $v;

DataTable/Table/AbstractTable.php

Spacing +4 added lines, -4 removed lines

@@ -138,7 +138,7 @@ 
      *
      * @param OptionsResolver $resolver
      */
-    public function configureOptions(OptionsResolver $resolver){}
+    public function configureOptions(OptionsResolver $resolver) {}
     /**
      * @return array
      */
@@ -178,7 +178,7 @@ 
     {
         $t = [];
         foreach ($this->getDataIterator() as $item) {
-            $formatted = $this->formatter->formatRow($item,  $this);
+            $formatted = $this->formatter->formatRow($item, $this);
             $t[] = $formatted;
         }
 
@@ -216,7 +216,7 @@ 
      */
     public function getFilterForm()
     {
-        if (null===$this->filterForm) {
+        if (null === $this->filterForm) {
             $this->filterForm = $this->buildFilterForm(
                 $this->formFactory->createNamedBuilder($this->getTableId().'_filter')
                     ->add('dofilter', 'button')
@@ -280,7 +280,7 @@ 
         $this->buildColumns($builder);
 
         $this->columns = $builder->getColumns();
-        $this->columnsInitialized =  true;
+        $this->columnsInitialized = true;
     }
     /**
      * Sets the table identifier

DataTable/Table/Element/Column/Link.php

Spacing +2 added lines, -2 removed lines

@@ -11,10 +11,10 @@
     {
         parent::configureOptions($resolver);
         $resolver->setRequired('LinkTextField');
-        $resolver->setDefault('AltTextField', function (Options $options) {
+        $resolver->setDefault('AltTextField', function(Options $options) {
             return $options['LinkTextField'];
         });
-        $resolver->setDefault('UrlField', function (Options $options) {
+        $resolver->setDefault('UrlField', function(Options $options) {
             return $options['LinkTextField'];
         });
         $resolver->setDefault('UrlCallback', function($value, $row) {

DataTable/Formatter/DefaultFormatter.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
                 $value = $rawValue;
             }
 
-            $colVal = $column->formatCell($value, $row);//Column definition
+            $colVal = $column->formatCell($value, $row); //Column definition
             $newRow[$colIdentifier] = $colVal;
         }
 

Tests/DataTable/ColumnBuilderTest.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
     {
         $builder = new ColumnBuilder();
 
-        for ($i=0;$i<3;$i++) {
+        for ($i = 0; $i<3; $i++) {
             $builder->add("testcolumn".$i, new Column("test column ".$i));
         }
 

Tests/DataTable/Column/ColumnTest.php

Indentation +2 added lines, -2 removed lines

@@ -28,10 +28,10 @@
     public function testBasicGetterSetter()
     {
         $column = (new Column("test", ['auto_escape' => false]))
-                   ->setFormatValueCallback(function() {
+                    ->setFormatValueCallback(function() {
                         return 'test ok';
                     })
-                   ->setIdentifier('test');
+                    ->setIdentifier('test');
 
         $this->assertEquals("test", $column->getOptions()['title']);
         $this->assertEquals("test ok", call_user_func($column->getFormatValueCallback()));

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
     public function testFormatCellUsesCallback()
     {
         $column = new Column("test");
-        $column->setFormatValueCallback(function($val, $row){
+        $column->setFormatValueCallback(function($val, $row) {
             return $val.'ok';
         });
 

Tests/DataTable/Renderer/TwigRendererTest.php

Indentation +3 added lines, -3 removed lines

@@ -20,11 +20,11 @@
                     ->getMock();
 
         $templateMock = $this->getMockBuilder(\Twig_TemplateInterface::class)
-                             ->disableOriginalConstructor()
-                             ->getMock();
+                                ->disableOriginalConstructor()
+                                ->getMock();
 
         $twig->expects($this->once())->method('loadTemplate')
-             ->will($this->returnValue($templateMock));
+                ->will($this->returnValue($templateMock));
 
         $renderer = new TwigRenderer($twig);
         $renderer->render($mock);