AppBundle\Service\FileUploaderService

Complexity

Total Complexity

5

Size/Duplication

Total Lines	68
Duplicated Lines	0 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getFullPath()	0	5	1
A	remove()	0	4	1
A	__construct()	0	3	1
A	upload()	0	7	1
A	getTargetDir()	0	3	1

<?php

/**
 * Copyright (c) 2017 Francois-Xavier Soubirou.
 *
 * This file is part of ci-report.
 *
 * ci-report is free software: you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
 * the Free Software Foundation, either version 3 of the License, or
 * (at your option) any later version.
 *
 * ci-report is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with ci-report. If not, see <http://www.gnu.org/licenses/>.
 */
declare(strict_types=1);

namespace AppBundle\Service;

use Symfony\Component\HttpFoundation\File\UploadedFile;

/**
 * File uploader service class. Inspired from Symfony example:
 * https://symfony.com/doc/current/controller/upload_file.html.
 *
 * @category  ci-report app
 *
 * @author    Francois-Xavier Soubirou <soubirou@yahoo.fr>
 * @copyright 2017 Francois-Xavier Soubirou
 * @license   http://www.gnu.org/licenses/   GPLv3
 *
 * @see      https://www.ci-report.io
 */
class FileUploaderService
{
    /**
     * @var string
     */
    private $targetDir;

    /**
     * Constructor.
     *
     * @param string $targetDir Directory to save temporary files
     */
    public function __construct(string $targetDir)
    {
        $this->targetDir = $targetDir;
    }

    /**
     * Move file temporary directory.
     *
     * @param UploadedFile $file File to move
     *
     * @return string
     */
    public function upload(UploadedFile $file): string
    {
        $fileName = md5(uniqid()).'.'.$file->guessExtension();

        $file->move($this->getTargetDir(), $fileName);

        return $fileName;
    }

    /**
     * Remove file from temporary directory.
     *
     * @param string $fileName File to remove
     *
     * @return string
     */
    public function remove(string $fileName): string
    {
        $fileNameWithoutPath = basename($fileName);
        unlink($this->getTargetDir().'/'.$fileNameWithoutPath);

In this branch, the function will implicitly return null which is incompatible with the type-hinted return string. Consider adding a return statement or allowing null as return value.

    }

    /**
     * Get full path of the file.
     *
     * @param string $fileName File name
     *
     * @return string
     */
    public function getFullPath(string $fileName): string
    {
        $fileNameWithoutPath = basename($fileName);

        return $this->targetDir.'/'.$fileNameWithoutPath;
    }

    /**
     * Get temporary directory name.
     *
     * @return string
     */
    public function getTargetDir(): string
    {
        return $this->targetDir;
    }
}