SearchKit - Fix clearing selection behavior

[colemanw]

Overview

This fixes the bug documented in https://lab.civicrm.org/dev/core/-/issues/2419

Before

After selecting all rows and updating search criteria, the rows would all still appear to be selected but the Actions menu would be disabled.

After

After selecting all (or any) rows and updating search criteria, the selection will be cleared (which was the original intent).

Technical Details

The variables allRowsSelected and selectedRows were sometimes getting out-of-sync.
This adds a function to help ensure they both get updated at the same time when the selection needs to be cleared.

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[eileenmcnaughton]

Yep tested this & it does what is says on the packet - good catch as this would be confusing

[dptarrant]

Yes, fixed. One small step for man ...