Hello,
I don't use apachesolr_search because I have my own rendering result.
So the dependency of apachesolr_search is not necessary for me.
Specially when i read this code (apachesolr_multilingual_apachesolr_query_alter):

else {
  // A custom query has been created by a third party module.
  // Assume default multilingual settings.
  // TODO This behavior should be somehow configurable.
}

What do you think?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Status: Needs review » Needs work

The last submitted patch, remove_dependency_apachesolr_search.patch, failed testing.

NerOcrO’s picture

Status: Needs work » Needs review
FileSize
385 bytes
mkalkbrenner’s picture

Category: Bug report » Feature request
Status: Needs review » Needs work

Sounds reasonable. But the patch is incomplete. At least function apachesolr_multilingual_requirements() in apachesolr_multilingual.install needs to be modified.

NerOcrO’s picture

Status: Needs work » Needs review
FileSize
892 bytes

Ok thanks for your comment :)
I changed the .info with apachesolr finally, it makes sense.

mkalkbrenner’s picture

I think the check for apachesolr_search in hook_requirements needs to be inverted.

NerOcrO’s picture

Hi mkalkbrenner,
Sounds good for me, thank's!