Closed (fixed)
Project:
Date
Version:
7.x-2.x-dev
Component:
Views Filter
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
23 Aug 2013 at 13:16 UTC
Updated:
20 May 2015 at 22:44 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
leewillis77 commentedPatch attached. This resolves the error for me.
Comment #3
leewillis77 commented#1: no-results-when-multiple-dates-included-in-views-filter-2072679-2.diff queued for re-testing.
Comment #5
leewillis77 commentedhttps://drupal.org/node/2077839 has been marked as a duplicate of this.
Comment #6
leewillis77 commentedAs far as I can tell the test failures are nothing to do with this patch. Can anyone else confirm?
Comment #7
leewillis77 commented[Removed comment, the 8.x branch seems largely irrelevant at this point]
Comment #8
cafuego commented#1: no-results-when-multiple-dates-included-in-views-filter-2072679-2.diff queued for re-testing.
Comment #10
leewillis77 commentedHi - as far as I can see the failing tests are nothing to do with this page - they seem to relate directly to date migrations which aren't touched by this patch.
Comment #11
morbiD commentedThe above patch seems to fix the problem for my particular use case ("Content: Last comment time" and "Content: Updated date" in the same date filter).
However, I should probably point out that the
} else {syntax in the patch doesn't comply with Drupal coding standards which state: "The closing curly should be on a line by itself and indented to the same level as the opening statement."Comment #13
cafuego commentedI've applied the patch and fixed the code style issue. Pushed to 7.x-2.x-dev.