Minor tweaks to Footer/SecondaryHeader to ensure presence of callback prior to executing

Author: lrljoe

src/Views/Columns/Traits/HasFooter.php

@@ -53,11 +53,20 @@ public function getFooterCallback()
 
     public function footerCallbackIsString(): bool
     {
+        if(!$this->hasFooterCallback())
+        {
+            return false;
+        }
         return is_string($this->getFooterCallback());
     }
 
     public function footerCallbackIsFilter(): bool
     {
+        if(!$this->hasFooterCallback())
+        {
+            return false;
+        }
+
         $callback = $this->getFooterCallback();
 
         return $callback instanceof Filter;
@@ -66,9 +75,10 @@ public function footerCallbackIsFilter(): bool
     public function getFooterContents(mixed $rows, array $filterGenericData): \Illuminate\Contracts\Foundation\Application|\Illuminate\View\Factory|\Illuminate\View\View|string|HtmlString
     {
         $value = null;
-        $callback = $this->getFooterCallback();
-
         if ($this->hasFooterCallback()) {
+
+            $callback = $this->getFooterCallback();
+
             if (is_callable($callback)) {
                 $value = call_user_func($callback, $rows);
 

src/Views/Columns/Traits/HasSecondaryHeader.php

@@ -50,11 +50,21 @@ public function getSecondaryHeaderCallback(): mixed
 
     public function secondaryHeaderCallbackIsString(): bool
     {
+        if(!$this->hasSecondaryHeaderCallback())
+        {
+            return false;
+        }
+
         return is_string($this->getSecondaryHeaderCallback());
     }
 
     public function secondaryHeaderCallbackIsFilter(): bool
     {
+        if(!$this->hasSecondaryHeaderCallback())
+        {
+            return false;
+        }
+
         $callback = $this->getSecondaryHeaderCallback();
 
         return $callback instanceof Filter;
@@ -63,9 +73,11 @@ public function secondaryHeaderCallbackIsFilter(): bool
     public function getSecondaryHeaderContents(mixed $rows, array $filterGenericData): \Illuminate\Contracts\Foundation\Application|\Illuminate\View\Factory|\Illuminate\View\View|string|HtmlString
     {
         $value = null;
-        $callback = $this->getSecondaryHeaderCallback();
 
         if ($this->hasSecondaryHeaderCallback()) {
+
+            $callback = $this->getSecondaryHeaderCallback();
+
             if (is_callable($callback)) {
                 $value = call_user_func($callback, $rows);
                 if ($this->isHtml()) {