If I enable the Language filter and put in the URL a filter for a language not present on the site, e.g. ?f[]=language:fr, I get a couple Notices:

* Notice: Uninitialized string offset: 0 in FacetapiAdapter->setBreadcrumb() (line 526 of /Users/Shared/www/modules/facetapi/plugins/facetapi/adapter.inc).
* Notice: Uninitialized string offset: 0 in FacetapiAdapter->buildCurrentSearch() (line 568 of /Users/Shared/www/modules/facetapi/plugins/facetapi/adapter.inc).

It also makes an invalid unclick link in the current search block.

I get the same Notices if I manually enter an invalid uid for an author filter, e.g. f[]=uid:589 for a site with 100 users.

CommentFileSizeAuthor
#1 1172034-1.patch1.34 KBpwolanin

Comments

pwolanin’s picture

pwolanin commented
Status: Active » Needs review
StatusFileSize
new 1172034-1.patch1.34 KB
cpliakas’s picture

cpliakas commented

Good catch, and the code looks good. Will test out on my local instance.

cpliakas’s picture

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

This looks good on my end. Commit when ready.

Thanks,
Chris

pwolanin’s picture

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

committed

Status: Fixed » Closed (fixed)

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