View GitHub Repository BotRiconferme

Schedule Inspection
Subscribe
Passed
Push — master ( db90a8...2e33cb )
by Daimona
13:14 queued 11:27
created
Status
Category
Toggle Patches
includes/TaskManager.php 1 patch
Spacing   +10 added lines, -10 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 
 
@@ -64,7 +64,7 @@  discard block
 
 block discarded – undo
64 64 
 	 * @param MessageProvider $mp
65 65 
 	 * @param PageBotList $pbl
66 66 
 	 */
67 
-	public function __construct(
67 
+	public function __construct (
68 68 
 		LoggerInterface $logger,
69 69 
 		WikiGroup $wikiGroup,
70 70 
 		MessageProvider $mp,
 
@@ -89,7 +89,7 @@  discard block
 
 block discarded – undo
89 89 
 	 * @param string[] $tasks Only used in MODE_TASK and MODE_SUBTASK
90 90 
 	 * @return TaskResult
91 91 
 	 */
92 
-	public function run( string $mode, array $tasks = [] ) : TaskResult {
92 
+	public function run ( string $mode, array $tasks = [ ] ) : TaskResult {
93 93 
 		if ( $mode === self::MODE_COMPLETE ) {
94 94 
 			return $this->runAllTasks();
95 95 
 		}
 
@@ -104,7 +104,7 @@  discard block
 
 block discarded – undo
104 104 
 	 *
105 105 
 	 * @return TaskResult
106 106 
 	 */
107 
-	protected function runAllTasks() : TaskResult {
107 
+	protected function runAllTasks () : TaskResult {
108 108 
 		$orderedList = [
109 109 
 			'update-list',
110 110 
 			'start-new',
 
@@ -121,7 +121,7 @@  discard block
 
 block discarded – undo
121 121 
 	 * @param string[] $tasks
122 122 
 	 * @return TaskResult
123 123 
 	 */
124 
-	private function runTasks( array $tasks ) : TaskResult {
124 
+	private function runTasks ( array $tasks ) : TaskResult {
125 125 
 		$res = new TaskResult( TaskResult::STATUS_GOOD );
126 126 
 		do {
127 127 
 			$res->merge( $this->runTask( current( $tasks ) ) );
 
@@ -136,7 +136,7 @@  discard block
 
 block discarded – undo
136 136 
 	 * @param string $name
137 137 
 	 * @return TaskResult
138 138 
 	 */
139 
-	protected function runTask( string $name ) : TaskResult {
139 
+	protected function runTask ( string $name ) : TaskResult {
140 140 
 		if ( !isset( self::TASKS_MAP[ $name ] ) ) {
141 141 
 			throw new \InvalidArgumentException( "'$name' is not a valid task." );
142 142 
 		}
 
@@ -150,7 +150,7 @@  discard block
 
 block discarded – undo
150 150 
 	 * @param string[] $subtasks
151 151 
 	 * @return TaskResult
152 152 
 	 */
153 
-	private function runSubtasks( array $subtasks ) : TaskResult {
153 
+	private function runSubtasks ( array $subtasks ) : TaskResult {
154 154 
 		$res = new TaskResult( TaskResult::STATUS_GOOD );
155 155 
 		do {
156 156 
 			$res->merge( $this->runSubtask( current( $subtasks ) ) );
 
@@ -165,7 +165,7 @@  discard block
 
 block discarded – undo
165 165 
 	 * @param string $name
166 166 
 	 * @return TaskResult
167 167 
 	 */
168 
-	protected function runSubtask( string $name ) : TaskResult {
168 
+	protected function runSubtask ( string $name ) : TaskResult {
169 169 
 		if ( !isset( self::SUBTASKS_MAP[ $name ] ) ) {
170 170 
 			throw new \InvalidArgumentException( "'$name' is not a valid subtask." );
171 171 
 		}
 
@@ -180,7 +180,7 @@  discard block
 
 block discarded – undo
180 180 
 	 * @param string $name
181 181 
 	 * @return Task
182 182 
 	 */
183 
-	private function getTaskInstance( string $name ) : Task {
183 
+	private function getTaskInstance ( string $name ) : Task {
184 184 
 		$class = self::TASKS_MAP[ $name ];
185 185 
 		/** @var Task $ret */
186 186 
 		$ret = new $class(
 
@@ -199,7 +199,7 @@  discard block
 
 block discarded – undo
199 199 
 	 * @param string $class
200 200 
 	 * @return Subtask
201 201 
 	 */
202 
-	private function getSubtaskInstance( string $class ) : Subtask {
202 
+	private function getSubtaskInstance ( string $class ) : Subtask {
203 203 
 		/** @var Subtask $ret */
204 204 
 		$ret = new $class(
205 205 
 			$this->logger,
Please login to merge, or discard this patch.