EntityNotificationProcessor::getAllNotifications() - Code Metrics - Inspection of "[!!!][TASK] Change PHP classes architecture" - CuyZ/NotiZ - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#47)

by Romain

created 2018-02-13 13:57 UTC

EntityNotificationProcessor::getAllNotifications() A

↳ Parent: EntityNotificationProcessor

Complexity

Conditions	1
Paths	1

Size

Total Lines	5
Code Lines	3

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	3
nc	1
nop	0
dl	0
loc	5
rs	9.4285
c	0
b	0
f	0

<?php

/*
 * Copyright (C) 2018
 * Nathan Boiron <[email protected]>
 * Romain Canon <[email protected]>
 *
 * This file is part of the TYPO3 NotiZ project.
 * It is free software; you can redistribute it and/or modify it
 * under the terms of the GNU General Public License, either
 * version 3 of the License, or any later version.
 *
 * For the full copyright and license information, see:
 * http://www.gnu.org/licenses/gpl-3.0.html
 */

namespace CuyZ\Notiz\Core\Notification\Processor;

use CuyZ\Notiz\Core\Definition\Tree\EventGroup\Event\EventDefinition;
use CuyZ\Notiz\Core\Notification\Notification;
use CuyZ\Notiz\Domain\Repository\EntityNotificationRepository;

abstract class EntityNotificationProcessor extends NotificationProcessor
{
    /**
     * The repository needs to be injected in the child class.
     *
     * @var EntityNotificationRepository
     */
    protected $notificationRepository;

    /**
     * @param EventDefinition $definition
     * @return Notification[]
     */
    public function getNotificationsFromEventDefinition(EventDefinition $definition)
    {
        return $this->notificationRepository
            ->findFromEventDefinition($definition)
            ->toArray();
    }

    /**
     * @return Notification[]
     */
    public function getAllNotifications()
    {
        return $this->notificationRepository
            ->findAll()
            ->toArray();
    }

    /**
     * @return int
     */
    public function getTotalNumber()
    {
        return $this->notificationRepository
            ->findAll()
            ->count();
    }
}


1			<?php
2
3			/*
4			* Copyright (C) 2018
5			* Nathan Boiron <[email protected]>
6			* Romain Canon <[email protected]>
7			*
8			* This file is part of the TYPO3 NotiZ project.
9			* It is free software; you can redistribute it and/or modify it
10			* under the terms of the GNU General Public License, either
11			* version 3 of the License, or any later version.
12			*
13			* For the full copyright and license information, see:
14			* http://www.gnu.org/licenses/gpl-3.0.html
15			*/
16
17			namespace CuyZ\Notiz\Core\Notification\Processor;
18
19			use CuyZ\Notiz\Core\Definition\Tree\EventGroup\Event\EventDefinition;
20			use CuyZ\Notiz\Core\Notification\Notification;
21			use CuyZ\Notiz\Domain\Repository\EntityNotificationRepository;
22
23			abstract class EntityNotificationProcessor extends NotificationProcessor
24			{
25			/**
26			* The repository needs to be injected in the child class.
27			*
28			* @var EntityNotificationRepository
29			*/
30			protected $notificationRepository;
31
32			/**
33			* @param EventDefinition $definition
34			* @return Notification[]
35			*/
36			public function getNotificationsFromEventDefinition(EventDefinition $definition)
37			{
38			return $this->notificationRepository
39			->findFromEventDefinition($definition)
40			->toArray();
41			}
42
43			/**
44			* @return Notification[]
45			*/
46			public function getAllNotifications()
47			{
48			return $this->notificationRepository
49			->findAll()
50			->toArray();
51			}
52
53			/**
54			* @return int
55			*/
56			public function getTotalNumber()
57			{
58			return $this->notificationRepository
59			->findAll()
60			->count();
61			}
62			}
63

CuyZ / NotiZ

Pull Request — master (#47)

EntityNotificationProcessor::getAllNotifications() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like