[wp-trac] [WordPress Trac] #29908: date_query inclusive logic still not quite right
WordPress Trac
noreply at wordpress.org
Tue Oct 14 00:10:18 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 boonebgorges):
Replying to [comment:8 dd32]:
> 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?
Nope, no harm. I agree the `/` format is not used much when writing code.
I just threw it in at the last minute because it was a minor mod to the
regular expression. It's pretty arbitrary where we draw the line between
supported formats and unsupported formats, but maybe saying "we support
subsets of MySQL format" is a bit less arbitrary than other choices. Happy
to go with majority opinion on this point.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/29908#comment:9>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list