TPermission::checkRequirements() - Code Metrics - iPublikuj/permissions - Measure and Improve Code Quality continuously with Scrutinizer

TPermission::checkRequirements() A
last analyzed 2018-12-21 19:08 UTC

↳ Parent: TPermission

Complexity

Conditions	4
Paths	7

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	4.016

Importance

Changes

Metric	Value
dl	0
loc	22
ccs	9
cts	10
cp	0.9
rs	9.568
c	0
b	0
f	0
cc	4
nc	7
nop	1
crap	4.016

<?php
/**
 * TPermission.php
 *
 * @copyright      More in license.md
 * @license        https://www.ipublikuj.eu
 * @author         Adam Kadlec <[email protected]>
 * @package        iPublikuj:Permissions!
 * @subpackage     common
 * @since          1.0.0
 *
 * @date           13.10.14
 */

declare(strict_types = 1);

namespace IPub\Permissions;

use Nette\Application;

/**
 * Helper trait
 *
 * @package        iPublikuj:Permissions!
 * @subpackage     common
 *
 * @author         Adam Kadlec <[email protected]>
 */
trait TPermission
{
	/**
	 * @var Configuration
	 */
	protected $permissionConfiguration;

	/**
	 * @var Access\ICheckRequirements
	 */
	protected $requirementsChecker;

	/**
	 * @param Access\ICheckRequirements $requirementsChecker
	 * @param Configuration $configuration
	 */
	public function injectPermission(
		Access\ICheckRequirements $requirementsChecker,
		Configuration $configuration
	) {
		$this->requirementsChecker = $requirementsChecker;
		$this->permissionConfiguration = $configuration;
	}

	/**
	 * @param mixed $element
	 *
	 * @return void
	 *
	 * @throws Application\ForbiddenRequestException
	 * @throws Application\UI\InvalidLinkException
	 */
	public function checkRequirements($element) : void
	{
		$redirectUrl = $this->permissionConfiguration->getRedirectUrl([
			'backlink' => $this->storeRequest(),
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
		]);

		try {
			parent::checkRequirements($element);

			if (!$this->requirementsChecker->isAllowed($element)) {
				throw new Application\ForbiddenRequestException;
			}

		} catch (Application\ForbiddenRequestException $ex) {

			if ($redirectUrl) {
				$this->getPresenter()->redirectUrl($redirectUrl);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

			} else {
				throw $ex;
			}
		}
	}
}


1		<?php
2		/**
3		* TPermission.php
4		*
5		* @copyright More in license.md
6		* @license https://www.ipublikuj.eu
7		* @author Adam Kadlec <[email protected]>
8		* @package iPublikuj:Permissions!
9		* @subpackage common
10		* @since 1.0.0
11		*
12		* @date 13.10.14
13		*/
14
15		declare(strict_types = 1);
16
17		namespace IPub\Permissions;
18
19		use Nette\Application;
20
21		/**
22		* Helper trait
23		*
24		* @package iPublikuj:Permissions!
25		* @subpackage common
26		*
27		* @author Adam Kadlec <[email protected]>
28		*/
29	1	trait TPermission
30		{
31		/**
32		* @var Configuration
33		*/
34		protected $permissionConfiguration;
35
36		/**
37		* @var Access\ICheckRequirements
38		*/
39		protected $requirementsChecker;
40
41		/**
42		* @param Access\ICheckRequirements $requirementsChecker
43		* @param Configuration $configuration
44		*/
45		public function injectPermission(
46		Access\ICheckRequirements $requirementsChecker,
47		Configuration $configuration
48		) {
49	1	$this->requirementsChecker = $requirementsChecker;
50	1	$this->permissionConfiguration = $configuration;
51	1	}
52
53		/**
54		* @param mixed $element
55		*
56		* @return void
57		*
58		* @throws Application\ForbiddenRequestException
59		* @throws Application\UI\InvalidLinkException
60		*/
61		public function checkRequirements($element) : void
62		{
63	1	$redirectUrl = $this->permissionConfiguration->getRedirectUrl([
64	1	'backlink' => $this->storeRequest(),
		0 ignored issues – show Bug introduced 2016-11-30 22:57 UTC by Report Bug Copy Issue Report It seems like `storeRequest()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
65		]);
66
67		try {
68	1	parent::checkRequirements($element);
69
70	1	if (!$this->requirementsChecker->isAllowed($element)) {
71	1	throw new Application\ForbiddenRequestException;
72		}
73
74	1	} catch (Application\ForbiddenRequestException $ex) {
		0 ignored issues – show Bug introduced 2018-03-18 15:41 UTC by Report Bug Copy Issue Report The class `Nette\Application\ForbiddenRequestException` 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...
75	1	if ($redirectUrl) {
76		$this->getPresenter()->redirectUrl($redirectUrl);
		0 ignored issues – show Bug introduced 2016-11-30 22:57 UTC by Report Bug Copy Issue Report It seems like `getPresenter()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
77
78		} else {
79	1	throw $ex;
80		}
81		}
82	1	}
83		}
84

iPublikuj / permissions

TPermission::checkRequirements() A last analyzed 2018-12-21 19:08 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

TPermission::checkRequirements() A
last analyzed 2018-12-21 19:08 UTC