Fatal error: [] operator not supported for strings in /private/var/aegir/platforms/drupal7/html/sites/all/modules/b_test/facetapi/plugins/facetapi/url_processor_standard.inc on line 45

This can be replicated with search_facetapi module. Say the search is active, e.g. a path like "search/content/et". I manually input something like "?f=blah" in the url - not "?f[0]=blah". The error occurs.

I think you may want to remove $qstring if it's not an array, or maybe build an array out of it?

CommentFileSizeAuthor
#2 facetapi-1315488-2.patch611 bytescpliakas

Comments

cpliakas’s picture

cpliakas commented
Issue tags: +low hanging fruit

Good catch. There are some sanity checks that remove unexpected values, but clearly this use case has been missed. I will try to replicate, and patches are definitely welcome.

Thanks,
Chris

cpliakas’s picture

cpliakas commented
Status: Active » Needs review
StatusFileSize
new facetapi-1315488-2.patch611 bytes

The attached patch seems to work for me. Would love to get some testers.

Thanks,
Chris

makara’s picture

makara commented
Component: Code » Code (functionality)
Status: Needs review » Reviewed & tested by the community

Works for me.

cpliakas’s picture

cpliakas commented

Thanks for testing!

cpliakas’s picture

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

Resolved in commit 4508262.

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