Graviton\SwaggerBundle\Controller\SwaggerController

Complexity

Total Complexity

4

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	swaggerAction()	0	19	3

<?php
/**
 * controller for rendering our swagger spec
 */

namespace Graviton\SwaggerBundle\Controller;

use Symfony\Component\Finder\Finder;
use Symfony\Component\HttpKernel\Exception\FatalErrorException;
use Symfony\Component\HttpFoundation\Response;

/**
 * SwaggerController
 *
 * @author   List of contributors <https://github.com/libgraviton/graviton/graphs/contributors>
 * @license  http://opensource.org/licenses/gpl-license.php GNU Public License
 * @link     http://swisscom.ch
 */
class SwaggerController
{
    /*
     * @var Finder
     */
    private $finder;

    /**
     * @var string
     */
    private $rootDir;

    /**
     * @param Finder $finder  symfony/finder instance
     * @param string $rootDir symfomy root dir
     */
    public function __construct(Finder $finder, $rootDir)
    {
        $this->finder = $finder;
        $this->rootDir = $rootDir;
    }

    /**
     * @return JsonResponse Response with result or error

Should the return type not be Response|null?

     */
    public function swaggerAction()
    {
        $this->finder->files()->in($this->rootDir . '/cache')->name('swagger.json');

        if ($this->finder->count() != 1) {
            throw new FatalErrorException('Failed to find a generated swagger file');

The class Symfony\Component\HttpKernel\Exception\FatalErrorException has been deprecated with message: Deprecated in 2.3, to be removed in 3.0. Use the same class from the Debug component instead.

        }

        foreach ($this->finder as $file) {
            return new Response(
                $file->getContents(),
                Response::HTTP_OK,
                [
                    'Content-Type' => 'application/json'
                ],
                false

The call to Response::__construct() has too many arguments starting with false.

            );
        }
    }
}