We have several identical installations with one where a taxonomy used for an exposed filter has no terms (the view makes sense on that installation even if there are no terms in this filter). This triggers the following error, because the array that provides the filter options is NULL:

Error: Unsupported operand types in views_handler_filter->exposed_translate() (line 1193 of /var/www/html/sites/all/modules/contrib/views/handlers/views_handler_filter.inc).

This is resolved by initializing the options array a few lines earlier than it is currently done in the module.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

boromino created an issue. See original summary.

boromino’s picture

joseph.olstad’s picture

patch needs to be rerolled with 'git'
see documentation here:

joseph.olstad’s picture

Status: Needs review » Needs work
Michael-IDA’s picture

does patching with diff a/b no longer work on drupal.org? Forcing only git is going to lose people...

DamienMcKenna’s picture

Version: 7.x-3.23 » 7.x-3.x-dev

@Michael-IDA: we need correctly formatted diffs so that the testbot can run, as you can see this patch didn't work.

boromino’s picture

Status: Needs work » Needs review
Michael-IDA’s picture

@DamienMcKenna,

Correctly formatted diffs is understood. The question is: Is the original, non-git, method (that worked with testbot last I knew) no longer supported?

Best,
Michael

DamienMcKenna’s picture

@Michael-IDA: Ultimately no, whatever tool you used to create that patch did not create it in the correct format, you might check your IDE to see if it has options for creating them in a different format. That said, someone else can then recreate your changes and upload a correctly formatted patch.

ron_s’s picture

Status: Needs review » Reviewed & tested by the community

Rolled out the latest Views release yesterday, and started getting complaints of a "500 ajax error" when attempting to open an Entity Reference View Widget.

Could not reproduce the issue locally, until we realized the view has an exposed filter term, and the content editors had previously removed all the terms for that taxonomy on the production site.

Applying patch #7 resolved the issue. Thank you for the quick efforts.

joseph.olstad’s picture

Nice reroll for the patch, I was going to do it for you but I thought it'd be more helpful if I just put the link to the documentation. Give a man a fish, eats for a day, teach a man to fish and he can catch his own.

Great work everyone! :)
Thanks!

DamienMcKenna’s picture

Status: Reviewed & tested by the community » Fixed
Parent issue: » #3054023: Plan for Views 7.x-3.24 release

Committed. Thanks all.

Status: Fixed » Closed (fixed)

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