[wp-trac] [WordPress Trac] #29908: date_query inclusive logic still not quite right
WordPress Trac
noreply at wordpress.org
Mon Oct 13 23:31:39 UTC 2014
#29908: date_query inclusive logic still not quite right
-------------------------------------+---------------------------
Reporter: magicroundabout | Owner: boonebgorges
Type: defect (bug) | Status: accepted
Priority: normal | Milestone: 4.1
Component: Query | Version: 4.0
Severity: normal | Resolution:
Keywords: has-patch needs-testing | Focuses:
-------------------------------------+---------------------------
Comment (by dd32):
Replying to [comment:5 boonebgorges]:
> The real challenge here is figuring out what date formats to accept.
[attachment:29908.patch] adds inclusive support for `Y`, `Y-m`, `Y-m-d`,
`Y-m-d H:i`, and `Y-m-d H:i:s`, as well as the same formats with `/`
instead of `-`.
I'd question the inclusion of `/` personally, It's not a date format used
within code very often (IMHO) and could increase confusion of it
supporting `Y/d/m`. Is there any harm in only supporting the standard
`Y-m-d H:i:s` (or part of) mysql format?
--
Ticket URL: <https://core.trac.wordpress.org/ticket/29908#comment:8>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list