We are getting this error since upgrading to 7.x-3.23:

Notice: Undefined index: value in views_handler_filter->accept_exposed_input() (line 1390 of .../views/handlers/views_handler_filter.inc).

It is caused by commit 50fe583 from #477984: CCK field is empty/not empty filter doesn't work when exposed.

This is the line causing the problem:

$value = is_array($value) ? $value['value'] : $value;

We are getting this error on admin/people. The value of $value is array('all' => 'all'). So, it counts as an array, but $value['value'] doesn't exist. This may be due to some customization on our end.

CommentFileSizeAuthor
#10 views_php-notice-value_3060019.patch1.15 KBjenlampton
#5 views-undefined_index_value-3060019-5-D7.patch828 bytesLiam Morland
#2 views-undefined_index_value-3060019-2-D7.patch808 bytesLiam Morland
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Liam Morland created an issue. See original summary.

Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland at University of Waterloo commented
Status: Active » Needs review
FileSize
views-undefined_index_value-3060019-2-D7.patch808 bytes

Fix.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Assigned: Liam Morland » Unassigned
Parent issue: #477984: CCK field is empty/not empty filter doesn't work when exposed » #3054023: Plan for Views 7.x-3.24 release
Related issues: +#477984: CCK field is empty/not empty filter doesn't work when exposed

Good catch, thank you.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs review » Needs work

I think it'd be more readable if it was changed to an if() statement.

Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland at University of Waterloo commented
Status: Needs work » Needs review
FileSize
views-undefined_index_value-3060019-5-D7.patch828 bytes

Fix as an if statement.

tomdearden’s picture

tomdearden CreditAttribution: tomdearden commented

I'm seeing this issue, too- with grouped filters set up using Views UI on v7.x-3.23. Both the patches (#3 and #5) above work..

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs review » Fixed

Committed. Thank you.

Status: Fixed » Closed (fixed)

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

jenlampton’s picture

jenlampton
she / her
CreditAttribution: jenlampton at Jeneration Web Development for Sirius Puppy Training commented
FileSize
views_php-notice-value_3060019.patch1.15 KB

I'm seeing this same notice for numeric handler:

Undefined index: value in views_handler_filter_numeric->accept_exposed_input() (line 399 of www/sites/all/modules/contrib/views/handlers/views_handler_filter_numeric.inc).
Liam Morland’s picture

Liam Morland
Primary language English
Location Ontario, CA 🇨🇦
CreditAttribution: Liam Morland commented

That looks like something that should have a new issue.