This is a bit of an edge case, but when checking the server status on the settings page the environment's service class is not used. Regardless of what is defined it always defaults to DrupalApacheSolrService.

CommentFileSizeAuthor
#5 service-class-1613328-5.patch1.09 KBcpliakas
#1 service-class-1613328-1.patch528 bytescpliakas

Comments

cpliakas’s picture

cpliakas commented
Status: Active » Needs review
StatusFileSize
new service-class-1613328-1.patch528 bytes

The attached patch passes the service class to apachesolr_server_status().

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented

The code you found was inherently wrong so fix accepted. Does this fix the problem you encountered?

Let's wait till testbot works

cpliakas’s picture

cpliakas commented

It did. There are actually a couple of other instances where the class is not passed to the apachesolr_server_status() function. Do you want me to file separate issues for those or re-roll a patch with all instances accounted for in this thread?

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented

You can do it in this thread

cpliakas’s picture

cpliakas commented
StatusFileSize
new service-class-1613328-5.patch1.09 KB

So not bad. Only one more instance.

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented
Status: Needs review » Reviewed & tested by the community
nick_vh’s picture

nick_vh
he/him
Location Ghent
commented
Version: 7.x-1.0-beta19 » 6.x-3.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Committed to D7

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented
Status: Patch (to be ported) » Fixed

pushed to D6

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented
Status: Fixed » Closed (fixed)