App\Controller\API\HookController

Complexity

Total Complexity

3

Size/Duplication

Total Lines	25
Duplicated Lines	36 %

Coupling/Cohesion

Components	0
Dependencies	6

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	fetchGuilds()	9	9	2
A	fetchCharacters()	0	7	1

<?php

namespace App\Controller\API;

use App\Entity\Guild;
use App\Entity\Setting;
use App\Handler\ApiHandler;
use App\Utils\AbstractCrawler;
use App\Utils\CharacterCrawler;
use App\Utils\GuildCrawler;
use FOS\RestBundle\Controller\FOSRestController;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
use Nelmio\ApiDocBundle\Annotation\Model;
use Swagger\Annotations as SWG;
use Symfony\Component\HttpFoundation\JsonResponse;

/**
 * @Route("/api/hook")
 */
class HookController extends FOSRestController
{
    /**
     * @Route("/fetch-guild", name="api_guild_collect_users")
     */
    public function fetchGuilds(GuildCrawler $crawler)

This method seems to be duplicated in your project.

    {
        $guilds = $this->getDoctrine()->getRepository(Guild::class)->findAll();
        foreach ($guilds as $guild) {
            $crawler->crawl($guild);
        }

        return JsonResponse::create(['success']);
    }
    /**
     * @Route("/fetch-characters", name="api_collect_characters")
     */
    public function fetchCharacters(CharacterCrawler $crawler)
    {
        $settings = $this->getDoctrine()->getRepository(Setting::class)->findOneByCode('swgoh');

The method findOneByCode() does not exist on Doctrine\Common\Persistence\ObjectRepository. Did you maybe mean findOneBy()?

        $crawler->crawl($settings);

        return JsonResponse::create(['success']);
    }
}