Chamilo\CourseBundle\Repository\CAttendanceSheetRepository

Complexity

Total Complexity

3

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getUserScore()	0	17	2
A	__construct()	0	3	1

<?php

declare(strict_types=1);

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

namespace Chamilo\CourseBundle\Repository;

use Chamilo\CoreBundle\Repository\ResourceRepository;
use Chamilo\CourseBundle\Entity\CAttendanceSheet;
use Doctrine\Persistence\ManagerRegistry;

final class CAttendanceSheetRepository extends ResourceRepository
{
    public function __construct(ManagerRegistry $registry)
    {
        parent::__construct($registry, CAttendanceSheet::class);
    }

    public function getUserScore(int $userId, int $attendanceId, ?int $groupId = null): int
    {
        $qb = $this->createQueryBuilder('sheet')
            ->select('SUM(sheet.presence) as score')
            ->join('sheet.attendanceCalendar', 'calendar')
            ->where('calendar.attendance = :attendanceId')
            ->andWhere('sheet.user = :userId')
            ->setParameter('attendanceId', $attendanceId)
            ->setParameter('userId', $userId);

        if ($groupId) {

The expression $groupId of type integer|null is loosely compared to true; this is ambiguous if the integer can be 0. You might want to explicitly use !== null instead.

            $qb->join('calendar.groups', 'groups')
                ->andWhere('groups.group = :groupId')
                ->setParameter('groupId', $groupId);
        }

        return (int) $qb->getQuery()->getSingleScalarResult();
    }
}