Skip to content

Commit 12ccbd2

Browse files
adhocoreStyleCIBot
adhocore
authored and
StyleCIBot
committed
Apply fixes from StyleCI
[ci skip] [skip ci]
1 parent d9276f5 commit 12ccbd2

File tree

2 files changed

+63
-63
lines changed

2 files changed

+63
-63
lines changed

‎src/Expression.php‎

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -51,8 +51,8 @@ class Expression
5151
/**
5252
* Parse cron expression to decide if it can be run on given time (or default now).
5353
*
54-
* @param string $expr The cron expression.
55-
* @param int $time The timestamp to validate the cron expr against. Defaults to now.
54+
* @param string $expr The cron expression.
55+
* @param int $time The timestamp to validate the cron expr against. Defaults to now.
5656
*
5757
* @return bool
5858
*/
@@ -123,12 +123,12 @@ public static function isDue($expr, $time = null)
123123

124124
/**
125125
* Check if modifiers [L C W #] are satisfied.
126-
*
126+
*
127127
* @internal
128128
*
129-
* @param string $value
130-
* @param int $pos
131-
* @param int $time
129+
* @param string $value
130+
* @param int $pos
131+
* @param int $time
132132
*
133133
* @return bool
134134
*/
@@ -165,7 +165,7 @@ protected static function checkModifier($value, $pos, $time)
165165
if ($pos === 4) {
166166
if ($pos = strpos($value, 'L')) {
167167
$value = explode('L', str_replace('7L', '0L', $value));
168-
$decr = $time[6];
168+
$decr = $time[6];
169169
for ($i = 0; $i < 7; $i++) {
170170
$decr -= $i;
171171
if (date('w', strtotime("$time[5]-$month-$decr")) == $value[0]) {

‎tests/ExpressionTest.php‎

Lines changed: 56 additions & 56 deletions
Original file line numberDiff line numberDiff line change
@@ -35,84 +35,84 @@ public function test_isDue_throws_if_modifier_invalid()
3535

3636
/**
3737
* The test cases are taken from awesome mtdowling/cron-expression package. Thank you.
38-
*
38+
*
3939
* @link https://github.com/mtdowling/cron-expression/
40-
*
40+
*
4141
* Data provider for cron schedule
4242
*
4343
* @return array
4444
*/
4545
public function scheduleProvider()
4646
{
47-
return array(
48-
array('@always', time(), '', true),
49-
array('@5minutes', new \DateTime('2017-05-10 02:30:00'), '', true),
50-
array('* * 7W * *', '2017-10-15 20:00:00', '', false),
47+
return [
48+
['@always', time(), '', true],
49+
['@5minutes', new \DateTime('2017-05-10 02:30:00'), '', true],
50+
['* * 7W * *', '2017-10-15 20:00:00', '', false],
5151

52-
array('*/2 */2 * * *', '2015-08-10 21:47:27', '2015-08-10 22:00:00', false),
53-
array('* * * * *', '2015-08-10 21:50:37', '2015-08-10 21:50:00', true),
54-
array('* 20,21,22 * * *', '2015-08-10 21:50:00', '2015-08-10 21:50:00', true),
52+
['*/2 */2 * * *', '2015-08-10 21:47:27', '2015-08-10 22:00:00', false],
53+
['* * * * *', '2015-08-10 21:50:37', '2015-08-10 21:50:00', true],
54+
['* 20,21,22 * * *', '2015-08-10 21:50:00', '2015-08-10 21:50:00', true],
5555
// Handles CSV values
56-
array('* 20,22 * * *', '2015-08-10 21:50:00', '2015-08-10 22:00:00', false),
56+
['* 20,22 * * *', '2015-08-10 21:50:00', '2015-08-10 22:00:00', false],
5757
// CSV values can be complex
58-
array('* 5,21-22 * * *', '2015-08-10 21:50:00', '2015-08-10 21:50:00', true),
59-
array('7-9 * */9 * *', '2015-08-10 22:02:33', '2015-08-18 00:07:00', false),
58+
['* 5,21-22 * * *', '2015-08-10 21:50:00', '2015-08-10 21:50:00', true],
59+
['7-9 * */9 * *', '2015-08-10 22:02:33', '2015-08-18 00:07:00', false],
6060
// 15th minute, of the second hour, every 15 days, in January, every Friday
61-
array('1 * * * 7', '2015-08-10 21:47:27', '2015-08-16 00:01:00', false),
61+
['1 * * * 7', '2015-08-10 21:47:27', '2015-08-16 00:01:00', false],
6262
// Test with exact times
63-
array('47 21 * * *', strtotime('2015-08-10 21:47:30'), '2015-08-10 21:47:00', true),
63+
['47 21 * * *', strtotime('2015-08-10 21:47:30'), '2015-08-10 21:47:00', true],
6464
// Test Day of the week (issue #1)
6565
// According cron implementation, 0|7 = sunday, 1 => monday, etc
66-
array('* * * * 0', strtotime('2011-06-15 23:09:00'), '2011-06-19 00:00:00', false),
67-
array('* * * * 7', strtotime('2011-06-15 23:09:00'), '2011-06-19 00:00:00', false),
68-
array('* * * * 1', strtotime('2011-06-15 23:09:00'), '2011-06-20 00:00:00', false),
66+
['* * * * 0', strtotime('2011-06-15 23:09:00'), '2011-06-19 00:00:00', false],
67+
['* * * * 7', strtotime('2011-06-15 23:09:00'), '2011-06-19 00:00:00', false],
68+
['* * * * 1', strtotime('2011-06-15 23:09:00'), '2011-06-20 00:00:00', false],
6969
// Should return the sunday date as 7 equals 0
70-
array('0 0 * * MON,SUN', strtotime('2011-06-15 23:09:00'), '2011-06-19 00:00:00', false),
71-
array('0 0 * * 1,7', strtotime('2011-06-15 23:09:00'), '2011-06-19 00:00:00', false),
72-
array('0 0 * * 0-4', strtotime('2011-06-15 23:09:00'), '2011-06-16 00:00:00', false),
73-
array('0 0 * * 7-4', strtotime('2011-06-15 23:09:00'), '2011-06-16 00:00:00', false),
74-
array('0 0 * * 4-7', strtotime('2011-06-15 23:09:00'), '2011-06-16 00:00:00', false),
75-
array('0 0 * * 7-3', strtotime('2011-06-15 23:09:00'), '2011-06-19 00:00:00', false),
76-
array('0 0 * * 3-7', strtotime('2011-06-15 23:09:00'), '2011-06-16 00:00:00', false),
77-
array('0 0 * * 3-7', strtotime('2011-06-18 23:09:00'), '2011-06-19 00:00:00', false),
70+
['0 0 * * MON,SUN', strtotime('2011-06-15 23:09:00'), '2011-06-19 00:00:00', false],
71+
['0 0 * * 1,7', strtotime('2011-06-15 23:09:00'), '2011-06-19 00:00:00', false],
72+
['0 0 * * 0-4', strtotime('2011-06-15 23:09:00'), '2011-06-16 00:00:00', false],
73+
['0 0 * * 7-4', strtotime('2011-06-15 23:09:00'), '2011-06-16 00:00:00', false],
74+
['0 0 * * 4-7', strtotime('2011-06-15 23:09:00'), '2011-06-16 00:00:00', false],
75+
['0 0 * * 7-3', strtotime('2011-06-15 23:09:00'), '2011-06-19 00:00:00', false],
76+
['0 0 * * 3-7', strtotime('2011-06-15 23:09:00'), '2011-06-16 00:00:00', false],
77+
['0 0 * * 3-7', strtotime('2011-06-18 23:09:00'), '2011-06-19 00:00:00', false],
7878
// Test lists of values and ranges (Abhoryo)
79-
array('0 0 * * 2-7', strtotime('2011-06-20 23:09:00'), '2011-06-21 00:00:00', false),
80-
array('0 0 * * 0,2-6', strtotime('2011-06-20 23:09:00'), '2011-06-21 00:00:00', false),
81-
array('0 0 * * 2-7', strtotime('2011-06-18 23:09:00'), '2011-06-19 00:00:00', false),
82-
array('0 0 * * 4-7', strtotime('2011-07-19 00:00:00'), '2011-07-21 00:00:00', false),
79+
['0 0 * * 2-7', strtotime('2011-06-20 23:09:00'), '2011-06-21 00:00:00', false],
80+
['0 0 * * 0,2-6', strtotime('2011-06-20 23:09:00'), '2011-06-21 00:00:00', false],
81+
['0 0 * * 2-7', strtotime('2011-06-18 23:09:00'), '2011-06-19 00:00:00', false],
82+
['0 0 * * 4-7', strtotime('2011-07-19 00:00:00'), '2011-07-21 00:00:00', false],
8383
// Test increments of ranges
84-
array('0-12/4 * * * *', strtotime('2011-06-20 12:04:00'), '2011-06-20 12:04:00', true),
85-
array('4-59/2 * * * *', strtotime('2011-06-20 12:04:00'), '2011-06-20 12:04:00', true),
86-
array('4-59/2 * * * *', strtotime('2011-06-20 12:06:00'), '2011-06-20 12:06:00', true),
87-
array('4-59/3 * * * *', strtotime('2011-06-20 12:06:00'), '2011-06-20 12:07:00', false),
84+
['0-12/4 * * * *', strtotime('2011-06-20 12:04:00'), '2011-06-20 12:04:00', true],
85+
['4-59/2 * * * *', strtotime('2011-06-20 12:04:00'), '2011-06-20 12:04:00', true],
86+
['4-59/2 * * * *', strtotime('2011-06-20 12:06:00'), '2011-06-20 12:06:00', true],
87+
['4-59/3 * * * *', strtotime('2011-06-20 12:06:00'), '2011-06-20 12:07:00', false],
8888
//array('0 0 * * 0,2-6', strtotime('2011-06-20 23:09:00'), '2011-06-21 00:00:00', false),
8989
// Test Day of the Week and the Day of the Month (issue #1)
90-
array('0 0 1 1 0', strtotime('2011-06-15 23:09:00'), '2012-01-01 00:00:00', false),
91-
array('0 0 1 JAN 0', strtotime('2011-06-15 23:09:00'), '2012-01-01 00:00:00', false),
92-
array('0 0 1 * 0', strtotime('2011-06-15 23:09:00'), '2012-01-01 00:00:00', false),
93-
array('0 0 L * *', strtotime('2011-07-15 00:00:00'), '2011-07-31 00:00:00', false),
90+
['0 0 1 1 0', strtotime('2011-06-15 23:09:00'), '2012-01-01 00:00:00', false],
91+
['0 0 1 JAN 0', strtotime('2011-06-15 23:09:00'), '2012-01-01 00:00:00', false],
92+
['0 0 1 * 0', strtotime('2011-06-15 23:09:00'), '2012-01-01 00:00:00', false],
93+
['0 0 L * *', strtotime('2011-07-15 00:00:00'), '2011-07-31 00:00:00', false],
9494
// Test the W day of the week modifier for day of the month field
95-
array('0 0 2W * *', strtotime('2011-07-01 00:00:00'), '2011-07-01 00:00:00', true),
96-
array('0 0 1W * *', strtotime('2011-05-01 00:00:00'), '2011-05-02 00:00:00', false),
97-
array('0 0 1W * *', strtotime('2011-07-01 00:00:00'), '2011-07-01 00:00:00', true),
98-
array('0 0 3W * *', strtotime('2011-07-01 00:00:00'), '2011-07-04 00:00:00', false),
99-
array('0 0 16W * *', strtotime('2011-07-01 00:00:00'), '2011-07-15 00:00:00', false),
100-
array('0 0 28W * *', strtotime('2011-07-01 00:00:00'), '2011-07-28 00:00:00', false),
101-
array('0 0 30W * *', strtotime('2011-07-01 00:00:00'), '2011-07-29 00:00:00', false),
102-
array('0 0 31W * *', strtotime('2011-07-01 00:00:00'), '2011-07-29 00:00:00', false),
95+
['0 0 2W * *', strtotime('2011-07-01 00:00:00'), '2011-07-01 00:00:00', true],
96+
['0 0 1W * *', strtotime('2011-05-01 00:00:00'), '2011-05-02 00:00:00', false],
97+
['0 0 1W * *', strtotime('2011-07-01 00:00:00'), '2011-07-01 00:00:00', true],
98+
['0 0 3W * *', strtotime('2011-07-01 00:00:00'), '2011-07-04 00:00:00', false],
99+
['0 0 16W * *', strtotime('2011-07-01 00:00:00'), '2011-07-15 00:00:00', false],
100+
['0 0 28W * *', strtotime('2011-07-01 00:00:00'), '2011-07-28 00:00:00', false],
101+
['0 0 30W * *', strtotime('2011-07-01 00:00:00'), '2011-07-29 00:00:00', false],
102+
['0 0 31W * *', strtotime('2011-07-01 00:00:00'), '2011-07-29 00:00:00', false],
103103
// Test the year field
104-
array('* * * * * 2012', strtotime('2011-05-01 00:00:00'), '2012-01-01 00:00:00', false),
104+
['* * * * * 2012', strtotime('2011-05-01 00:00:00'), '2012-01-01 00:00:00', false],
105105
// Test the last weekday of a month
106-
array('* * * * 5L', strtotime('2011-07-01 00:00:00'), '2011-07-29 00:00:00', false),
107-
array('* * * * 6L', strtotime('2011-07-01 00:00:00'), '2011-07-30 00:00:00', false),
108-
array('* * * * 7L', strtotime('2011-07-01 00:00:00'), '2011-07-31 00:00:00', false),
109-
array('* * * * 1L', strtotime('2011-07-24 00:00:00'), '2011-07-25 00:00:00', false),
110-
array('* * * * TUEL', strtotime('2011-07-24 00:00:00'), '2011-07-26 00:00:00', false),
111-
array('* * * 1 5L', strtotime('2011-12-25 00:00:00'), '2012-01-27 00:00:00', false),
106+
['* * * * 5L', strtotime('2011-07-01 00:00:00'), '2011-07-29 00:00:00', false],
107+
['* * * * 6L', strtotime('2011-07-01 00:00:00'), '2011-07-30 00:00:00', false],
108+
['* * * * 7L', strtotime('2011-07-01 00:00:00'), '2011-07-31 00:00:00', false],
109+
['* * * * 1L', strtotime('2011-07-24 00:00:00'), '2011-07-25 00:00:00', false],
110+
['* * * * TUEL', strtotime('2011-07-24 00:00:00'), '2011-07-26 00:00:00', false],
111+
['* * * 1 5L', strtotime('2011-12-25 00:00:00'), '2012-01-27 00:00:00', false],
112112
// Test the hash symbol for the nth weekday of a given month
113-
array('* * * * 5#2', strtotime('2011-07-01 00:00:00'), '2011-07-08 00:00:00', false),
114-
array('* * * * 5#1', strtotime('2011-07-01 00:00:00'), '2011-07-01 00:00:00', true),
115-
array('* * * * 3#4', strtotime('2011-07-01 00:00:00'), '2011-07-27 00:00:00', false),
116-
);
113+
['* * * * 5#2', strtotime('2011-07-01 00:00:00'), '2011-07-08 00:00:00', false],
114+
['* * * * 5#1', strtotime('2011-07-01 00:00:00'), '2011-07-01 00:00:00', true],
115+
['* * * * 3#4', strtotime('2011-07-01 00:00:00'), '2011-07-27 00:00:00', false],
116+
];
117117
}
118118
}

0 commit comments

Comments
 (0)