Problem/Motivation

The service "search.index" uses database queries and should therefore be backend_overridable. This is blocking the database driver for MongoDB.

Proposed resolution

Make the service "search.index" backend_overridable.

Remaining tasks

User interface changes

None

API changes

None

Data model changes

None

Release notes snippet

None

Issue fork drupal-3209453

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

daffie created an issue. See original summary.

daffie’s picture

daffie commented
Assigned: Unassigned » daffie
daffie’s picture

daffie commented
Assigned: daffie » Unassigned
Status: Active » Needs review

daffie opened merge request !572

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Needs review » Reviewed & tested by the community

Looks straightforward enough. Not even sure we need the test, we don't seem to really test the backend_overridable feature at all, but I guess it can't hurt.

daffie’s picture

daffie commented
Parent issue: » #3209798: [META] Make the contrib database driver for MongoDB work without patching core

@longwave: Thank you for the review!

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
commented
Issue tags: +Needs change notice

Can we get a change notice here?
Of if there's a few of these issues you're working on for mongodb, perhaps a bulk one.

daffie’s picture

daffie commented
Issue tags: -Needs change notice

Added a CR.

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
commented
Status: Reviewed & tested by the community » Fixed

Thanks

  • larowlan committed 83c20c7 on 9.2.x authored by daffie 
    Issue #3209453 by daffie: The service "search.index" should be...

Status: Fixed » Closed (fixed)

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