SubscriptionRepository::getUserSubscribersCountById() - Code Metrics - Inspection of "UsersRenamedEvent now dispatches instead of Generi..." - skobkin/point-tools - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 3db25e...88be9e )

by Alexey

created 2017-01-07 18:46 UTC

getUserSubscribersCountById() A

↳ Parent: SubscriptionRepository

Complexity

Conditions	2
Paths	2

Size

Total Lines	15
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	2.0054

Importance

Changes	2
Bugs	1	Features	0

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

<?php

namespace Skobkin\Bundle\PointToolsBundle\Repository;

use Doctrine\ORM\EntityRepository;
use Skobkin\Bundle\PointToolsBundle\Entity\User;

class SubscriptionRepository extends EntityRepository
{
    /**
     * @param int $id
     *
     * @return int
     */
    public function getUserSubscribersCountById($id): int
    {
        if (!is_int($id)) {
            throw new \InvalidArgumentException('$id must be an integer');
        }

        $qb = $this->createQueryBuilder('s');
        return $qb
            ->select('COUNT(s.subscriber)')
            ->innerJoin('s.author', 'sa')
            ->where('sa.id = :id')
            ->setParameter('id', $id)
            ->getQuery()->getSingleScalarResult()
        ;
    }

    /**
     * @param User $user
     * @param User[] $subscribers
     */
    public function removeSubscribers(User $user, array $subscribers)
    {
        $qb = $this->createQueryBuilder('s');
        $qb
            ->delete()
            ->where('s.author = :author')
            ->andWhere('s.subscriber IN (:subscribers)')
            ->setParameter('author', $user->getId())
            ->setParameter('subscribers', $subscribers)
            ->getQuery()->execute();
        ;
    }
}

1		<?php
2
3		namespace Skobkin\Bundle\PointToolsBundle\Repository;
4
5		use Doctrine\ORM\EntityRepository;
6		use Skobkin\Bundle\PointToolsBundle\Entity\User;
7
8		class SubscriptionRepository extends EntityRepository
9		{
10		/**
11		* @param int $id
12		*
13		* @return int
14		*/
15	4	public function getUserSubscribersCountById($id): int
16		{
17	4	if (!is_int($id)) {
18		throw new \InvalidArgumentException('$id must be an integer');
19		}
20
21	4	$qb = $this->createQueryBuilder('s');
22		return $qb
23	4	->select('COUNT(s.subscriber)')
24	4	->innerJoin('s.author', 'sa')
25	4	->where('sa.id = :id')
26	4	->setParameter('id', $id)
27	4	->getQuery()->getSingleScalarResult()
28		;
29		}
30
31		/**
32		* @param User $user
33		* @param User[] $subscribers
34		*/
35		public function removeSubscribers(User $user, array $subscribers)
36		{
37		$qb = $this->createQueryBuilder('s');
38		$qb
39		->delete()
40		->where('s.author = :author')
41		->andWhere('s.subscriber IN (:subscribers)')
42		->setParameter('author', $user->getId())
43		->setParameter('subscribers', $subscribers)
44		->getQuery()->execute();
45		;
46		}
47		}

skobkin / point-tools

Push — master ( 3db25e...88be9e )

getUserSubscribersCountById() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like