Issue

I think an issue was made in #3173836: Skip status report for disabled servers which set to ignore disabled indexes in search_api_solr_requirements.

However, when looking at the code, the else that will say "Solr not reachable" is associated with if ($server->status()) { which is the config entity status check.

The if ($backend->isAvailable() && $connector->pingCore()) { which checks the actual connection, has no else (and thus no error reporting) associated with it.

I believe the else that reports "Solr not reachable" should've been indented one level more so it's attached to if ($backend->isAvailable() && $connector->pingCore()) { rather than to the status check.

Issue fork search_api_solr-3338311

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

Kingdutch created an issue. See original summary.

mkalkbrenner’s picture

mkalkbrenner
Primary language German
Location 🇩🇪
commented

You're right!

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

mkalkbrenner’s picture

mkalkbrenner
Primary language German
Location 🇩🇪
commented

I already committed a patch to github

sakthi_dev opened merge request !25

  • mkalkbrenner committed 062a15d0 on 4.x 
    Issue #3338311 by Kingdutch, mkalkbrenner: Disabled servers are reported...
mkalkbrenner’s picture

mkalkbrenner
Primary language German
Location 🇩🇪
commented
Status: Active » Fixed

Status: Fixed » Closed (fixed)

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