App\Serializer\Processor\LessonCompletedProcessor::process()

Complexity

Conditions	3
Paths	3

Size

Total Lines	12
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function process($object, array $data): array
    {
        if ($object instanceof LessonInterface) {
            $userLesson = $this->userLessonRepository->getOneByUserAndLesson($this->security->getUser(), $object);

It seems like $this->security->getUser() can also be of type null; however, parameter $user of App\Repository\UserLessonRepositoryInterface::getOneByUserAndLesson() does only seem to accept Symfony\Component\Security\Core\User\UserInterface, maybe add an additional type check?

            if (null === $userLesson) {
                $data['completed'] = null;
            } else {
                $data['completed'] = $userLesson->getCompleted();
            }
        }

        return $data;
    }