While building a custom search, I noticed the apachesolr_search_taxonomy_get_term() code wasn't running. Looks like 'field_name' in the facet definitions has been renamed to 'facet_field'. When that was fixed, everything broke when apachesolr_search_taxonomy_get_term() used the return from taxonomy_get_term() as an array. Patch for both attached.

CommentFileSizeAuthor
#1 facet_definition_field.patch1.4 KB13rac1

Comments

13rac1’s picture

13rac1 commented
StatusFileSize
new facet_definition_field.patch1.4 KB

Argh. Really attached this time.

jpmckinney’s picture

jpmckinney commented

#809068: Problem with apachesolr_search_nested_facet_items when called from apachesolr_search_currentsearch_block for CCK fields is possibly related.

davyvdb’s picture

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

Works great. Works for taxonomies too. We could close #862434: apachesolr_search_taxonomy_get_term broken when this one's committed too.

robertdouglass’s picture

robertdouglass commented
Version: 6.x-2.x-dev » 7.x-1.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

#828412 by eosrei | jpmckinney: Fixed apachesolr_get_facet_definition() searches incorrect field for definition.

Committed to 6.2. Thanks! Need to check if upstream needs fixing.

jpmckinney’s picture

jpmckinney commented
Status: Patch (to be ported) » Fixed

apachesolr_search_taxonomy_get_term (now apachesolr_taxonomy_get_term) is fixed in 7.x-1.x and the other issue doesn't exist in 7.x.

Status: Fixed » Closed (fixed)

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