Applications\Service\UploadHandler

Complexity

Total Complexity

6

Size/Duplication

Total Lines	108
Duplicated Lines	0 %

Importance

Changes	1
Bugs	1	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	findApplication()	0	3	1
A	__construct()	0	9	1
A	handleImageUpload()	0	23	2
A	doUploadFile()	0	19	1
A	handleAttachmentUpload()	0	14	1

<?php

declare(strict_types=1);

namespace Applications\Service;


use Applications\Entity\Application;
use Applications\Entity\ApplicationInterface;
use Applications\Entity\Attachment;
use Core\Entity\FileMetadata;
use Core\Service\FileManager;
use Core\Service\UploadedFileInfo;
use Doctrine\ODM\MongoDB\DocumentManager;
use Doctrine\Persistence\ObjectRepository;

class UploadHandler
{
    /**
     * @var DocumentManager
     */
    private DocumentManager $dm;
    /**
     * @var FileManager
     */
    private FileManager $fileManager;

    /**
     * @var ObjectRepository
     */
    private ObjectRepository $appRepository;

    public function __construct(
        DocumentManager $dm,
        FileManager $fileManager,
        ObjectRepository $appRepository
    )
    {
        $this->dm = $dm;
        $this->fileManager = $fileManager;
        $this->appRepository = $appRepository;
    }

    /**
     * @param string $id
     * @return object|null|ApplicationInterface
     */
    public function findApplication(string $id)
    {
        return $this->appRepository->find($id);
    }

    /**
     * @param string $appId
     * @param array $info
     * @return object|Attachment
     * @throws \Doctrine\ODM\MongoDB\MongoDBException
     */
    public function handleAttachmentUpload(
        string $appId,
        array $info
    )
    {
        $application = $this->findApplication($appId);
        $attachment = $this->doUploadFile($application, $info);

It seems like $application can also be of type null; however, parameter $application of Applications\Service\UploadHandler::doUploadFile() does only seem to accept Applications\Entity\ApplicationInterface, maybe add an additional type check?

        $dm = $this->dm;

        $application->getAttachments()->add($attachment);
        $dm->persist($application);
        $dm->flush();

        return $attachment;
    }

    /**
     * @param string $appID
     * @param array $info
     * @return object|ApplicationInterface
     * @throws \Doctrine\ODM\MongoDB\MongoDBException
     */
    public function handleImageUpload(
        string $appID,
        array $info
    )
    {
        $dm = $this->dm;
        $application = $this->findApplication($appID);
        $attachment = $this->doUploadFile($application, $info);

It seems like $application can also be of type null; however, parameter $application of Applications\Service\UploadHandler::doUploadFile() does only seem to accept Applications\Entity\ApplicationInterface, maybe add an additional type check?

        // remove existing image
        if(!is_null($application->getContact()->getImage())){
            $image = $application->getContact()->getImage();
            $application->getContact()->setImage(null);
            $dm->remove($image);
            $dm->persist($application);
            $dm->flush();
        }

        $application->getContact()->setImage($attachment);
        $dm->persist($application);
        $dm->flush();

        return $application;
    }

    private function doUploadFile(
        ApplicationInterface $application,
        array $info
    )
    {
        $fileManager = $this->fileManager;
        $user = $application->getUser();

        $metadata = new FileMetadata();
        $metadata->setUser($user);

It seems like $user can also be of type null; however, parameter $user of Core\Entity\FileMetadata::setUser() does only seem to accept Auth\Entity\UserInterface, maybe add an additional type check?

        $metadata->setContentType($info['type']);
        $metadata->setName($info['name']);

        $this->dm->persist($user);
        return $fileManager->uploadFromFile(
            Attachment::class,
            $metadata,
            $info['tmp_name'],
            $info['name']
        );
    }
}