- Merge request !2483Issue #3294700: Warning: Undefined array key "type" in Drupal\views\Plugin\views\filter\Date->acceptExposedInput() → (Open) created by dench0
- 🇺🇸United States alison
**maybe** duplicate of 🐛 Views exposed group filter (with option allow multiple selections) generates notices when using date and date range fields Needs work ?
- First commit to issue fork.
- last update
over 1 year ago 30,330 pass - 🇮🇳India bharath-kondeti Hyderabad
Made few changes on top of the MR. I will update the test cases as well.
33:47 32:45 Running- last update
over 1 year ago Custom Commands Failed - 🇨🇦Canada jodavidson
After applying the patch of #7 (which fixed the 'type' issue, well done.) I found this error:
Warning: Undefined array key "value" in Drupal\views\Plugin\views\filter\Date->acceptExposedInput() (line 153 of /var/www/html/web/core/modules/views/src/Plugin/views/filter/Date.php)
Looking into the code it seems that $operators[$operator]['values'] can be 1 without $this->value['value'] being set.
Perhaps we can add an isset test to the if statement in this block like:
if (!isset($this->value['value']) || $this->value['value'] == '') { return FALSE; }
I'm suggesting this as a todo for you, as you already have the code and the existing patch available to you for tweaking. If it's preferable for me to try putting the combined patch together I can give it a shot. I hope this isn't too much of an imposition on your time.
- First commit to issue fork.
- 🇻🇳Vietnam phthlaap
This issue happened on my end; this is the view I tested: test_expose_date_filter.yml → or please see my screenshot →
- Merge request !7635re-roll in 11x and added a tests to cover the issue. → (Open) created by phthlaap
- Status changed to Needs review
8 months ago 2:02pm 21 April 2024 - 🇻🇳Vietnam phthlaap
phthlaap → changed the visibility of the branch 3294700-warning-undefined-array-key-type to hidden.
- Status changed to Needs work
8 months ago 6:21pm 22 April 2024 - 🇺🇸United States smustgrave
Removing tests tag as coverage appears to be there.
Left some comments on the MR but proposed solution doesn't match the MR