Turning multiSort ON breaks sorting persistence

[jillianbeann]

Livewire PowerGrid version

v6.9.0

Livewire version

v3.7.15

Laravel version

v12.58.0

Which PHP version are you using?

PHP 8.3

Which Theme are you using?

Tailwind

Have you published the resource files and customized them? (php artisan vendor:publish)

No

What is the problem?

Enabling the multiSort feature prevents the retention of the sort order.

Also, the documentation for the array passed to the persist method is incorrect.

Docs: Persist

public function setUp(): array
    {
        $this->persist(
                tableItems: ['columns', 'filters', 'sort'], !!------> should be 'sorting'
                prefix: auth()->id
        );
    }

Code snippets

Located in the Sorting trait

Problem area: return;

public function sortBy(string $field, string $direction = 'asc'): void
    {
        if ($this->multiSort) {
            $this->sortByArray($field);

            return;
        }

        $this->sortDirection = $this->sortField === $field ? $this->reverseSort() : Sql::sanitizeSortDirection($direction);

        $this->sortField = $field;

        $this->persistState('sorting');
    }

THE FIX:

public function sortBy(string $field, string $direction = 'asc'): void
    {
        if ($this->multiSort) {
            $this->sortByArray($field);
        } else {
          $this->sortDirection = $this->sortField === $field ? $this->reverseSort() : Sql::sanitizeSortDirection($direction);
  
          $this->sortField = $field;
        }

        $this->persistState('sorting');
    }

How do you expect it to work?

Enabling the multisort feature should not break the persistence of the sort state.

Sortarray persistence is already implemented in the persistState method (Persist trait)

Please confirm (incomplete submissions will not be addressed)

• I have provided easy and step-by-step instructions to reproduce the bug.
• I have provided code samples as text and NOT images.
• I understand my bug report will be removed if I haven't met the criteria above.

[virtualtear]

hey this is fixed

[jillianbeann]

Thanks so much for the quick turnaround! Very much appreciated 😄