Fix select plugin with tests

Author: yajra

src/Html/Options/Plugins/Select.php

@@ -12,20 +12,6 @@
  */
 trait Select
 {
-    /**
-     * Set select option value.
-     *
-     * @param  bool|array  $value
-     * @return $this
-     * @see https://datatables.net/reference/option/select
-     */
-    public function select(bool|array $value = true): static
-    {
-        $this->attributes['select'] = $value;
-
-        return $this;
-    }
-
     /**
      * Set select blurable option value.
      *
@@ -35,9 +21,7 @@ public function select(bool|array $value = true): static
      */
     public function selectBlurable(bool $value = true): static
     {
-        $this->attributes['select']['blurable'] = $value;
-
-        return $this;
+        return $this->select(['blurable' => $value]);
     }
 
     /**
@@ -49,9 +33,7 @@ public function selectBlurable(bool $value = true): static
      */
     public function selectClassName(string $value = 'selected'): static
     {
-        $this->attributes['select']['className'] = $value;
-
-        return $this;
+        return $this->select(['className' => $value]);
     }
 
     /**
@@ -80,9 +62,7 @@ public function selectAddClassName(string $class): static
      */
     public function selectInfo(bool $value = true): static
     {
-        $this->attributes['select']['info'] = $value;
-
-        return $this;
+        return $this->select(['info' => $value]);
     }
 
     /**
@@ -94,9 +74,7 @@ public function selectInfo(bool $value = true): static
      */
     public function selectItems(string $value = 'row'): static
     {
-        $this->attributes['select']['items'] = $value;
-
-        return $this;
+        return $this->select(['items' => $value]);
     }
 
     /**
@@ -107,9 +85,7 @@ public function selectItems(string $value = 'row'): static
      */
     public function selectItemsRow(): static
     {
-        $this->attributes['select']['items'] = Builder::SELECT_ITEMS_ROW;
-
-        return $this;
+        return $this->select(['items' => Builder::SELECT_ITEMS_ROW]);
     }
 
     /**
@@ -120,9 +96,7 @@ public function selectItemsRow(): static
      */
     public function selectItemsColumn(): static
     {
-        $this->attributes['select']['items'] = Builder::SELECT_ITEMS_COLUMN;
-
-        return $this;
+        return $this->select(['items' => Builder::SELECT_ITEMS_COLUMN]);
     }
 
     /**
@@ -133,9 +107,7 @@ public function selectItemsColumn(): static
      */
     public function selectItemsCell(): static
     {
-        $this->attributes['select']['items'] = Builder::SELECT_ITEMS_CELL;
-
-        return $this;
+        return $this->select(['items' => Builder::SELECT_ITEMS_CELL]);
     }
 
     /**
@@ -147,9 +119,7 @@ public function selectItemsCell(): static
      */
     public function selectSelector(string $value = 'td'): static
     {
-        $this->attributes['select']['selector'] = $value;
-
-        return $this;
+        return $this->select(['selector' => $value]);
     }
 
     /**
@@ -161,9 +131,7 @@ public function selectSelector(string $value = 'td'): static
      */
     public function selectStyle(string $value = 'os'): static
     {
-        $this->attributes['select']['style'] = $value;
-
-        return $this;
+        return $this->select(['style' => $value]);
     }
 
     /**
@@ -174,9 +142,7 @@ public function selectStyle(string $value = 'os'): static
      */
     public function selectStyleApi(): static
     {
-        $this->attributes['select']['style'] = Builder::SELECT_STYLE_API;
-
-        return $this;
+        return $this->select(['style' => Builder::SELECT_STYLE_API]);
     }
 
     /**
@@ -187,7 +153,23 @@ public function selectStyleApi(): static
      */
     public function selectStyleSingle(): static
     {
-        $this->attributes['select']['style'] = Builder::SELECT_STYLE_SINGLE;
+        return $this->select(['style' => Builder::SELECT_STYLE_SINGLE]);
+    }
+
+    /**
+     * Set select option value.
+     *
+     * @param  bool|array  $value
+     * @return $this
+     * @see https://datatables.net/reference/option/select
+     */
+    public function select(bool|array $value = true): static
+    {
+        if (is_array($value)) {
+            $this->attributes['select'] = array_merge((array) $this->attributes['select'], $value);
+        } else {
+            $this->attributes['select'] = $value;
+        }
 
         return $this;
     }
@@ -200,9 +182,7 @@ public function selectStyleSingle(): static
      */
     public function selectStyleMulti(): static
     {
-        $this->attributes['select']['style'] = Builder::SELECT_STYLE_MULTI;
-
-        return $this;
+        return $this->select(['style' => Builder::SELECT_STYLE_MULTI]);
     }
 
     /**
@@ -213,9 +193,7 @@ public function selectStyleMulti(): static
      */
     public function selectStyleOS(): static
     {
-        $this->attributes['select']['style'] = Builder::SELECT_STYLE_OS;
-
-        return $this;
+        return $this->select(['style' => Builder::SELECT_STYLE_OS]);
     }
 
     /**
@@ -226,8 +204,19 @@ public function selectStyleOS(): static
      */
     public function selectStyleMultiShift(): static
     {
-        $this->attributes['select']['style'] = Builder::SELECT_STYLE_MULTI_SHIFT;
+        return $this->select(['style' => Builder::SELECT_STYLE_MULTI_SHIFT]);
+    }
 
-        return $this;
+    /**
+     * @param  string|null  $key
+     * @return mixed
+     */
+    public function getSelect(string $key = null): mixed
+    {
+        if (is_null($key)) {
+            return $this->attributes['select'] ?? true;
+        }
+
+        return $this->attributes['select'][$key] ?? false;
     }
 }

