Prevent editColumn when column is not shown

Author: Arkhas

src/DataTableAbstract.php

@@ -231,7 +231,9 @@ public function addIndexColumn(): static
      */
     public function editColumn($name, $content): static
     {
-        $this->columnDef['edit'][] = ['name' => $name, 'content' => $content];
+        if (!count($this->request->columns()) || in_array($name, Arr::pluck($this->request->columns(), 'name'))) {
+            $this->columnDef['edit'][] = ['name' => $name, 'content' => $content];
+        }
 
         return $this;
     }

tests/Integration/QueryDataTableTest.php

@@ -188,6 +188,19 @@ public function it_returns_only_the_selected_columns()
         $this->assertArrayHasKey('name', $json['data'][0]);
     }
 
+        /** @test */
+    public function it_edit_only_the_selected_columns()
+    {
+        $json = $this->call('GET', '/query/edit-columns', [
+            'columns' => [
+                ['data' => 'name', 'name' => 'name', 'searchable' => 'true', 'orderable' => 'true'],
+            ],
+        ])->json();
+
+        $this->assertEquals('edited', $json['data'][0]['name']);
+        $this->assertNotEquals('edited', $json['data'][0]['email']);
+    }
+
     /** @test */
     public function it_does_not_allow_raw_html_on_added_columns()
     {
@@ -417,6 +430,17 @@ protected function setUp(): void
                              ->toJson();
         });
 
+        $router->get('/query/edit-columns', function (DataTables $dataTable) {
+            return $dataTable->query(DB::table('users'))
+                             ->editColumn('name', function () {
+                                 return 'edited';
+                             })
+                             ->editColumn('email', function () {
+                                 return 'edited';
+                             })
+                             ->toJson();
+        });
+
         $router->get('/query/xss-add', function (DataTables $dataTable) {
             return $dataTable->query(DB::table('users'))
                              ->addColumn('foo', '<a href="#">Allowed</a>')