PushController::__construct() - Code Metrics - Inspection of "Allow devices to register for push notifications" - nextcloud/notifications - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#59)

by Joas

created 2017-01-26 11:35 UTC

PushController::__construct() A

↳ Parent: PushController

Complexity

Conditions	1
Paths	1

Size

Total Lines	10
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	10
ccs	0
cts	9
cp	0
rs	9.4285
cc	1
eloc	8
nc	1
nop	8
crap	2

How to fix Many Parameters

<?php
/**
 * @copyright Copyright (c) 2017 Joas Schilling <[email protected]>
 *
 * @license GNU AGPL version 3 or any later version
 *
 * This program is free software: you can redistribute it and/or modify
 * it under the terms of the GNU Affero General Public License as
 * published by the Free Software Foundation, either version 3 of the
 * License, or (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU Affero General Public License for more details.
 *
 * You should have received a copy of the GNU Affero General Public License
 * along with this program.  If not, see <http://www.gnu.org/licenses/>.
 *
 */

namespace OCA\Notifications\Controller;

use OC\Authentication\Exceptions\InvalidTokenException;
use OC\Authentication\Token\IProvider;
use OC\Authentication\Token\IToken;
use OC\Security\IdentityProof\Crypto;
use OC\Security\IdentityProof\Manager;
use OCP\AppFramework\Http;
use OCP\AppFramework\Http\JSONResponse;
use OCP\AppFramework\OCSController;
use OCP\DB\QueryBuilder\IQueryBuilder;
use OCP\IDBConnection;
use OCP\IRequest;
use OCP\ISession;
use OCP\IUser;
use OCP\IUserSession;

class PushController extends OCSController {

	/** @var IDBConnection */
	private $db;

	/** @var ISession */
	private $session;

	/** @var IUserSession */
	private $userSession;

	/** @var IProvider */
	private $tokenProvider;

	/** @var Manager */
	private $identityProof;

	/** @var Crypto */
	private $crypto;

	/**
	 * @param string $appName
	 * @param IRequest $request
	 * @param IDBConnection $db
	 * @param ISession $session
	 * @param IUserSession $userSession
	 * @param IProvider $tokenProvider
	 * @param Manager $identityProof
	 * @param Crypto $crypto
	 */
	public function __construct($appName, IRequest $request, IDBConnection $db, ISession $session, IUserSession $userSession, IProvider $tokenProvider, Manager $identityProof, Crypto $crypto) {
		parent::__construct($appName, $request);

		$this->db = $db;
		$this->session = $session;
		$this->userSession = $userSession;
		$this->tokenProvider = $tokenProvider;
		$this->identityProof = $identityProof;
		$this->crypto = $crypto;
	}

	/**
	 * @NoAdminRequired
	 * @NoCSRFRequired
	 *
	 * @param string $pushTokenHash
	 * @param string $devicePublicKey
	 * @return JSONResponse
	 */
	public function registerDevice($pushTokenHash, $devicePublicKey) {
		$user = $this->userSession->getUser();
		if (!$user instanceof IUser) {
if ($x instanceof DoesNotExist) {
    // Do something.
}
			return new JSONResponse([], Http::STATUS_UNAUTHORIZED);
		}

		if (!preg_match('/^([a-f0-9]{128})$/', $pushTokenHash)) {
			return new JSONResponse(['message' => 'Invalid hashed push token'], Http::STATUS_BAD_REQUEST);
		}

		if (strlen($devicePublicKey) !== 450 ||

			strpos($devicePublicKey, '-----BEGIN PUBLIC KEY-----') !== 0 ||
			strpos($devicePublicKey, '-----END PUBLIC KEY-----') !== 426) {
			return new JSONResponse(['message' => 'Invalid device public key'], Http::STATUS_BAD_REQUEST);
		}

		$tokenId = $this->session->get('token-id');
		try {
			$token = $this->tokenProvider->getTokenById($tokenId);
		} catch (InvalidTokenException $e) {

			return new JSONResponse(['message' => 'Could not identify session token'], Http::STATUS_BAD_REQUEST);
		}

		$key = $this->identityProof->getKey($user);

		try {
			$created = $this->savePushToken($user, $token, $devicePublicKey, $pushTokenHash);
		} catch (\BadMethodCallException $e) {
			return new JSONResponse(['message' => 'Invalid device public key'], Http::STATUS_BAD_REQUEST);
		}

		$encryptedData = $this->crypto->encrypt(sha1(json_encode([$user->getCloudId(), $token->getId()])), $user);
		return new JSONResponse([
			'publicKey' => $key->getPublic(),
			'deviceIdentifier' => $encryptedData['message'],
			'signature' => base64_encode($encryptedData['signature']),
		], $created ? Http::STATUS_CREATED : Http::STATUS_OK);
	}

