SilverStripe\AssetAdmin\GraphQL\MoveFilesMutationCreator

Complexity

Total Complexity

13

Size/Duplication

Total Lines	91
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	7

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	attributes()	0	6	1
A	type()	0	6	1
A	args()	0	11	1
C	resolve()	0	50	9

<?php
namespace SilverStripe\AssetAdmin\GraphQL;

use GraphQL\Type\Definition\Type;
use GraphQL\Type\Definition\ResolveInfo;
use SilverStripe\Assets\File;
use SilverStripe\Assets\Folder;
use SilverStripe\GraphQL\MutationCreator;
use SilverStripe\GraphQL\OperationResolver;
use SilverStripe\ORM\Versioning\Versioned;
use SilverStripe\GraphQL\Util\CaseInsensitiveFieldAccessor;
use SilverStripe\GraphQL\Manager;

class MoveFilesMutationCreator extends MutationCreator implements OperationResolver
{
    /**
     * @var CaseInsensitiveFieldAccessor
     */
    protected $accessor;

    public function __construct(Manager $manager = null)
    {
        $this->accessor = new CaseInsensitiveFieldAccessor();

        parent::__construct($manager);
    }

    public function attributes()
    {
        return [
            'name' => 'moveFiles',
        ];
    }

    public function type()
    {
        return function () {
            return $this->manager->getType('FileInterface');
        };
    }

    public function args()
    {
        return [
            'folderId' => [
                'type' => Type::nonNull(Type::id())
            ],
            'fileIds' => [
                'type' => Type::listOf(Type::id())
            ],
        ];
    }

    public function resolve($object, array $args, $context, ResolveInfo $info)
    {
        $folderId = (isset($args['folderId'])) ? $args['folderId'] : 0;
        
        if ($folderId) {
            /** @var Folder $folder */
            $folder = Versioned::get_by_stage(Folder::class, Versioned::DRAFT)
                ->byID($folderId);
            if (!$folder) {
                throw new \InvalidArgumentException(sprintf(
                    '%s#%s not found',
                    Folder::class,
                    $folderId
                ));
            }
    
            // Check permission
            if (!$folder->canEdit($context['currentUser'])) {
                throw new \InvalidArgumentException(sprintf(
                    '%s edit not allowed',
                    Folder::class
                ));
            }
        }
        $files = Versioned::get_by_stage(File::class, Versioned::DRAFT)
            ->byIDs($args['fileIds']);
        $errorFiles = [];
        
        foreach($files as $file) {
            if ($file->canEdit($context['currentUser'])) {
                $this->accessor->setValue($file, 'ParentID', $folderId);
                $file->writeToStage(Versioned::DRAFT);
            } else {
                $errorFiles[] = $file->ID;
            }
        }
        
        if ($errorFiles) {

The expression $errorFiles of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            throw new \InvalidArgumentException(sprintf(
                '%s (%s) edit not allowed',
                File::class,
                implode(', ', $errorFiles)
            ));
        }

        if (!isset($folder)) {
            return Folder::singleton();
        }
        return $folder;
    }
}