fix: DataTableBuilder hassers utilizing "isset" instead of "array_key…

…_exists"
Kreyu · Feb 5, 2024 · 8dfe3c6 · 8dfe3c6
1 parent 1d3539d
commit 8dfe3c6
Showing 1 changed file with 6 additions and 12 deletions.
diff --git a/src/DataTableBuilder.php b/src/DataTableBuilder.php
@@ -225,8 +225,7 @@ public function hasColumn(string $name): bool
             throw $this->createBuilderLockedException();
         }
 
-        return array_key_exists($name, $this->columns)
-            || array_key_exists($name, $this->unresolvedColumns);
+        return isset($this->columns[$name]) || isset($this->unresolvedColumns[$name]);
     }
 
     public function removeColumn(string $name): static
@@ -294,8 +293,7 @@ public function hasFilter(string $name): bool
             throw $this->createBuilderLockedException();
         }
 
-        return array_key_exists($name, $this->filters)
-            || array_key_exists($name, $this->unresolvedFilters);
+        return isset($this->filters[$name]) || isset($this->unresolvedFilters[$name]);
     }
 
     public function removeFilter(string $name): static
@@ -403,8 +401,7 @@ public function hasAction(string $name): bool
             throw $this->createBuilderLockedException();
         }
 
-        return array_key_exists($name, $this->actions)
-            || array_key_exists($name, $this->unresolvedActions);
+        return isset($this->actions[$name]) || isset($this->unresolvedActions[$name]);
     }
 
     public function removeAction(string $name): static
@@ -452,8 +449,7 @@ public function hasBatchAction(string $name): bool
             throw $this->createBuilderLockedException();
         }
 
-        return array_key_exists($name, $this->batchActions)
-            || array_key_exists($name, $this->unresolvedBatchActions);
+        return isset($this->batchActions[$name]) || isset($this->unresolvedBatchActions[$name]);
     }
 
     public function addBatchAction(string|ActionBuilderInterface $action, string $type = null, array $options = []): static
@@ -541,8 +537,7 @@ public function hasRowAction(string $name): bool
             throw $this->createBuilderLockedException();
         }
 
-        return array_key_exists($name, $this->rowActions)
-            || array_key_exists($name, $this->unresolvedRowActions);
+        return isset($this->rowActions[$name]) || isset($this->unresolvedRowActions[$name]);
     }
 
     public function addRowAction(string|ActionBuilderInterface $action, string $type = null, array $options = []): static
@@ -650,8 +645,7 @@ public function hasExporter(string $name): bool
             throw $this->createBuilderLockedException();
         }
 
-        return array_key_exists($name, $this->exporters)
-            || array_key_exists($name, $this->unresolvedExporters);
+        return isset($this->exporters[$name]) || isset($this->unresolvedExporters[$name]);
     }
 
     public function removeExporter(string $name): static