	/**
	 * @NoAdminRequired
	 * @NoCSRFRequired
	 *
	 * @param string $devicePublicKey
	 * @return JSONResponse
	 */
	public function removeDevice($devicePublicKey) {
		$user = $this->userSession->getUser();
		if (!$user instanceof IUser) {
if ($x instanceof DoesNotExist) {
    // Do something.
}
			return new JSONResponse([], Http::STATUS_UNAUTHORIZED);
		}

		if (strlen($devicePublicKey) !== 450 ||

			strpos($devicePublicKey, '-----BEGIN PUBLIC KEY-----') !== 0 ||
			strpos($devicePublicKey, '-----END PUBLIC KEY-----') !== 426) {
			return new JSONResponse(['message' => 'Invalid device public key'], Http::STATUS_BAD_REQUEST);
		}

		$sessionId = $this->session->getId();
		try {
			$token = $this->tokenProvider->getToken($sessionId);
		} catch (InvalidTokenException $e) {

			return new JSONResponse(['message' => 'Could not identify session token'], Http::STATUS_BAD_REQUEST);
		}

		try {
			$this->deletePushToken($user, $token, $devicePublicKey);
		} catch (\BadMethodCallException $e) {
			return new JSONResponse(['message' => 'Invalid device public key'], Http::STATUS_BAD_REQUEST);
		}

		return new JSONResponse();
	}

	/**
	 * @param IUser $user
	 * @param IToken $token
	 * @param string $devicePublicKey
	 * @param string $pushTokenHash
	 * @return bool If the hash was new to the database
	 * @throws \BadMethodCallException
	 */
	protected function savePushToken(IUser $user, IToken $token, $devicePublicKey, $pushTokenHash) {
		$query = $this->db->getQueryBuilder();
		$query->select('pushtokenhash')
			->from('notifications_pushtokens')
			->where($query->expr()->eq('uid', $query->createNamedParameter($user->getUID())))
			->andWhere($query->expr()->eq('token', $query->createNamedParameter($token->getId())))
			->andWhere($query->expr()->eq('devicepublickey', $query->createNamedParameter($devicePublicKey)));
		$result = $query->execute();
		$row = $result->fetch();
		$result->closeCursor();

		if (!$row) {
			return $this->insertPushToken($user, $token, $devicePublicKey, $pushTokenHash);
		} else if ($row['pushtokenhash'] !== $pushTokenHash) {
			return $this->updatePushToken($user, $token, $devicePublicKey, $pushTokenHash);
		}
		return false;
	}

	/**
	 * @param IUser $user
	 * @param IToken $token
	 * @param string $devicePublicKey
	 * @param string $pushTokenHash
	 * @return bool If the entry was created
	 */
	protected function insertPushToken(IUser $user, IToken $token, $devicePublicKey, $pushTokenHash) {
		$devicePublicKeyHash = hash('sha512', $devicePublicKey);

		$query = $this->db->getQueryBuilder();
		$query->insert('notifications_pushtokens')
			->values([
				'uid' => $query->createNamedParameter($user->getUID()),
				'token' => $query->createNamedParameter($token->getId(), IQueryBuilder::PARAM_INT),
				'devicepublickey' => $query->createNamedParameter($devicePublicKey),
				'devicepublickeyhash' => $query->createNamedParameter($devicePublicKeyHash),
				'pushtokenhash' => $query->createNamedParameter($pushTokenHash),
			]);
		return $query->execute() > 0;
	}

