BeforeCreateNotificationEvent::hasNotification() - Code Metrics - Inspection of "(wip)" - ezsystems/ezpublish-kernel - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — ezp-30616 ( ab7e14...128c78 )

unknown

created 2019-06-07 11:12 UTC

BeforeCreateNotificationEvent::hasNotification() A

↳ Parent: BeforeCreateNotificationEvent

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
nc	1
nop	0
dl	0
loc	4
rs	10
c	0
b	0
f	0

<?php

declare(strict_types=1);

namespace eZ\Publish\Core\Event\Notification;

use eZ\Publish\API\Repository\Values\Notification\CreateStruct;
use eZ\Publish\API\Repository\Values\Notification\Notification;
use eZ\Publish\Core\Event\BeforeEvent;

final class BeforeCreateNotificationEvent extends BeforeEvent
{
	public const NAME = 'ezplatform.event.notification.create.before';

	/**
     * @var \eZ\Publish\API\Repository\Values\Notification\CreateStruct
     */
	private $createStruct;

	/**
     * @var \eZ\Publish\API\Repository\Values\Notification\Notification|null
     */
	private $notification;


	public function __construct(CreateStruct $createStruct)
	{
		$this->createStruct = $createStruct;
	}


	public function getCreateStruct(): CreateStruct
	{
		return $this->createStruct;
	}


	function getNotification(): ?Notification

	{
		return $this->notification;
	}


	function setNotification(?Notification $notification): void

	{
		$this->notification = $notification;
	}


	function hasNotification(): bool

	{
		return $this->notification instanceof Notification;
	}
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace eZ\Publish\Core\Event\Notification;
6
7			use eZ\Publish\API\Repository\Values\Notification\CreateStruct;
8			use eZ\Publish\API\Repository\Values\Notification\Notification;
9			use eZ\Publish\Core\Event\BeforeEvent;
10
11			final class BeforeCreateNotificationEvent extends BeforeEvent
12			{
13			public const NAME = 'ezplatform.event.notification.create.before';
14
15			/**
16			* @var \eZ\Publish\API\Repository\Values\Notification\CreateStruct
17			*/
18			private $createStruct;
19
20			/**
21			* @var \eZ\Publish\API\Repository\Values\Notification\Notification\|null
22			*/
23			private $notification;
24
25
26			public function __construct(CreateStruct $createStruct)
27			{
28			$this->createStruct = $createStruct;
29			}
30
31
32			public function getCreateStruct(): CreateStruct
33			{
34			return $this->createStruct;
35			}
36
37
38			function getNotification(): ?Notification
			0 ignored issues – show Best Practice introduced 2019-06-07 11:29 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
39			{
40			return $this->notification;
41			}
42
43
44			function setNotification(?Notification $notification): void
			0 ignored issues – show Best Practice introduced 2019-06-07 11:29 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
45			{
46			$this->notification = $notification;
47			}
48
49
50			function hasNotification(): bool
			0 ignored issues – show Best Practice introduced 2019-06-07 11:29 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
51			{
52			return $this->notification instanceof Notification;
53			}
54			}
55

ezsystems / ezpublish-kernel

Push — ezp-30616 ( ab7e14...128c78 )

BeforeCreateNotificationEvent::hasNotification() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like