- Issue created by @AaronBauman
- Status changed to Needs review
about 1 year ago 4:45pm 12 October 2023 - πΊπΈUnited States AaronBauman Philadelphia
OK, I've narrowed down the issue (in my case) to exposed date filter using the "between" operator.
The wrapper-handling logic in FilterWidgetBase is choking. Not sure how widely this applies otherwise.Here's a test that should fail to demonstrate the issue.
Will follow up with a one-liner to fix.In this patch:
- Add a field "bef_date" to the "bef_test" content type
- Add the field as an exposed filter on the "bef_test" viewObserved behavior:
Fatal error when loading the view, because of the error referenced in title. - last update
about 1 year ago 17 pass, 24 fail - last update
about 1 year ago 20 pass, 22 fail - πΊπΈUnited States AaronBauman Philadelphia
Test was failing, but for the wrong reason.
I forgot simpletest doesn't know how to figure out dependencies.
Added "datetime" to the test modules, should give the appropriate result. - last update
about 1 year ago 37 pass - last update
about 1 year ago 27 pass, 7 fail - πΊπΈUnited States AaronBauman Philadelphia
Interesting, seems like this is only an issue for D10.
But BEF builds are failing tests for D10, so I can't provide a test to demonstrate the failure.Once builds are passing on D10, we should revisit the tests.
In the meantime, the MR is a one-liner that fixes the issue for me.
- πΊπΈUnited States AaronBauman Philadelphia
Did you try adding a datetime field, and datetime exposed filter with "between" operator?
I have replicated it consistently on a vanilla D10.1 install. - π¨π¦Canada optasy
I tested this on a 10.0.x Drupal installation and couldn't replicate this, I'll try on a D10.1.x and see
- Status changed to Postponed: needs info
9 months ago 8:10pm 10 April 2024 - πΊπΈUnited States smustgrave
Tried replicate on 6.0.x and 10.3.x but not able to replicate, is this still an issue?
- last update
9 months ago Patch Failed to Apply - Status changed to Needs review
8 months ago 3:21pm 11 April 2024 - πΊπΈUnited States AaronBauman Philadelphia
Doesn't seem like it's still happening, but let's see what testbot says.
Here's a re-roll of the TEST-ONLY patch against latest dev.
- last update
8 months ago 36 pass, 1 fail - πΊπΈUnited States smustgrave
So just FYI gitlab has a test-only feature so you don't need to do test-only patches. Plus drupalCI is wildly behind and soon to be shutdown.
- Status changed to Closed: cannot reproduce
8 months ago 3:26pm 11 April 2024 - πΊπΈUnited States AaronBauman Philadelphia
Yeah, I did see that - very cool!
Looks like BEF tests are failing, but this one passes, so ... i assume that's a known issue and this is closeable.
https://git.drupalcode.org/issue/better_exposed_filters-3393662/-/jobs/1304335