Bone\Mvc\Controller\DownloadController

Complexity

Total Complexity

7

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

16%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	2
A	downloadAction()	0	30	5

<?php

namespace Bone\Mvc\Controller;

use Bone\Exception;
use League\Route\Http\Exception\NotFoundException;
use InvalidArgumentException;
use Psr\Http\Message\ResponseInterface;
use Psr\Http\Message\ServerRequestInterface;
use Zend\Diactoros\Response;
use Zend\Diactoros\Stream;

class DownloadController
{
    /** @var string $uploadsDirectory */
    private $uploadsDirectory;

    public function __construct(string $uploadsDirectory)
    {
        if (!is_dir($uploadsDirectory)) {
            throw new InvalidArgumentException('Directory ' . $uploadsDirectory . 'not found');
        }

        $this->uploadsDirectory = $uploadsDirectory;
    }

    /**
     * @param ServerRequestInterface $request
     * @param array $args
     * @return ResponseInterface
     */
    public function downloadAction(ServerRequestInterface $request, array $args): ResponseInterface

The parameter $args is not used and could be removed.

    {
        $file = $request->getQueryParams()['file'];

        if (file_exists('public' . $file)) {
            $path = 'public' . $file;
        } else if (!file_exists($this->uploadsDirectory . $file)) {
            throw new Exception($path . ' not found.', 404);
        } else {
            $path = $this->uploadsDirectory . $file;
        }
        
        // magic_mime module installed?
        if (function_exists('mime_content_type')) {
            $mimeType = mime_content_type($path);
        } else if (function_exists('finfo_file')) { // or fileinfo module installed?
            $finfo = finfo_open(FILEINFO_MIME); // return mime type
            $mimeType = finfo_file($finfo, $path);
            finfo_close($finfo);
        }

        $contents = file_get_contents($path);
        $stream = new Stream('php://memory', 'r+');
        $stream->write($contents);
        $response = new Response();
        $response = $response->withBody($stream);
        $response = $response->withHeader('Content-Type', $mimeType);

The variable $mimeType does not seem to be defined for all execution paths leading up to this point.

        return $response;
    }
}