View GitHub Repository BotRiconferme

Schedule Inspection
Subscribe
Passed
Push — master ( 93b1f3...010026 )
by Daimona
02:17 queued 39s
created
Status
Category
Toggle Patches
includes/TaskManager.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch 
@@ -1,4 +1,4 @@  discard block
 
 block discarded – undo
1 
-<?php declare( strict_types=1 );
1 
+<?php declare(strict_types=1);
2 2
3 3 
 namespace BotRiconferme;
4 4 
 
@@ -72,7 +72,7 @@  discard block
 
 block discarded – undo
72 72 
 	 * @param MessageProvider $mp
73 73 
 	 * @param PageBotList $pbl
74 74 
 	 */
75 
-	public function __construct(
75 
+	public function __construct (
76 76 
 		LoggerInterface $logger,
77 77 
 		WikiGroup $wikiGroup,
78 78 
 		MessageProvider $mp,
 
@@ -97,7 +97,7 @@  discard block
 
 block discarded – undo
97 97 
 	 * @param string[] $tasks Only used in MODE_TASK and MODE_SUBTASK
98 98 
 	 * @return TaskResult
99 99 
 	 */
100 
-	public function run( string $mode, array $tasks = [] ) : TaskResult {
100 
+	public function run ( string $mode, array $tasks = [ ] ) : TaskResult {
101 101 
 		if ( $mode === self::MODE_COMPLETE ) {
102 102 
 			return $this->runTasks( self::FULL_RUN_ORDERED );
103 103 
 		}
 
@@ -113,7 +113,7 @@  discard block
 
 block discarded – undo
113 113 
 	 * @param string[] $tasks
114 114 
 	 * @return TaskResult
115 115 
 	 */
116 
-	private function runTasks( array $tasks ) : TaskResult {
116 
+	private function runTasks ( array $tasks ) : TaskResult {
117 117 
 		$res = new TaskResult( TaskResult::STATUS_GOOD );
118 118 
 		do {
119 119 
 			$res->merge( $this->runTask( current( $tasks ) ) );
 
@@ -128,7 +128,7 @@  discard block
 
 block discarded – undo
128 128 
 	 * @param string $name
129 129 
 	 * @return TaskResult
130 130 
 	 */
131 
-	protected function runTask( string $name ) : TaskResult {
131 
+	protected function runTask ( string $name ) : TaskResult {
132 132 
 		if ( !isset( self::TASKS_MAP[ $name ] ) ) {
133 133 
 			throw new InvalidArgumentException( "'$name' is not a valid task." );
134 134 
 		}
 
@@ -142,7 +142,7 @@  discard block
 
 block discarded – undo
142 142 
 	 * @param string[] $subtasks
143 143 
 	 * @return TaskResult
144 144 
 	 */
145 
-	private function runSubtasks( array $subtasks ) : TaskResult {
145 
+	private function runSubtasks ( array $subtasks ) : TaskResult {
146 146 
 		$res = new TaskResult( TaskResult::STATUS_GOOD );
147 147 
 		do {
148 148 
 			$res->merge( $this->runSubtask( current( $subtasks ) ) );
 
@@ -157,7 +157,7 @@  discard block
 
 block discarded – undo
157 157 
 	 * @param string $name
158 158 
 	 * @return TaskResult
159 159 
 	 */
160 
-	protected function runSubtask( string $name ) : TaskResult {
160 
+	protected function runSubtask ( string $name ) : TaskResult {
161 161 
 		if ( !isset( self::SUBTASKS_MAP[ $name ] ) ) {
162 162 
 			throw new InvalidArgumentException( "'$name' is not a valid subtask." );
163 163 
 		}
 
@@ -172,7 +172,7 @@  discard block
 
 block discarded – undo
172 172 
 	 * @param string $name
173 173 
 	 * @return Task
174 174 
 	 */
175 
-	private function getTaskInstance( string $name ) : Task {
175 
+	private function getTaskInstance ( string $name ) : Task {
176 176 
 		$class = self::TASKS_MAP[ $name ];
177 177 
 		return new $class(
178 178 
 			$this->logger,
 
@@ -189,7 +189,7 @@  discard block
 
 block discarded – undo
189 189 
 	 * @param string $class
190 190 
 	 * @return Subtask
191 191 
 	 */
192 
-	private function getSubtaskInstance( string $class ) : Subtask {
192 
+	private function getSubtaskInstance ( string $class ) : Subtask {
193 193 
 		return new $class(
194 194 
 			$this->logger,
195 195 
 			$this->wikiGroup,
Please login to merge, or discard this patch.