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);
@@ -94,7 +94,7 @@ 
     {
         if (is_callable($this->formatValueCallback)) {
             return call_user_func_array($this->formatValueCallback, [$value, $rowData, $context]);
-        } else{
+        } else {
             return $value;
         }
     }
@@ -109,7 +109,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/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) {

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/RegistryTest.php

Indentation +2 added lines, -2 removed lines

@@ -12,8 +12,8 @@
     public function testRetrieveTableById()
     {
         $table = $this->getMockBuilder(AbstractTable::class)
-                      ->disableOriginalConstructor()
-                      ->getMock();
+                        ->disableOriginalConstructor()
+                        ->getMock();
 
         $registry = new DataTableRegistry(['test' => $table]);
         $this->assertEquals($table, $registry->retrieveByTableId('test'));

Tests/DataTable/Request/PaginateRequestTest.php

Indentation +3 added lines, -3 removed lines

@@ -13,9 +13,9 @@
     {
         $tableMock = $this->getTableMock();
         $request = new Request([], [
-           'order' => [['column' => '1', 'dir' => 'desc']],
-           'start'  => 242,
-           'length' => 342
+            'order' => [['column' => '1', 'dir' => 'desc']],
+            'start'  => 242,
+            'length' => 342
         ]);
 
         $paginateRequest = PaginateRequest::fromHttpRequest($request, $tableMock);

Tests/DataTable/Formatter/DefaultFormatterTest.php

Indentation +2 added lines, -2 removed lines

@@ -24,8 +24,8 @@
         $formatter = new DefaultFormatter();
 
         $row = [
-          'escapedcolumn' => '<testval',
-          'unescapedcolumn' => '<testval',
+            'escapedcolumn' => '<testval',
+            'unescapedcolumn' => '<testval',
         ];
 
         $expected = [

Tests/Controller/DataTableControllerTest.php

Indentation +7 added lines, -7 removed lines

@@ -19,19 +19,19 @@
     {
         $tableMock = $this->getDataTableMock();
         $registryMock = $this->getMockBuilder('CrossKnowledge\DataTableBundle\DataTable\DataTableRegistry')
-                             ->disableOriginalConstructor()
-                             ->getMock();
+                                ->disableOriginalConstructor()
+                                ->getMock();
 
         $registryMock->expects($this->once())
-                     ->method('retrieveByTableId')
-                     ->will($this->returnValue($tableMock));
+                        ->method('retrieveByTableId')
+                        ->will($this->returnValue($tableMock));
 
         $rendererMock = $this->getMockBuilder('CrossKnowledge\DataTableBundle\DataTable\Renderer\JsonRenderer')
-                             ->getMock();
+                                ->getMock();
 
         $rendererMock->expects($this->once())
-                         ->method('renderJsonResponse')
-                         ->with($tableMock);
+                            ->method('renderJsonResponse')
+                            ->with($tableMock);
 
         $request = new Request([], [], ['tableid' => 'testtable_id']);