PortfolioRepository::findTemplates() - Code Metrics - Inspection of "Portfolio: Fixing queries related to resource node..." - chamilo/chamilo-lms - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( d772ec...7132be )

by Angel Fernando Quiroz

created 2025-11-25 14:36 UTC

PortfolioRepository::findTemplates() A

↳ Parent: PortfolioRepository

Complexity

Conditions	1
Paths	1

Size

Total Lines	10
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
eloc	6
c	1
b	0
f	0
nc	1
nop	3
dl	0
loc	10
rs	10

<?php

/* For licensing terms, see /license.txt */

declare(strict_types=1);

namespace Chamilo\CoreBundle\Repository\Node;

use Chamilo\CoreBundle\Entity\Course;
use Chamilo\CoreBundle\Entity\Portfolio;
use Chamilo\CoreBundle\Entity\Session;
use Chamilo\CoreBundle\Entity\User;
use Chamilo\CoreBundle\Repository\ResourceRepository;
use Doctrine\Persistence\ManagerRegistry;

class PortfolioRepository extends ResourceRepository
{
    public function __construct(ManagerRegistry $registry)
    {
        parent::__construct($registry, Portfolio::class);
    }

    public function findItemsByUser(
        User $user,
        ?Course $course,
        ?Session $session,
        ?array $orderBy = null,
        array $visibility = []
    ): array {
        $criteria = [];
        $criteria['user'] = $user;

        if ($course) {
            $criteria['course'] = $course;
            $criteria['session'] = $session;
        }

        if ($visibility) {

            $criteria['visibility'] = $visibility;
        }

        return $this->findBy($criteria, $orderBy);
    }

    public function findTemplates(User $creator, ?Course $course, ?Session $session)
    {
        $qb = $this->getResourcesByCourse($course, $session);

        $this->addCreatorQueryBuilder($creator, $qb);

        return $qb
            ->andWhere($qb->expr()->eq('resource.isTemplate',true))
            ->getQuery()
            ->getResult()
        ;
    }
}


1			<?php
2
3			/* For licensing terms, see /license.txt */
4
5			declare(strict_types=1);
6
7			namespace Chamilo\CoreBundle\Repository\Node;
8
9			use Chamilo\CoreBundle\Entity\Course;
10			use Chamilo\CoreBundle\Entity\Portfolio;
11			use Chamilo\CoreBundle\Entity\Session;
12			use Chamilo\CoreBundle\Entity\User;
13			use Chamilo\CoreBundle\Repository\ResourceRepository;
14			use Doctrine\Persistence\ManagerRegistry;
15
16			class PortfolioRepository extends ResourceRepository
17			{
18			public function __construct(ManagerRegistry $registry)
19			{
20			parent::__construct($registry, Portfolio::class);
21			}
22
23			public function findItemsByUser(
24			User $user,
25			?Course $course,
26			?Session $session,
27			?array $orderBy = null,
28			array $visibility = []
29			): array {
30			$criteria = [];
31			$criteria['user'] = $user;
32
33			if ($course) {
34			$criteria['course'] = $course;
35			$criteria['session'] = $session;
36			}
37
38			if ($visibility) {
			0 ignored issues – show Bug Best Practice introduced 2022-08-16 23:58 UTC by Report Bug Copy Issue Report The expression `$visibility` of type `array` is implicitly converted to a boolean; are you sure this is intended? If so, consider using `! empty($expr)` instead to make it clear that you intend to check for an array without elements. This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history...
39			$criteria['visibility'] = $visibility;
40			}
41
42			return $this->findBy($criteria, $orderBy);
43			}
44
45			public function findTemplates(User $creator, ?Course $course, ?Session $session)
46			{
47			$qb = $this->getResourcesByCourse($course, $session);
48
49			$this->addCreatorQueryBuilder($creator, $qb);
50
51			return $qb
52			->andWhere($qb->expr()->eq('resource.isTemplate',true))
53			->getQuery()
54			->getResult()
55			;
56			}
57			}
58

chamilo / chamilo-lms

Push — master ( d772ec...7132be )

PortfolioRepository::findTemplates() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like