Problem/Motivation

The SolrDocumentDeriver checks if the index $server is not empty but does not check if the index has a valid server first. This can cause issues when the server is external and perhaps connection fails, it becomes impossible to actually get to the Admin > Config > Search API list of servers to see the server status of each via the Search API UI.

Steps to reproduce

Cause a server to be invalid on an index with solr document deriver use

Proposed resolution

Check server is valid

Remaining tasks

MR

User interface changes

N/A

API changes

N/A

Data model changes

N/A

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:

Comments

scott_euser created an issue. See original summary.

scott_euser’s picture

Assigned: scott_euser » Unassigned
Status: Active » Needs review

Like #3581376: Facet display deriver should check if it has a valid server instance, I think this is quite a safe extra check to prevent a fatal error as the code is already checking that the server is not empty, so its already not assuming it has a valid one

scott_euser’s picture

Failing test on composer re drupal/search_api_spellcheck is unrelated to this

mkalkbrenner made their first commit to this issue’s fork.

mkalkbrenner’s picture

Status: Needs review » Fixed

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

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

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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