	/**
	 * @param IUser $user
	 * @param IToken $token
	 * @param string $devicePublicKey
	 * @param string $pushTokenHash
	 * @return bool If the entry was updated
	 * @throws \BadMethodCallException
	 */
	protected function updatePushToken(IUser $user, IToken $token, $devicePublicKey, $pushTokenHash) {

		$devicePublicKeyHash = hash('sha512', $devicePublicKey);

		$query = $this->db->getQueryBuilder();
		$query->update('notifications_pushtokens')
			->set('pushtokenhash', $query->createNamedParameter($pushTokenHash))
			->where($query->expr()->eq('uid', $query->createNamedParameter($user->getUID())))
			->andWhere($query->expr()->eq('token', $query->createNamedParameter($token->getId(), IQueryBuilder::PARAM_INT)))
			->andWhere($query->expr()->eq('devicepublickeyhash', $query->createNamedParameter($devicePublicKeyHash)));

		if ($query->execute() !== 0) {
			throw new \BadMethodCallException();
		}

		return true;
	}

	/**
	 * @param IUser $user
	 * @param IToken $token
	 * @param string $devicePublicKey
	 * @return bool If the entry was deleted
	 * @throws \BadMethodCallException
	 */
	protected function deletePushToken(IUser $user, IToken $token, $devicePublicKey) {

		$devicePublicKeyHash = hash('sha512', $devicePublicKey);

		$query = $this->db->getQueryBuilder();
		$query->delete('notifications_pushtokens')
			->where($query->expr()->eq('uid', $query->createNamedParameter($user->getUID())))
			->andWhere($query->expr()->eq('token', $query->createNamedParameter($token->getId(), IQueryBuilder::PARAM_INT)))
			->andWhere($query->expr()->eq('devicepublickeyhash', $query->createNamedParameter($devicePublicKeyHash)));

		if ($query->execute() !== 0) {
			throw new \BadMethodCallException();
		}

		return true;
	}
}


