TPermission::checkRequirements() - Code Metrics - Inspection of "- Fixing nette 2.3 compatibility" - iPublikuj/permissions - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 1932ea...b49884 )

by Adam

created 2016-12-01 23:20 UTC

TPermission::checkRequirements() B

↳ Parent: TPermission

Complexity

Conditions	4
Paths	7

Size

Total Lines	22
Code Lines	12

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	8.9197
c	0
b	0
f	0
cc	4
eloc	12
nc	7
nop	1
crap	4.016

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

namespace IPub\Permissions;

use Nette;
use Nette\Application;

use IPub;
use IPub\Permissions\Security;

/**
 * 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 $element
	 *
	 * @throws Application\ForbiddenRequestException
	 */
	public function checkRequirements($element)
	{
		$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 http://www.ipublikuj.eu
7		* @author Adam Kadlec http://www.ipublikuj.eu
8		* @package iPublikuj:Permissions!
9		* @subpackage common
10		* @since 1.0.0
11		*
12		* @date 13.10.14
13		*/
14
15		namespace IPub\Permissions;
16
17		use Nette;
18		use Nette\Application;
19
20		use IPub;
21		use IPub\Permissions\Security;
22
23		/**
24		* Helper trait
25		*
26		* @package iPublikuj:Permissions!
27		* @subpackage common
28		*
29		* @author Adam Kadlec <[email protected]>
30		*/
31	1	trait TPermission
32		{
33		/**
34		* @var Configuration
35		*/
36		protected $permissionConfiguration;
37
38		/**
39		* @var Access\ICheckRequirements
40		*/
41		protected $requirementsChecker;
42
43		/**
44		* @param Access\ICheckRequirements $requirementsChecker
45		* @param Configuration $configuration
46		*/
47		public function injectPermission(
48		Access\ICheckRequirements $requirementsChecker,
49		Configuration $configuration
50		) {
51	1	$this->requirementsChecker = $requirementsChecker;
52	1	$this->permissionConfiguration = $configuration;
53	1	}
54
55		/**
56		* @param $element
57		*
58		* @throws Application\ForbiddenRequestException
59		*/
60		public function checkRequirements($element)
61		{
62	1	$redirectUrl = $this->permissionConfiguration->getRedirectUrl([
63	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...
64		]);
65
66		try {
67	1	parent::checkRequirements($element);
68
69	1	if (!$this->requirementsChecker->isAllowed($element)) {
70	1	throw new Application\ForbiddenRequestException;
71		}
72
73	1	} catch (Application\ForbiddenRequestException $ex) {
74	1	if ($redirectUrl) {
75		$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...
76
77		} else {
78	1	throw $ex;
79		}
80		}
81	1	}
82		}
83

iPublikuj / permissions

Push — master ( 1932ea...b49884 )

TPermission::checkRequirements() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like