Merge pull request #223 from Arne1303/fix-unauthorized-columns

fix: unauthorized columns being displayed by the builder

Author: yajra

src/Html/Options/HasColumns.php

@@ -92,7 +92,11 @@ public function columns(array $columns): static
 
                 $this->collection->push(new Column($attributes));
             } else {
-                $this->collection->push($value);
+
+                // Only add the column if it is authorized, otherwise ignore it.
+                if ($value->isAuthorized()) {
+                    $this->collection->push($value);
+                }
             }
         }
 

tests/BuilderTest.php

@@ -287,4 +287,19 @@ public function it_has_editors()
         ]);
         $this->assertCount(2, $builder->getEditors());
     }
+
+    #[Test]
+    public function it_ignores_unauthorized_columns(): void
+    {
+        $builder = $this->getHtmlBuilder();
+
+        $builder->columns([
+            Column::makeIf(false)
+                ->title('unauthorized_column'),
+
+            Column::make('authorized_column'),
+        ]);
+
+        $this->assertCount(1, $builder->getColumns());
+    }
 }