Tweaks to Configure Approach

Author: lrljoe

src/Traits/ComponentUtilities.php

@@ -8,11 +8,13 @@
 use Rappasoft\LaravelLivewireTables\Exceptions\DataTableConfigurationException;
 use Rappasoft\LaravelLivewireTables\Traits\Configuration\ComponentConfiguration;
 use Rappasoft\LaravelLivewireTables\Traits\Helpers\ComponentHelpers;
+use Rappasoft\LaravelLivewireTables\Traits\Core\QueryStrings\HasQueryString;
 
 trait ComponentUtilities
 {
     use ComponentConfiguration,
         ComponentHelpers;
+    use HasQueryString;
 
     public array $table = [];
 
@@ -48,6 +50,8 @@ trait ComponentUtilities
 
     protected bool $useComputedProperties = true;
 
+    protected bool $hasRunConfigure = false;
+
     /**
      * Set any configuration options
      */
@@ -71,16 +75,7 @@ public function mountComponentUtilities(): void
      */
     public function bootedComponentUtilities(): void
     {
-        // Fire Lifecycle Hooks for configuring
-        $this->callHook('configuring');
-        $this->callTraitHook('configuring');
-
-        // Call the configure() method
-        $this->configure();
-
-        // Fire Lifecycle Hooks for configured
-        $this->callHook('configured');
-        $this->callTraitHook('configured');
+        $this->runCoreConfiguration();
 
         // Make sure a primary key is set
         if (! $this->hasPrimaryKey()) {
@@ -89,6 +84,26 @@ public function bootedComponentUtilities(): void
 
     }
 
+    protected function runCoreConfiguration(): void
+    {
+        if (!$this->hasRunConfigure)
+        {
+            // Fire Lifecycle Hooks for configuring
+            $this->callHook('configuring');
+            $this->callTraitHook('configuring');
+
+            // Call the configure() method
+            $this->configure();
+
+            // Fire Lifecycle Hooks for configured
+            $this->callHook('configured');
+            $this->callTraitHook('configured');
+            
+            $this->hasRunConfigure = true;
+
+        }
+    }
+
     /**
      * Returns a unique id for the table, used as an alias to identify one table from another session and query string to prevent conflicts
      */

src/Traits/Configuration/QueryStringConfiguration.php

@@ -14,7 +14,7 @@ public function setQueryStringAlias(string $queryStringAlias): self
     public function setupQueryStringStatus(): void
     {
         if (! $this->hasQueryStringStatus()) {
-            $this->configure();
+            $this->runCoreConfiguration();
             if (! $this->hasQueryStringStatus()) {
                 $this->setQueryStringEnabled();
             }

src/Traits/Core/QueryStrings/HasQueryStringForFilter.php

@@ -20,7 +20,7 @@ protected function queryStringHasQueryStringForFilter(): array
             ] : [];
     }
 
-    public function setupQueryStringStatusForFilter(): void
+    protected function setupQueryStringStatusForFilter(): void
     {
         if (! $this->hasQueryStringStatusForFilter()) {
             $this->setQueryStringForFilterEnabled();

src/Traits/Core/QueryStrings/HasQueryStringForSort.php

@@ -27,7 +27,7 @@ public function getQueryStringStatusForSort(): bool
         return $this->getQueryStringConfigStatus('sorts');
     }
 
-    protected function queryStringForSortEnabled(): bool
+    public function queryStringForSortEnabled(): bool
     {
         $this->setupQueryStringStatusForSort();
 

src/Traits/HasAllTraits.php

@@ -12,6 +12,7 @@ trait HasAllTraits
     use WithLoadingPlaceholder;
     use HasTheme;
     use ComponentUtilities,
+        WithQueryString,
         WithActions,
         WithData,
         WithColumns,
@@ -28,7 +29,6 @@ trait HasAllTraits
         WithEvents,
         WithFilters,
         WithFooter,
-        WithQueryString,
         WithRefresh,
         WithReordering,
         WithSecondaryHeader,