Uploader\Adapters\Psr7

Complexity

Total Complexity

5

Size/Duplication

Total Lines	34
Duplicated Lines	35.29 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	check()	0	4	1
A	save()	0	4	1
A	fixDestination()	12	12	3

<?php

namespace Uploader\Adapters;

use Uploader\Uploader;
use Psr\Http\Message\UploadedFileInterface;

/**
 * Adapter to save a file from Psr7's UploadedFileInterface.
 */
class Psr7 implements AdapterInterface
{
    /**
     * {@inheritdoc}
     */
    public static function check($original)
    {
        return ($original instanceof UploadedFileInterface);
    }

    /**
     * {@inheritdoc}
     */
    public static function fixDestination(Uploader $uploader, $original)

This method seems to be duplicated in your project.

    {
        $path = Uploader::parsePath($original->getClientFilename());

        if (!$uploader->getFilename()) {

The expression $uploader->getFilename() of type null|string is loosely compared to false; this is ambiguous if the string can be empty. You might want to explicitly use === null instead.

            $uploader->setFilename($path['filename']);
        }

        if (!$uploader->getExtension()) {
            $uploader->setExtension($path['extension']);
        }
    }

    /**
     * {@inheritdoc}
     */
    public static function save($original, $destination)
    {
        $original->moveTo($destination);
    }
}