Passed
Push — master ( 48f051...d93b00 )
by Morris
14:34 queued 12s
created

LookupServerSendCheck::run()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 2
Code Lines 1

Duplication

Lines 0
Ratio 0 %

Importance

Changes 1
Bugs 0 Features 0
Metric Value
cc 1
eloc 1
nc 1
nop 1
dl 0
loc 2
rs 10
c 1
b 0
f 0
1
<?php
2
3
declare(strict_types=1);
4
5
/**
6
 * @copyright Copyright (c) 2021 Roeland Jago Douma <[email protected]>
7
 *
8
 * @author Roeland Jago Douma <[email protected]>
9
 *
10
 * @license GNU AGPL version 3 or any later version
11
 *
12
 * This program is free software: you can redistribute it and/or modify
13
 * it under the terms of the GNU Affero General Public License as
14
 * published by the Free Software Foundation, either version 3 of the
15
 * License, or (at your option) any later version.
16
 *
17
 * This program is distributed in the hope that it will be useful,
18
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
19
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
20
 * GNU Affero General Public License for more details.
21
 *
22
 * You should have received a copy of the GNU Affero General Public License
23
 * along with this program. If not, see <http://www.gnu.org/licenses/>.
24
 *
25
 */
26
27
namespace OC\Repair\NC22;
28
29
use OC\Core\BackgroundJobs\LookupServerSendCheckBackgroundJob;
30
use OCP\BackgroundJob\IJobList;
31
use OCP\IConfig;
32
use OCP\Migration\IOutput;
33
use OCP\Migration\IRepairStep;
34
35
class LookupServerSendCheck implements IRepairStep {
36
37
	/** @var IJobList */
38
	private $jobList;
39
40
	/** @var IConfig */
41
	private $config;
42
43
	public function __construct(IJobList $jobList, IConfig $config) {
44
		$this->jobList = $jobList;
45
		$this->config = $config;
46
	}
47
48
	public function getName(): string {
49
		return 'Add background job to set the lookup server share state for users';
50
	}
51
52
	private function shouldRun(): bool {
0 ignored issues
show
Unused Code introduced by
The method shouldRun() is not used, and could be removed.

This check looks for private methods that have been defined, but are not used inside the class.

Loading history...
53
		$versionFromBeforeUpdate = $this->config->getSystemValue('version', '0.0.0.0');
54
55
		// was added to 22.0.0.3
56
		return version_compare($versionFromBeforeUpdate, '22.0.0.3', '<');
0 ignored issues
show
Bug Best Practice introduced by
The expression return version_compare($...pdate, '22.0.0.3', '<') could return the type integer which is incompatible with the type-hinted return boolean. Consider adding an additional type-check to rule them out.
Loading history...
57
	}
58
59
	public function run(IOutput $output): void {
60
		$this->jobList->add(LookupServerSendCheckBackgroundJob::class);
61
	}
62
}
63