On line 1189, there's a typo -- it reads this:

function apacehsolr_get_parent_terms($tids) {

where it should read this:

function apachesolr_get_parent_terms($tids) {

Did a search here and didn't see anyone reporting it, so I thought I would. Cheers.

Comments

lazysoundsystem’s picture

...called on line 1230:

$all_terms = apacehsolr_get_parent_terms(array_keys($facets));

pwolanin’s picture

Version: 6.x-1.0-rc5 » 6.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new1.93 KB

Well, if they are using the same name, I guess that explains why it works.

robertdouglass’s picture

Version: 6.x-1.x-dev » 6.x-2.x-dev
StatusFileSize
new2.45 KB

The problem is bigger. Will have to do an update function for this one.

robertdouglass’s picture

HAHA! I'm grepping for the misspelling and recursively correcting the mistake :P

robertdouglass’s picture

Status: Needs review » Patch (to be ported)
StatusFileSize
new1.15 KB

Peter's patch has the md5 stuff in it too. Here's a patch against 6.2 that just corrects the spelling mistakes. Will fix and commit in all branches.

robertdouglass’s picture

Status: Patch (to be ported) » Fixed

#747346 by robertDouglass, pwolanin | lazysoundsystem: Fixed Typo in apachesolr_search().module.

Committed to all 3 active branches.

pwolanin’s picture

Wow - so much bad typing. Sorry...

Status: Fixed » Closed (fixed)

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