A lot of the methods in \Drupal\search_api\Plugin\search_api\tracker\Basic actually make much more sense in \Drupal\search_api\Tracker\TrackerPluginBase. Currently, most tracker plugins that don’t implement entirely new logic need to extend from Basic, which is terrible DX.

Issue fork search_api-3542807

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

drunken monkey created an issue. See original summary.

drunken monkey opened merge request !277

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Active » Needs review

Would be implemented in this MR.
I also wrote a change record.

Please review.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Needs review » Fixed
Issue tags: +Vienna2025

Merged.

Now that this issue is closed, please review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, please credit people who helped resolve this issue.

  • drunken monkey committed cf4d835c on 8.x-1.x 
    [#3542807] chore: Moved all methods from the Basic tracker plugin to...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.