Conditions | 1 |
Paths | 1 |
Total Lines | 59 |
Code Lines | 54 |
Lines | 0 |
Ratio | 0 % |
Changes | 5 | ||
Bugs | 0 | Features | 1 |
Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.
For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.
Commonly applied refactorings include:
If many parameters/temporary variables are present:
1 | <?php |
||
85 | public static function frequencyProvider(): array |
||
86 | { |
||
87 | return [ |
||
88 | [function() { return self::task(); }, '* * * * *'], |
||
89 | [function() { return self::task()->minutes(37)->cron('0 0,12 1 */2 *'); }, '0 0,12 1 */2 *'], |
||
90 | [function() { return self::task()->weekly()->everyMinute(); }, '* * * * *'], |
||
91 | [function() { return self::task()->weekly()->everyFiveMinutes(); }, '*/5 * * * *'], |
||
92 | [function() { return self::task()->weekly()->everyTenMinutes(); }, '*/10 * * * *'], |
||
93 | [function() { return self::task()->weekly()->everyFifteenMinutes(); }, '*/15 * * * *'], |
||
94 | [function() { return self::task()->weekly()->everyTwentyMinutes(); }, '*/20 * * * *'], |
||
95 | [function() { return self::task()->weekly()->everyThirtyMinutes(); }, '0,30 * * * *'], |
||
96 | [function() { return self::task()->minutes(37)->hourly(); }, '0 * * * *'], |
||
97 | [function() { return self::task()->minutes(37)->hourlyAt(2); }, '2 * * * *'], |
||
98 | [function() { return self::task()->minutes(37)->hourlyAt(2, 3, '4-5'); }, '2,3,4-5 * * * *'], |
||
99 | [function() { return self::task()->minutes(37)->daily(); }, '0 0 * * *'], |
||
100 | [function() { return self::task()->minutes(37)->dailyOn(2, 3, '4-5'); }, '0 2,3,4-5 * * *'], |
||
101 | [function() { return self::task()->minutes(37)->dailyBetween(9, 17); }, '0 9-17 * * *'], |
||
102 | [function() { return self::task()->minutes(37)->twiceDaily(); }, '0 1,13 * * *'], |
||
103 | [function() { return self::task()->minutes(37)->twiceDaily(2, 14); }, '0 2,14 * * *'], |
||
104 | [function() { return self::task()->minutes(37)->dailyAt(2); }, '0 2 * * *'], |
||
105 | [function() { return self::task()->minutes(37)->dailyAt('1:34'); }, '34 1 * * *'], |
||
106 | [function() { return self::task()->minutes(37)->weekly(); }, '0 0 * * 0'], |
||
107 | [function() { return self::task()->minutes(37)->weeklyOn(2, 3, '4-5'); }, '0 0 * * 2,3,4-5'], |
||
108 | [function() { return self::task()->minutes(37)->monthly(); }, '0 0 1 * *'], |
||
109 | [function() { return self::task()->minutes(37)->monthlyOn(2, 3, '4-5'); }, '0 0 2,3,4-5 * *'], |
||
110 | [function() { return self::task()->minutes(37)->twiceMonthly(); }, '0 0 1,16 * *'], |
||
111 | [function() { return self::task()->minutes(37)->twiceMonthly(3, 17); }, '0 0 3,17 * *'], |
||
112 | [function() { return self::task()->minutes(37)->quarterly(); }, '0 0 1 */3 *'], |
||
113 | [function() { return self::task()->minutes(37)->yearly(); }, '0 0 1 1 *'], |
||
114 | [function() { return self::task()->weekly()->minutes(2, 3, '4-5'); }, '2,3,4-5 0 * * 0'], |
||
115 | [function() { return self::task()->weekly()->hours(2, 3, '4-5'); }, '0 2,3,4-5 * * 0'], |
||
116 | [function() { return self::task()->weekly()->daysOfMonth(2, 3, '4-5'); }, '0 0 2,3,4-5 * 0'], |
||
117 | [function() { return self::task()->weekly()->months(2, 3, '4-5'); }, '0 0 * 2,3,4-5 0'], |
||
118 | [function() { return self::task()->monthly()->daysOfWeek(2, 3, '4-5'); }, '0 0 1 * 2,3,4-5'], |
||
119 | [function() { return self::task()->minutes(37)->weekdays(); }, '37 * * * 1-5'], |
||
120 | [function() { return self::task()->minutes(37)->weekends(); }, '37 * * * 0,6'], |
||
121 | [function() { return self::task()->minutes(37)->mondays(); }, '37 * * * 1'], |
||
122 | [function() { return self::task()->minutes(37)->tuesdays(); }, '37 * * * 2'], |
||
123 | [function() { return self::task()->minutes(37)->wednesdays(); }, '37 * * * 3'], |
||
124 | [function() { return self::task()->minutes(37)->thursdays(); }, '37 * * * 4'], |
||
125 | [function() { return self::task()->minutes(37)->fridays(); }, '37 * * * 5'], |
||
126 | [function() { return self::task()->minutes(37)->saturdays(); }, '37 * * * 6'], |
||
127 | [function() { return self::task()->minutes(37)->sundays(); }, '37 * * * 0'], |
||
128 | [function() { return self::task()->weekly()->at(1); }, '0 1 * * 0'], |
||
129 | [function() { return self::task()->weekly()->at('2:45'); }, '45 2 * * 0'], |
||
130 | |||
131 | [function() { return self::task()->cron('invalid...')->mondays(); }, '* * * * 1'], |
||
132 | |||
133 | [function() { return self::task()->cron('@hourly'); }, '@hourly'], |
||
134 | [function() { return self::task()->cron('@daily'); }, '@daily'], |
||
135 | [function() { return self::task()->cron('@weekly'); }, '@weekly'], |
||
136 | [function() { return self::task()->cron('@monthly'); }, '@monthly'], |
||
137 | [function() { return self::task()->cron('@yearly'); }, '@yearly'], |
||
138 | [function() { return self::task()->cron('@annually'); }, '@annually'], |
||
139 | |||
140 | [function() { return self::task('my task')->cron('# 0 * * *'); }, '56 0 * * *'], |
||
141 | [function() { return self::task('my task')->cron('#daily'); }, '56 20 * * *'], |
||
142 | [function() { return self::task('my task')->cron('#midnight'); }, '56 2 * * *'], |
||
143 | [function() { return self::task('my task')->cron('#midnight')->daily(); }, '0 0 * * *'], |
||
144 | ]; |
||
215 |
The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g.
excluded_paths: ["lib/*"]
, you can move it to the dependency path list as follows:For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths