[wp-trac] [WordPress Trac] #29908: date_query inclusive logic still not quite right
WordPress Trac
noreply at wordpress.org
Sun Oct 12 20:58:32 UTC 2014
#29908: date_query inclusive logic still not quite right
-----------------------------+-----------------------------
Reporter: magicroundabout | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: Future Release
Component: Query | Version: 4.0
Severity: normal | Resolution:
Keywords: needs-patch | Focuses:
-----------------------------+-----------------------------
Comment (by magicroundabout):
Ah. Yes. Of course. Thanks @boonebgorges (P.S. Love that you're doing
advanced query improvements for 4.1)
The strtotime() means that this is correct and, to be fair, documented (if
slightly unexpected) behaviour. I should have got that.
I'm happy not to patch this if the documentation is made a bit clearer.
I've added a note to the codex example to clarify to others!
Do you think that this needs fixing by doing the string-to-array
conversion that you suggest? If we do, we'll have to remember to remove
the note from the codex.
Thanks
--
Ticket URL: <https://core.trac.wordpress.org/ticket/29908#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list