Contrib.social

NumericFilter incorrectly expects to find form value keyed by 'value'

Open on Drupal.org →
Created on 22 January 2016, over 9 years ago
Updated 8 April 2025, 3 months ago

Problem/Motivation

Views filter plugins have a public "value" property in which values for a plugin can be injected. The form generated by this plugin can be quite nuanced and in some cases includes a separate 'value' form element, but all exposed forms are keyed by 'value' normaly so this is then ['value']['value']. While this is ok, that form element finds its default value in $this->value['value'], however this appears to have been copied over to the normal 'value' form element (which is produced in a different configuration circumstance). This is invalid as exposed forms are converted from the 'value' key to an $identifier key outside of this method before the form is rendered. Because of this, the exposed form's values will be submitted as that identifier. If a contrib module attempts to save that value, it can't properly inject it back into all views filter plugins because numeric's handling of this is wrong and affects all subclasses (date at the very least).

Proposed resolution

Set the default value by looking at $this->value[$identifier] instead

Remaining tasks

TBD

User interface changes

If anything this should cause certain situations in which submitted values might have disappeared to start working.

API changes

None

Data model changes

None

🐛 Bug report
Status

Postponed: needs info

Version

11.0 🔥

Component

views.module

Created by

🇺🇸United States eclipsegc

Live updates comments and jobs are added and updated live.
  • Contributed project blocker

    It denotes an issue that prevents porting of a contributed project to the stable version of Drupal due to missing APIs, regressions, and so on.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • Comment 3 months ago →
    🇳🇿New Zealand quietone

    Is this still a problem?

    I tested the patch on 11.x and the results was Warning: Undefined array key "field_number_value" in Drupal\views\Plugin\views\filter\NumericFilter->valueForm() (line 271 of core/modules/views/src/Plugin/views/filter/NumericFilter.php).. I don't know Views well enough to take this further.

    Can someone confirm that this problem is still valid?

    I am setting the status to Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

  • Status changed to Needs work 12 days ago
  • Comment 12 days ago →
    🇺🇸United States apmsooner

    It's still a problem and the patch doesn't fix it. The issue is that when you have a date filter exposed with a default value set and then you change the filter type to "Grouped filters", you get a validation error on the exposed filter in the views preview with something like this:

    The submitted value 2024-01-01 00:00:00 in the Date (field_date) element is not allowed.

    Once you save the view, the error goes away and doesn't come back but its in the initial ajax update from changing the filter type that appears to be triggering it. I havn't yet been able to figure out a solution.

  • Comment 12 days ago →
    🇺🇸United States apmsooner

    Okay closing this one in lieu of related ticket. That ticket isn't complete either but will continue the conversation over there as it has the root of where the issue is happening.

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024