When setting the filter value via URL it always passes the value through explode(',') to turn it into an array, which is problematic for filters that only expect a single value.

CommentFileSizeAuthor
#1 global_filter-2349201-value-from-url.patch903 bytesgapple
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

gapple’s picture

gapple
he/they
Primary language English
CreditAttribution: gapple commented
Status: Active » Needs review
FileSize
global_filter-2349201-value-from-url.patch903 bytes

The attached patch checks the filter widget info, and doesn't pass the value through explode if the widget is a single-value type.

RdeBoer’s picture

RdeBoer
Location Melbourne
CreditAttribution: RdeBoer as a volunteer commented
Assigned: Unassigned » RdeBoer
Status: Needs review » Closed (fixed)

Fixed in a more general way, long time ago.

gapple’s picture

gapple
he/they
Primary language English
CreditAttribution: gapple as a volunteer commented
Status: Closed (fixed) » Postponed (maintainer needs more info)

This was an issue on a recent stable release, and I wasn't able to see anything in the latest dev release that would address it.

I'll see if I can reproduce the issue on a clean install of just this module, or if it's a customization on the site where I encountered the problem that is causing the issue.

RdeBoer’s picture

RdeBoer
Location Melbourne
CreditAttribution: RdeBoer as a volunteer commented

Please do!
Thank you.
Rik

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented
Status: Postponed (maintainer needs more info) » Closed (outdated)

Closing to triage the issue queue, reopen if you feel this still needs to be addressed.