Passed
Push — master ( 91c768...85bc68 )
by
unknown
05:40
created

StorageScanner::__construct()   A

Complexity

Conditions 2
Paths 2

Size

Total Lines 16
Code Lines 8

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 7
CRAP Score 2.0078

Importance

Changes 1
Bugs 0 Features 0
Metric Value
eloc 8
c 1
b 0
f 0
dl 0
loc 16
ccs 7
cts 8
cp 0.875
rs 10
cc 2
nc 2
nop 2
crap 2.0078
1
<?php
2
/**
3
 * CMS Pico - Create websites using Pico CMS for Nextcloud.
4
 *
5
 * @copyright Copyright (c) 2022, Daniel Rudolf (<[email protected]>)
6
 *
7
 * @license GNU AGPL version 3 or any later version
8
 *
9
 * This program is free software: you can redistribute it and/or modify
10
 * it under the terms of the GNU Affero General Public License as
11
 * published by the Free Software Foundation, either version 3 of the
12
 * License, or (at your option) any later version.
13
 *
14
 * This program is distributed in the hope that it will be useful,
15
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
16
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
17
 * GNU Affero General Public License for more details.
18
 *
19
 * You should have received a copy of the GNU Affero General Public License
20
 * along with this program.  If not, see <http://www.gnu.org/licenses/>.
21
 */
22
23
declare(strict_types=1);
24
25
namespace OCA\CMSPico\Files;
26
27
use OC\Files\Utils\Scanner;
28
use OC\ForbiddenException;
29
use OCP\EventDispatcher\IEventDispatcher;
30
use OCP\Files\NotFoundException;
31
use OCP\Files\NotPermittedException;
32
use OCP\IDBConnection;
33
use OCP\ILogger;
34
use Psr\Log\LoggerInterface;
35
36
class StorageScanner
37
{
38
	/** @var Scanner */
39
	private $scanner;
40
41
	/**
42
	 * @param IDBConnection    $connection
43
	 * @param IEventDispatcher $eventDispatcher
44
	 */
45 1
	public function __construct(IDBConnection $connection, IEventDispatcher $eventDispatcher)
46
	{
47 1
		$this->connection = \OC::$server->query(IDBConnection::class);
0 ignored issues
show
Deprecated Code introduced by
The function OC\ServerContainer::query() has been deprecated: 20.0.0 use \Psr\Container\ContainerInterface::get ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-deprecated  annotation

47
		$this->connection = /** @scrutinizer ignore-deprecated */ \OC::$server->query(IDBConnection::class);

This function has been deprecated. The supplier of the function has supplied an explanatory message.

The explanatory message should give you some clue as to whether and when the function will be removed and what other function to use instead.

Loading history...
Bug Best Practice introduced by
The property connection does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.
Loading history...
48 1
		$this->eventDispatcher = \OC::$server->query(IEventDispatcher::class);
0 ignored issues
show
Bug Best Practice introduced by
The property eventDispatcher does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.
Loading history...
Deprecated Code introduced by
The function OC\ServerContainer::query() has been deprecated: 20.0.0 use \Psr\Container\ContainerInterface::get ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-deprecated  annotation

48
		$this->eventDispatcher = /** @scrutinizer ignore-deprecated */ \OC::$server->query(IEventDispatcher::class);

This function has been deprecated. The supplier of the function has supplied an explanatory message.

The explanatory message should give you some clue as to whether and when the function will be removed and what other function to use instead.

Loading history...
49
50
		// TODO >= NC 24: Remove version switch
51 1
		[ $majorVersion ] = \OC_Util::getVersion();
52 1
		if ($majorVersion >= 24) {
53
			/** @var LoggerInterface $logger */
54
			$logger = \OC::$server->query(LoggerInterface::class);
0 ignored issues
show
Deprecated Code introduced by
The function OC\ServerContainer::query() has been deprecated: 20.0.0 use \Psr\Container\ContainerInterface::get ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-deprecated  annotation

54
			$logger = /** @scrutinizer ignore-deprecated */ \OC::$server->query(LoggerInterface::class);

This function has been deprecated. The supplier of the function has supplied an explanatory message.

The explanatory message should give you some clue as to whether and when the function will be removed and what other function to use instead.

Loading history...
55
		} else {
56
			/** @var ILogger $logger */
57 1
			$logger = \OC::$server->query(ILogger::class);
0 ignored issues
show
Deprecated Code introduced by
The function OC\ServerContainer::query() has been deprecated: 20.0.0 use \Psr\Container\ContainerInterface::get ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-deprecated  annotation

57
			$logger = /** @scrutinizer ignore-deprecated */ \OC::$server->query(ILogger::class);

This function has been deprecated. The supplier of the function has supplied an explanatory message.

The explanatory message should give you some clue as to whether and when the function will be removed and what other function to use instead.

Loading history...
58
		}
59
60 1
		$this->scanner = new Scanner(null, $connection, $eventDispatcher, $logger);
0 ignored issues
show
Bug introduced by
It seems like $logger can also be of type Psr\Log\LoggerInterface; however, parameter $logger of OC\Files\Utils\Scanner::__construct() does only seem to accept OCP\ILogger, maybe add an additional type check? ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

60
		$this->scanner = new Scanner(null, $connection, $eventDispatcher, /** @scrutinizer ignore-type */ $logger);
Loading history...
61 1
	}
62
63
	/**
64
	 * @param string $path
65
	 * @param bool   $recursive
66
	 *
67
	 * @throws NotPermittedException
68
	 * @throws NotFoundException
69
	 */
70 21
	public function scan(string $path, bool $recursive = FolderInterface::SYNC_RECURSIVE): void
71
	{
72
		try {
73 21
			$this->scanner->scan($path, $recursive);
74
		} catch (ForbiddenException $e) {
75
			throw new NotPermittedException();
76
		}
77 21
	}
78
}
79