ProjetNormandie\ForumBundle\Service\ForumService

Complexity

Total Complexity

6

Size/Duplication

Total Lines	82
Duplicated Lines	0 %

Importance

Changes	4
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getForum()	0	7	2
A	__construct()	0	3	1
A	setNotRead()	0	6	1
A	countTopicNotRead()	0	4	1
A	countSubForumNotRead()	0	4	1

<?php

namespace ProjetNormandie\ForumBundle\Service;

use Doctrine\ORM\EntityManagerInterface;
use Doctrine\ORM\ORMException;
use ProjetNormandie\ForumBundle\Entity\Forum;

class ForumService
{
    private EntityManagerInterface $em;

    /**
     * ForumService constructor.
     * @param EntityManagerInterface $em
     */
    public function __construct(EntityManagerInterface $em)
    {
        $this->em = $em;
    }

    /**
     * @param $forum
     * @return Forum
     */
    private function getForum($forum): Forum

The method getForum() is not used, and could be removed.

    {
        if (!$forum instanceof Forum) {
            $forum = $this->em->getRepository('ProjetNormandie\ForumBundle\Entity\Forum')
                ->findOneBy(['id' => $forum]);
        }
        return $forum;

The expression return $forum could return the type null which is incompatible with the type-hinted return ProjetNormandie\ForumBundle\Entity\Forum. Consider adding an additional type-check to rule them out.

    }


    /**
     * @param $forum
     * @deprecated
     */
    /*public function majPosition($forum)
    {
        $forum = $this->getForum($forum);
        if ($forum->getIsParent()) {
            foreach ($forum->getChildrens() as $child) {
                foreach ($child->getTopics() as $topic) {
                    $this->topicService->majPositions($topic);
                }
            }
        } else {
            foreach ($forum->getTopics() as $topic) {
                $this->topicService->majPositions($topic);
            }
        }
    }*/


    /**
     * @param Forum $forum
     * @param       $user
     */
    public function setNotRead(Forum $forum, $user)
    {
        $forumUser = $this->em->getRepository('ProjetNormandie\ForumBundle\Entity\ForumUser')
                ->findOneBy(['forum' => $forum, 'user' => $user]);
        $forumUser->setBoolRead(false);
        $this->em->flush();
    }


    /**
     * @param Forum $forum
     * @param       $user
     * @return int
     */
    public function countTopicNotRead(Forum $forum, $user): int
    {
        return $this->em->getRepository('ProjetNormandie\ForumBundle\Entity\TopicUser')
                ->countNotRead($forum,$user);
    }


    /**
     * @param Forum $parent
     * @param       $user
     * @return int
     */
    public function countSubForumNotRead(Forum $parent, $user): int
    {
        return $this->em->getRepository('ProjetNormandie\ForumBundle\Entity\ForumUser')
                ->countSubForumNotRead($parent, $user);
    }
}