Pull Request — master (#59)

PushController::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Many Parameters

Many Parameters

1. Missing dependencies

2. Missing use statement

1. Missing dependencies

2. Missing use statement

1		<?php
2		/**
3		* @copyright Copyright (c) 2017 Joas Schilling <[email protected]>
4		*
5		* @license GNU AGPL version 3 or any later version
6		*
7		* This program is free software: you can redistribute it and/or modify
8		* it under the terms of the GNU Affero General Public License as
9		* published by the Free Software Foundation, either version 3 of the
10		* License, or (at your option) any later version.
11		*
12		* This program is distributed in the hope that it will be useful,
13		* but WITHOUT ANY WARRANTY; without even the implied warranty of
14		* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
15		* GNU Affero General Public License for more details.
16		*
17		* You should have received a copy of the GNU Affero General Public License
18		* along with this program. If not, see <http://www.gnu.org/licenses/>.
19		*
20		*/
21
22		namespace OCA\Notifications\Controller;
23
24		use OC\Authentication\Exceptions\InvalidTokenException;
25		use OC\Authentication\Token\IProvider;
26		use OC\Authentication\Token\IToken;
27		use OC\Security\IdentityProof\Crypto;
28		use OC\Security\IdentityProof\Manager;
29		use OCP\AppFramework\Http;
30		use OCP\AppFramework\Http\JSONResponse;
31		use OCP\AppFramework\OCSController;
32		use OCP\DB\QueryBuilder\IQueryBuilder;
33		use OCP\IDBConnection;
34		use OCP\IRequest;
35		use OCP\ISession;
36		use OCP\IUser;
37		use OCP\IUserSession;
38
39		class PushController extends OCSController {
40
41		/** @var IDBConnection */
42		private $db;
43
44		/** @var ISession */
45		private $session;
46
47		/** @var IUserSession */
48		private $userSession;
49
50		/** @var IProvider */
51		private $tokenProvider;
52
53		/** @var Manager */
54		private $identityProof;
55
56		/** @var Crypto */
57		private $crypto;
58
59		/**
60		* @param string $appName
61		* @param IRequest $request
62		* @param IDBConnection $db
63		* @param ISession $session
64		* @param IUserSession $userSession
65		* @param IProvider $tokenProvider
66		* @param Manager $identityProof
67		* @param Crypto $crypto
68		*/
69		public function __construct($appName, IRequest $request, IDBConnection $db, ISession $session, IUserSession $userSession, IProvider $tokenProvider, Manager $identityProof, Crypto $crypto) {
70		parent::__construct($appName, $request);
71
72		$this->db = $db;
73		$this->session = $session;
74		$this->userSession = $userSession;
75		$this->tokenProvider = $tokenProvider;
76		$this->identityProof = $identityProof;
77		$this->crypto = $crypto;
78		}
79
80		/**
81		* @NoAdminRequired
82		* @NoCSRFRequired
83		*
84		* @param string $pushTokenHash
85		* @param string $devicePublicKey
86		* @return JSONResponse
87		*/
88		public function registerDevice($pushTokenHash, $devicePublicKey) {
89		$user = $this->userSession->getUser();
90		if (!$user instanceof IUser) {
		0 ignored issues – show Bug introduced 2017-01-25 17:01 UTC by Report Bug Copy Issue Report The class `OCP\IUser` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history...
91		return new JSONResponse([], Http::STATUS_UNAUTHORIZED);
92		}
93
94		if (!preg_match('/^([a-f0-9]{128})$/', $pushTokenHash)) {
95		return new JSONResponse(['message' => 'Invalid hashed push token'], Http::STATUS_BAD_REQUEST);
96		}
97
98	View Code Duplication	if (strlen($devicePublicKey) !== 450 \|\|
		0 ignored issues – show Duplication introduced 2017-01-25 17:01 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
99		strpos($devicePublicKey, '-----BEGIN PUBLIC KEY-----') !== 0 \|\|
100		strpos($devicePublicKey, '-----END PUBLIC KEY-----') !== 426) {
101		return new JSONResponse(['message' => 'Invalid device public key'], Http::STATUS_BAD_REQUEST);
102		}
103
104		$tokenId = $this->session->get('token-id');
105		try {
106		$token = $this->tokenProvider->getTokenById($tokenId);
107		} catch (InvalidTokenException $e) {
		0 ignored issues – show Bug introduced 2017-01-25 17:01 UTC by Report Bug Copy Issue Report The class `OC\Authentication\Exceptions\InvalidTokenException` does not exist. Did you forget a USE statement, or did you not list all dependencies? Scrutinizer analyzes your `composer.json`/`composer.lock` file if available to determine the classes, and functions that are defined by your dependencies. It seems like the listed class was neither found in your dependencies, nor was it found in the analyzed files in your repository. If you are using some other form of dependency management, you might want to disable this analysis. Loading history...
108		return new JSONResponse(['message' => 'Could not identify session token'], Http::STATUS_BAD_REQUEST);
109		}
110
111		$key = $this->identityProof->getKey($user);
112
113		try {
114		$created = $this->savePushToken($user, $token, $devicePublicKey, $pushTokenHash);
115		} catch (\BadMethodCallException $e) {
116		return new JSONResponse(['message' => 'Invalid device public key'], Http::STATUS_BAD_REQUEST);
117		}
118
119		$encryptedData = $this->crypto->encrypt(sha1(json_encode([$user->getCloudId(), $token->getId()])), $user);
120		return new JSONResponse([
121		'publicKey' => $key->getPublic(),
122		'deviceIdentifier' => $encryptedData['message'],
123		'signature' => base64_encode($encryptedData['signature']),
124		], $created ? Http::STATUS_CREATED : Http::STATUS_OK);
125		}
126
127		/**
128		* @NoAdminRequired
129		* @NoCSRFRequired
130		*
131		* @param string $devicePublicKey
132		* @return JSONResponse
133		*/
134		public function removeDevice($devicePublicKey) {
135		$user = $this->userSession->getUser();
136		if (!$user instanceof IUser) {
		0 ignored issues – show Bug introduced 2017-01-25 17:01 UTC by Report Bug Copy Issue Report The class `OCP\IUser` does not exist. Did you forget a USE statement, or did you not list all dependencies? This error could be the result of: 1. Missing dependencies PHP Analyzer uses your `composer.json` file (if available) to determine the dependencies of your project and to determine all the available classes and functions. It expects the `composer.json` to be in the root folder of your repository. Are you sure this class is defined by one of your dependencies, or did you maybe not list a dependency in either the `require` or `require-dev` section? 2. Missing use statement PHP does not complain about undefined classes in `ìnstanceof` checks. For example, the following PHP code will work perfectly fine: if ($x instanceof DoesNotExist) { // Do something. } If you have not tested against this specific condition, such errors might go unnoticed. Loading history...
137		return new JSONResponse([], Http::STATUS_UNAUTHORIZED);
138		}
139
140	View Code Duplication	if (strlen($devicePublicKey) !== 450 \|\|
		0 ignored issues – show Duplication introduced 2017-01-25 17:01 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
141		strpos($devicePublicKey, '-----BEGIN PUBLIC KEY-----') !== 0 \|\|
142		strpos($devicePublicKey, '-----END PUBLIC KEY-----') !== 426) {
143		return new JSONResponse(['message' => 'Invalid device public key'], Http::STATUS_BAD_REQUEST);
144		}
145
146		$sessionId = $this->session->getId();
147		try {
148		$token = $this->tokenProvider->getToken($sessionId);
149		} catch (InvalidTokenException $e) {
		0 ignored issues – show Bug introduced 2017-01-25 17:01 UTC by Report Bug Copy Issue Report The class `OC\Authentication\Exceptions\InvalidTokenException` does not exist. Did you forget a USE statement, or did you not list all dependencies? Scrutinizer analyzes your `composer.json`/`composer.lock` file if available to determine the classes, and functions that are defined by your dependencies. It seems like the listed class was neither found in your dependencies, nor was it found in the analyzed files in your repository. If you are using some other form of dependency management, you might want to disable this analysis. Loading history...
150		return new JSONResponse(['message' => 'Could not identify session token'], Http::STATUS_BAD_REQUEST);
151		}
152
153		try {
154		$this->deletePushToken($user, $token, $devicePublicKey);
155		} catch (\BadMethodCallException $e) {
156		return new JSONResponse(['message' => 'Invalid device public key'], Http::STATUS_BAD_REQUEST);
157		}
158
159		return new JSONResponse();
160		}
161
162		/**
163		* @param IUser $user
164		* @param IToken $token
165		* @param string $devicePublicKey
166		* @param string $pushTokenHash
167		* @return bool If the hash was new to the database
168		* @throws \BadMethodCallException
169		*/
170		protected function savePushToken(IUser $user, IToken $token, $devicePublicKey, $pushTokenHash) {
171		$query = $this->db->getQueryBuilder();
172		$query->select('pushtokenhash')
173		->from('notifications_pushtokens')
174		->where($query->expr()->eq('uid', $query->createNamedParameter($user->getUID())))
175		->andWhere($query->expr()->eq('token', $query->createNamedParameter($token->getId())))
176		->andWhere($query->expr()->eq('devicepublickey', $query->createNamedParameter($devicePublicKey)));
177		$result = $query->execute();
178		$row = $result->fetch();
179		$result->closeCursor();
180
181		if (!$row) {
182		return $this->insertPushToken($user, $token, $devicePublicKey, $pushTokenHash);
183		} else if ($row['pushtokenhash'] !== $pushTokenHash) {
184		return $this->updatePushToken($user, $token, $devicePublicKey, $pushTokenHash);
185		}
186		return false;
187		}
188
189		/**
190		* @param IUser $user
191		* @param IToken $token
192		* @param string $devicePublicKey
193		* @param string $pushTokenHash
194		* @return bool If the entry was created
195		*/
196		protected function insertPushToken(IUser $user, IToken $token, $devicePublicKey, $pushTokenHash) {
197		$devicePublicKeyHash = hash('sha512', $devicePublicKey);
198
199		$query = $this->db->getQueryBuilder();
200		$query->insert('notifications_pushtokens')
201		->values([
202		'uid' => $query->createNamedParameter($user->getUID()),
203		'token' => $query->createNamedParameter($token->getId(), IQueryBuilder::PARAM_INT),
204		'devicepublickey' => $query->createNamedParameter($devicePublicKey),
205		'devicepublickeyhash' => $query->createNamedParameter($devicePublicKeyHash),
206		'pushtokenhash' => $query->createNamedParameter($pushTokenHash),
207		]);
208		return $query->execute() > 0;
209		}
210
211		/**
212		* @param IUser $user
213		* @param IToken $token
214		* @param string $devicePublicKey
215		* @param string $pushTokenHash
216		* @return bool If the entry was updated
217		* @throws \BadMethodCallException
218		*/
219	View Code Duplication	protected function updatePushToken(IUser $user, IToken $token, $devicePublicKey, $pushTokenHash) {
		0 ignored issues – show Duplication introduced 2017-01-25 17:01 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
220		$devicePublicKeyHash = hash('sha512', $devicePublicKey);
221
222		$query = $this->db->getQueryBuilder();
223		$query->update('notifications_pushtokens')
224		->set('pushtokenhash', $query->createNamedParameter($pushTokenHash))
225		->where($query->expr()->eq('uid', $query->createNamedParameter($user->getUID())))
226		->andWhere($query->expr()->eq('token', $query->createNamedParameter($token->getId(), IQueryBuilder::PARAM_INT)))
227		->andWhere($query->expr()->eq('devicepublickeyhash', $query->createNamedParameter($devicePublicKeyHash)));
228
229		if ($query->execute() !== 0) {
230		throw new \BadMethodCallException();
231		}
232
233		return true;
234		}
235
236		/**
237		* @param IUser $user
238		* @param IToken $token
239		* @param string $devicePublicKey
240		* @return bool If the entry was deleted
241		* @throws \BadMethodCallException
242		*/
243	View Code Duplication	protected function deletePushToken(IUser $user, IToken $token, $devicePublicKey) {
		0 ignored issues – show Duplication introduced 2017-01-25 17:01 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
244		$devicePublicKeyHash = hash('sha512', $devicePublicKey);
245
246		$query = $this->db->getQueryBuilder();
247		$query->delete('notifications_pushtokens')
248		->where($query->expr()->eq('uid', $query->createNamedParameter($user->getUID())))
249		->andWhere($query->expr()->eq('token', $query->createNamedParameter($token->getId(), IQueryBuilder::PARAM_INT)))
250		->andWhere($query->expr()->eq('devicepublickeyhash', $query->createNamedParameter($devicePublicKeyHash)));
251
252		if ($query->execute() !== 0) {
253		throw new \BadMethodCallException();
254		}
255
256		return true;
257		}
258		}
259

nextcloud / notifications

Pull Request — master (#59)

PushController::__construct() A

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Many Parameters

Many Parameters

1. Missing dependencies

2. Missing use statement

1. Missing dependencies

2. Missing use statement

Duplication Side-by-Side

Filter issues like