[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