FeedRepository::findOneByOriginAndReaderOptions() - Code Metrics - Inspection of "removed typehint" - treehouselabs/io-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 64827d...9ba65f )

by Peter

created 2017-10-04 11:58 UTC

FeedRepository::findOneByOriginAndReaderOptions() A

↳ Parent: FeedRepository

Complexity

Conditions	1
Paths	1

Size

Total Lines	16
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	16
ccs	0
cts	12
cp	0
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	12
nc	1
nop	2
crap	2

<?php

namespace TreeHouse\IoBundle\Entity;

use Doctrine\ORM\EntityRepository;

class FeedRepository extends EntityRepository
{
    /**
     * Returns the highest frequency value, meaning the actual value: the
     * frequency is stored as number of hours, so a higher number actually means
     * a smaller frequency.
     *
     * @return int
     */
    public function findHighestFrequencyValue()
    {
        return $this
            ->createQueryBuilder('f')
            ->select('MAX(f.frequency)')
            ->getQuery()
            ->getSingleScalarResult()
        ;
    }

    /**
     * Returns a feed that matches the given origin and reader options
     *
     * @param int $originId
     * @param array $readerOptions
     *
     * @return Feed|null
     */
    public function findOneByOriginAndReaderOptions(
        $originId,
        array $readerOptions
    ) {
        return $this
            ->createQueryBuilder('f')
            ->where('f.origin = :origin')
            ->andWhere('f.readerOptions = :readerOptions')
            ->setParameters([
                'origin' => $originId,
                'readerOptions' => json_encode($readerOptions),
            ])
            ->getQuery()
            ->getOneOrNullResult()
        ;
    }
}


1			<?php
2
3			namespace TreeHouse\IoBundle\Entity;
4
5			use Doctrine\ORM\EntityRepository;
6
7			class FeedRepository extends EntityRepository
8			{
9			/**
10			* Returns the highest frequency value, meaning the actual value: the
11			* frequency is stored as number of hours, so a higher number actually means
12			* a smaller frequency.
13			*
14			* @return int
15			*/
16			public function findHighestFrequencyValue()
17			{
18			return $this
19			->createQueryBuilder('f')
20			->select('MAX(f.frequency)')
21			->getQuery()
22			->getSingleScalarResult()
23			;
24			}
25
26			/**
27			* Returns a feed that matches the given origin and reader options
28			*
29			* @param int $originId
30			* @param array $readerOptions
31			*
32			* @return Feed\|null
33			*/
34			public function findOneByOriginAndReaderOptions(
35			$originId,
36			array $readerOptions
37			) {
38			return $this
39			->createQueryBuilder('f')
40			->where('f.origin = :origin')
41			->andWhere('f.readerOptions = :readerOptions')
42			->setParameters([
43			'origin' => $originId,
44			'readerOptions' => json_encode($readerOptions),
45			])
46			->getQuery()
47			->getOneOrNullResult()
48			;
49			}
50			}
51

treehouselabs / io-bundle

Push — master ( 64827d...9ba65f )

FeedRepository::findOneByOriginAndReaderOptions() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like