psalm

Author: dhirtzbruch

src/ColumnSetting.php

@@ -30,7 +30,7 @@ class ColumnSetting
     private ?array $headerStyle;
     private ?array $dataStyle;
 
-    /** @var callable|string name of the get method (like getId) or a callable taking the object an argument */
+    /** @var string|callable|null name of the get method (like getId) or a callable taking the object an argument */
     private $getter;
 
 
@@ -109,19 +109,12 @@ public function setFormat(string $format): ColumnSetting
         return $this;
     }
 
-    /**
-     * @return string|callable
-     */
-    public function getGetter()
+    public function getGetter(): string|callable|null
     {
         return $this->getter;
     }
 
-    /**
-     * @param string|callable $getter
-     * @return $this
-     */
-    public function setGetter($getter): self
+    public function setGetter(string|callable|null $getter): self
     {
         $this->getter = $getter;
 

src/DataConverter.php

@@ -25,7 +25,8 @@ public function convertEntityToArray(array $entities, array $cols): array
         //apply custom order of columns
         $getters = [];
         foreach ($cols as $col) {
-            if (!$col->getGetter()) {
+            $getter = $col->getGetter();
+            if (null === $getter || '' === $getter) {
                 throw new Exception(
                     sprintf(
                         "All getters need to be set in the ColumnSettings when using entities. Missing getter for column '%s'",
@@ -34,7 +35,7 @@ public function convertEntityToArray(array $entities, array $cols): array
                 );
             }
 
-            $getters[] = $col->getGetter();
+            $getters[] = $getter;
         }
 
         //call all getters