Due to #3065385: Drush commands entirely broken because of Search API changes, this module is now incompatible with Search API Solr 3.1, 3.0 and any 2.x version. The 1.x versions are not affected. We should explicitly declare this in our composer.json to avoid anyone running into problems.

CommentFileSizeAuthor
#3 3065550.patch267 bytesmkalkbrenner
#2 3065550-2--declare_solr_version_conflict.patch453 bytesdrunken monkey

Comments

drunken monkey created an issue. See original summary.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Active » Needs review
StatusFileSize
new 3065550-2--declare_solr_version_conflict.patch453 bytes

This should do it.

mkalkbrenner’s picture

mkalkbrenner
Primary language German
Location 🇩🇪
commented
Title: Declare conflict for Search API Solr versions lower than 3.2 » Declare conflict for Search API Solr versions 2.x and 3.x lower than 3.2
StatusFileSize
new 3065550.patch267 bytes

Only search_api_solr 2.x and 3.x are affected. 1.x is still supported until December 2019 and is not affected. I adjusted the patch accordingly.
I assume that this is the correct / best syntax.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented

Ah, alright, thanks for clarifying! Then I’ll commit this right before releasing 1.14 this Sunday.

cspitzlay’s picture

cspitzlay
Primary language German
Location 🇩🇪🇪🇺
commented
Issue summary: View changes
mkalkbrenner’s picture

mkalkbrenner
Primary language German
Location 🇩🇪
commented
Status: Needs review » Reviewed & tested by the community

I suggest that this gets committed right now. People who use composer to install dev versions should be able to declare that search_api_solr 3.x-dev satisfies 3.2 ;-)

The advantage would be that we get the constraint tested on travis with search_api_solr 1.x and 3.x because we install search_api 1.x-dev via composer there.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Reviewed & tested by the community » Fixed

Alright, makes sense. Committed.
Thanks again!

Status: Fixed » Closed (fixed)

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