This is a two part patch...

1) There is a fix for the use of a Drupal 6 function (taxonomy_vocabulary_load) where it should be taxonomy_get_vocabulary for Drupal 5.

2) To get the system working with Multisite again, I believe we need to add some facets to the the response... We at least need to make sure the URL's returned are absolute otherwise the multisite URL's end up referring to the external site on internal URL's.

CommentFileSizeAuthor
#8 apachsolr-vocab.patch919 bytesnicholasthompson
#2 apachesolr.multisite-vocab.patch6.28 KBnicholasthompson
apachesolr.multisite-vocab.patch1.79 KBnicholasthompson

Comments

nicholasthompson’s picture

nicholasthompson
Primary language English
commented

Ok - new patch - this one actually WORKS.

It fixes the broken vocab function as before.

Multisite now works. There is an optional facet block + a configuration setting (in admin/settings/apachesolr, under Advanced Settings). This configuration allows you to specify "friendly" names for the sites (rather than the site Protocol + URL being used).

I'm going to roll this out onto a live site today if you need to see a demo of it.

nicholasthompson’s picture

nicholasthompson
Primary language English
commented
StatusFileSize
new apachesolr.multisite-vocab.patch6.28 KB

Crap - forgot the patch...

nicholasthompson’s picture

nicholasthompson
Primary language English
commented

Fyi - it's live here:
http://www.pponline.co.uk/search/apachesolr_search/boxing

claudiu.cristea’s picture

claudiu.cristea
Primary language Romanian
Location Arad 🇷🇴
commented
Title: Incorrect function for getting vocabulary + Multisite implementation fix » Incorrect function for getting vocabulary
Assigned: Unassigned » claudiu.cristea
Status: Needs review » Needs work

Let's split the issue. In fact there are 2 different issues here. I'll make this only the vocabulary one. Please open another one for the multisite issue.

Thank you.

robertdouglass’s picture

robertdouglass commented

Agree with splitting the issue. We'll want to forward port this to the 6.x branches as well.

robertdouglass’s picture

robertdouglass commented

Doesn't look like this actually affects 6.x branches? Please confirm.

claudiu.cristea’s picture

claudiu.cristea
Primary language Romanian
Location Arad 🇷🇴
commented

Didn't have time to look into, but it seems an incomplete backport from 6.x-1.x. It seems that I forgot inside a D6 taxonomy function. While I didn't test ApacheSolr with taxonomy in my 5.x-2.x production site, this error didn't popup.

Yes. It's a "D5 only" issue. If I will discover that it isn't, I will ping you.

nicholasthompson’s picture

nicholasthompson
Primary language English
commented
StatusFileSize
new apachsolr-vocab.patch919 bytes

Vocab only patch

nicholasthompson’s picture

nicholasthompson
Primary language English
commented
Status: Needs work » Needs review

Setting status...

jaydub’s picture

jaydub commented
Status: Needs review » Reviewed & tested by the community

I ran into this problem as well. The patch to swap the functions works as intended and the taxonomy facet blocks work fine.

robertdouglass’s picture

robertdouglass commented
Status: Reviewed & tested by the community » Fixed

#627606 by nicholasThompson | jaydub: Fixed Incorrect function for getting vocabulary.

Status: Fixed » Closed (fixed)

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