[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