v3 - Hooks And Pre-Check Moves

[lrljoe]

This PR adds:

Tweaks

• Moving checks for certain methods earlier in the lifecycle:
  • Move checks for Search out to WithData (i.e. if ($this->searchIsEnabled() && $this->hasSearch()))
  • Move checks for Filters out to WithData (i.e. if ($this->filtersAreEnabled() && $this->hasFilters() && $this->hasAppliedFiltersWithValues()))

Hooks

Added:

• rowsRetrieved (fires in getRows() - after query execution, and before return, and provides the collection of rows

All Submissions:

• [ ] Have you followed the guidelines in our Contributing document?
• [ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Feature Submissions:

1. [ ] Does your submission pass tests and did you add any new tests needed for your feature?
2. [ ] Did you update all templates (if applicable)?
3. [ ] Did you add the relevant documentation (if applicable)?
4. [ ] Did you test locally to make sure your feature works as intended?

Changes to Core Features:

• [ ] Have you added an explanation of what your changes do and why you'd like us to include them?
• [ ] Have you written new tests for your core changes, as applicable?
• [ ] Have you successfully ran tests with your changes locally?

[codecov[bot]]

Codecov Report

Merging #1538 (d7197b4) into develop (f73b59e) will increase coverage by 0.00%. The diff coverage is 95.23%.

@@            Coverage Diff             @@
##             develop    #1538   +/-   ##
==========================================
  Coverage      88.98%   88.99%           
  Complexity      1264     1264           
==========================================
  Files             96       96           
  Lines           2997     2999    +2     
==========================================
+ Hits            2667     2669    +2     
  Misses           330      330           

Files	Coverage Δ
src/Traits/WithData.php	79.41% <100.00%> (+0.84%)	:arrow_up:
src/Traits/WithSearch.php	93.75% <100.00%> (-0.37%)	:arrow_down:
src/Traits/WithFilters.php	88.23% <85.71%> (-0.66%)	:arrow_down:

[stale[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.