CompoLab\Application\Http\Controller\GitlabController

Complexity

Total Complexity

9

Size/Duplication

Total Lines	58
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
B	handle()	0	44	8

<?php

namespace CompoLab\Application\Http\Controller;

use CompoLab\Application\GitlabRepositoryManager;
use Gitlab\Client as Gitlab;
use Gitlab\Model\Project;
use Symfony\Component\HttpFoundation\JsonResponse;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\HttpKernel\Exception\BadRequestHttpException;

final class GitlabController
{
    /** @var Gitlab */
    private $gitlab;

    /** @var GitlabRepositoryManager */
    private $repositoryManager;

    public function __construct(Gitlab $gitlab, GitlabRepositoryManager $repositoryManager)
    {
        $this->gitlab = $gitlab;
        $this->repositoryManager = $repositoryManager;
    }

    public function handle(Request $request): Response
    {
        $event = json_decode($request->getContent(), true);

        if (!isset($event['project_id'])) {
            throw new BadRequestHttpException('Missing project_id from body');
        }

        if (!isset($event['event_name'])) {
            throw new BadRequestHttpException('Missing event_name from body');
        }

        if (!isset($event['project_id']) or !$project = Project::fromArray(
            $this->gitlab,
            $this->gitlab->projects()->show($event['project_id']))

It seems like $this->gitlab->projects()->show($event['project_id']) can also be of type string; however, parameter $data of Gitlab\Model\Project::fromArray() does only seem to accept array, maybe add an additional type check?

        ){
            throw new BadRequestHttpException('Impossible te retrieve a Gitlab project from the request');
        }

        switch ($event['event_name']) {
            case 'project_destroy':
                $this->repositoryManager->deleteProject($project);
                break;

            case 'push':
            case 'tag_push':
                $this->repositoryManager->registerProject($project);
                break;

            default: return new JsonResponse([
                'status' => 200,
                'message' => 'CompoLab has NOT handled the Gitlab event',
                'project_id' => $event['project_id'],
                'event_name' => $event['event_name'],
            ]);
        }

        $this->repositoryManager->save();

        return new JsonResponse([
            'status' => 200,
            'message' => 'CompoLab has successfully handled the Gitlab event',
            'project_id' => $event['project_id'],
            'event_name' => $event['event_name'],
        ]);
    }
}