This is a great module! It doesn't need to be limited to Solr, though. The dependency on search_api_solr is unnecessary; I'm using the module with an ElasticSearch backend right now. I'd love to be able to use the module without including search_api_solr. I'll submit a patch in a moment that does just that.

I'd +1 an effort to rename the module to exclude the world Solr since this works with ElasticSearch too. I'd be happy to work on that for you, but don't want to be presumptive.

Thanks for your work on this module!

CommentFileSizeAuthor
#1 search_api_solr-2358069-1.patch704 byteslukewertz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lukewertz’s picture

lukewertz
He/Him
CreditAttribution: lukewertz commented
FileSize
search_api_solr-2358069-1.patch704 bytes

Patch to remove the Solr references in the description and the dependencies.

kevinquillen’s picture

kevinquillen CreditAttribution: kevinquillen at Velir commented
Version: 7.x-1.0 » 7.x-1.x-dev
Assigned: Unassigned » kevinquillen
kevinquillen’s picture

kevinquillen CreditAttribution: kevinquillen at Velir commented

Interesting. I just looked at SearchAPI for 8.x and still lacks this functionality.

I should rewrite this to dump the Solr stuff (its on the name, etc) as a more generic thing for SearchAPI.

I will fix this for the time being in 7.x, but it will still be a little confusing due to the name of it.

kevinquillen’s picture

kevinquillen CreditAttribution: kevinquillen at Velir commented

The patch above will be the last applied patch on this module, and users will be directed to use the new version for 7 and 8 here:

https://www.drupal.org/project/search_api_view_modes

kevinquillen’s picture

kevinquillen CreditAttribution: kevinquillen at Velir commented
Status: Active » Reviewed & tested by the community

kevinquillen’s picture

kevinquillen CreditAttribution: kevinquillen at Velir commented
Status: Reviewed & tested by the community » Fixed
kevinquillen’s picture

kevinquillen CreditAttribution: kevinquillen at Velir commented
kevinquillen’s picture

kevinquillen CreditAttribution: kevinquillen at Velir commented
Title: search_api_solr is not a dependency » Solr should not be a dependency

Status: Fixed » Closed (fixed)

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