Obblm\Core\Helper\FileUploader\AbstractFileUploader

Complexity

Total Complexity

8

Size/Duplication

Total Lines	57
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	removeOldFile()	0	8	2
A	__construct()	0	5	1
A	getObjectDirectory()	0	3	1
A	setObjectSubDirectory()	0	4	1
A	upload()	0	13	2
A	getTargetDirectory()	0	3	1

<?php

namespace Obblm\Core\Helper\FileUploader;

use Symfony\Component\Filesystem\Exception\IOExceptionInterface;
use Symfony\Component\Filesystem\Filesystem;
use Symfony\Component\HttpFoundation\File\Exception\FileException;
use Symfony\Component\HttpFoundation\File\File;
use Symfony\Component\HttpFoundation\File\UploadedFile;
use Symfony\Component\String\Slugger\SluggerInterface;

class AbstractFileUploader
{
    private $targetDirectory;
    private $uploadDirectory;
    private $slugger;

    public function __construct($targetDirectory, SluggerInterface $slugger)
    {
        $this->targetDirectory = $targetDirectory;
        $this->uploadDirectory = $targetDirectory;
        $this->slugger = $slugger;
    }

    public function removeOldFile(string $filename = '')
    {
        $filesystem = new Filesystem();

        try {
            $filesystem->remove($this->getObjectDirectory() . '/' . $filename);
        } catch (IOExceptionInterface $exception) {
            echo "An error occurred while creating your directory at ".$exception->getPath();
        }
    }

    public function getTargetDirectory(): ?string
    {
        return $this->targetDirectory;
    }

    public function getObjectDirectory(): ?string
    {
        return $this->uploadDirectory;
    }

    public function setObjectSubDirectory(string $uploadDirectory): self
    {
        $this->uploadDirectory = $this->targetDirectory .'/' . $uploadDirectory;
        return $this;
    }

    /**
     * @param UploadedFile $file
     * @return File|null
     */
    public function upload(UploadedFile $file):?File
    {
        $originalFilename = pathinfo($file->getClientOriginalName(), PATHINFO_FILENAME);
        $safeFilename = $this->slugger->slug($originalFilename);
        $fileName = $safeFilename.'-'.uniqid().'.'.$file->guessExtension();

        try {
            $newFile = $file->move($this->getObjectDirectory(), $fileName);

It seems like $this->getObjectDirectory() can also be of type null; however, parameter $directory of Symfony\Component\HttpFoundation\File\UploadedFile::move() does only seem to accept string, maybe add an additional type check?

        } catch (FileException $e) {
            return null;
        }

        return $newFile;
    }
}