New notices and warnings in WP_Date_Query

[dlh]

Some parameters of WP_Date_Query now generate Array to string conversion and _doing_it_wrong() notices when passed an array.

For example:

new WP_Query( array(
    'date_query' => array(
        array(
            'compare' => 'BETWEEN',
            'day' => array( 4, 7 ),
        ),
    ),
) );

new WP_Query( array(
    'date_query' => array(
        array(
            'compare' => 'BETWEEN',
            'dayofweek' => array( 2, 6 ),
        ),
    ),
) );

dayofyear, month, year, week, and hour, minute, and second also generate notices.

Passing an array to year also generates new warnings:

• mktime() expects parameter 6 to be long, array given in /wp-includes/date.php on line 309
• DateTime::setISODate() expects parameter 1 to be long, array given in /wp-includes/date.php on line 342

Related: #29822.

[boonebgorges]

Thanks for the ticket and for the research, dlh. Date validation was introduced in [29925] #25834, but it did not properly account for arrays being passed as the value of 'day', 'dayofweek', etc.

31001.diff​ should fix the problem. Could you give it a look?

[dlh]

boonebgorges: I tested the patch against my original queries, and each query worked as expected without notices or warnings.

One minor question: If my array has two incorrect values, such as 'hour' => array( 0, 25 ), should I get two notices? Right now I get only a notice for the 0.

Thanks also for pointing me to the better source of context for this in #25834.

[boonebgorges]

dlh - Thanks for having a look. I'll make a fix to ensure that notices are thrown for all incorrect values.

[boonebgorges]

In 31179:

Support array values in WP_Date_Query::validate_date_values().

Introduced in [29925], validate_date_values() throws _doing_it_wrong()
notices when values passed as part of a WP_Date_Query do not reflect actual
dates. However, the validation did not account properly for the case where an
array of multiple values is passed, as when doing IN or BETWEEN queries.

Props dlh.
Fixes #31001 for trunk.

[boonebgorges]

Reopening for 4.1.1 consideration.

[dlh]

31001.2.patch​ attempts to update the documentation with the support for arrays, but I'm not sure about the best way to handle the repetition involved.

[boonebgorges]

dlh - Excellent, thanks. I've opened #31036 for the docs issue, as I'd like to keep it out of the 4.1.1 milestone.

[dd32]

In 31396:

Support array values in WP_Date_Query::validate_date_values().

Introduced in [29925], validate_date_values() throws _doing_it_wrong()
notices when values passed as part of a WP_Date_Query do not reflect actual
dates. However, the validation did not account properly for the case where an
array of multiple values is passed, as when doing IN or BETWEEN queries.

Props dlh.
Merges [31179] to the 4.1 branch.
Fixes #31001.