tests/BuilderOptionsPluginsTest.php

@@ -2,6 +2,7 @@
 
 namespace Yajra\DataTables\Html\Tests;
 
+use Yajra\DataTables\Html\Builder;
 use Yajra\DataTables\Html\Button;
 use Yajra\DataTables\Html\SearchPane;
 
@@ -279,4 +280,56 @@ public function it_has_search_panes_plugin()
         $builder->searchPanes(SearchPane::make()->show()->cascadePanes());
         $this->assertEquals(['show' => true, 'cascadePanes' => true], $builder->getAttribute('searchPanes'));
     }
+
+    /** @test */
+    public function it_has_select_plugin()
+    {
+        $builder = $this->getHtmlBuilder();
+        $builder->select();
+
+        $this->assertTrue($builder->getAttribute('select'));
+        $this->assertTrue($builder->getSelect());
+
+        $builder->selectBlurable()
+                ->selectClassName()
+                ->selectInfo()
+                ->selectItems()
+                ->selectSelector()
+                ->selectStyle();
+
+        $this->assertEquals(true, $builder->getSelect('blurable'));
+        $this->assertEquals('selected', $builder->getSelect('className'));
+        $this->assertEquals(true, $builder->getSelect('info'));
+        $this->assertEquals('row', $builder->getSelect('items'));
+        $this->assertEquals('td', $builder->getSelect('selector'));
+        $this->assertEquals('os', $builder->getSelect('style'));
+
+        $builder->selectAddClassName('test');
+        $this->assertEquals('selected test', $builder->getSelect('className'));
+
+        $builder->selectItemsRow();
+        $this->assertEquals(Builder::SELECT_ITEMS_ROW, $builder->getSelect('items'));
+
+        $builder->selectItemsColumn();
+        $this->assertEquals(Builder::SELECT_ITEMS_COLUMN, $builder->getSelect('items'));
+
+        $builder->selectItemsCell();
+        $this->assertEquals(Builder::SELECT_ITEMS_CELL, $builder->getSelect('items'));
+
+        $builder->selectStyleSingle();
+        $this->assertEquals(Builder::SELECT_STYLE_SINGLE, $builder->getSelect('style'));
+
+        $builder->selectStyleMulti();
+        $this->assertEquals(Builder::SELECT_STYLE_MULTI, $builder->getSelect('style'));
+
+        $builder->selectStyleOS();
+        $this->assertEquals(Builder::SELECT_STYLE_OS, $builder->getSelect('style'));
+
+        $builder->selectStyleMultiShift();
+        $this->assertEquals(Builder::SELECT_STYLE_MULTI_SHIFT, $builder->getSelect('style'));
+
+        $builder->selectStyleApi();
+        $this->assertEquals(Builder::SELECT_STYLE_API, $builder->getSelect('style'));
+    }
+
 }