Zenstruck\Browser\Dom\Form\Field\FileField

Complexity

Total Complexity

9

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
B	upload()	0	39	9

<?php

namespace Zenstruck\Browser\Dom\Form\Field;

use Symfony\Component\DomCrawler\Field\FileFormField;
use Symfony\Component\Panther\DomCrawler\Field\FileFormField as PantherFileFormField;
use Zenstruck\Browser\Dom\Form\Field;

/**
 * @mixin FileFormField
 *
 * @author Kevin Bond <kevinbond@gmail.com>
 */
final class FileField extends Field
{
    /**
     * @param string|array $files
     */
    public function upload($files): void
    {
        $files = (array) $files;

        if (!\count($files)) {
            throw new \InvalidArgumentException('Must provide at least one file.');
        }

        foreach ($files as $file) {
            if (!\is_file($file)) {
                throw new \InvalidArgumentException("File \"{$file}\" does not exist.");
            }
        }

        if ($this->inner instanceof PantherFileFormField) {
            foreach ($files as $file) {
                $this->inner->upload($file);

The method upload() does not exist on Symfony\Component\DomCrawler\Field\FormField. It seems like you code against a sub-type of Symfony\Component\DomCrawler\Field\FormField such as Symfony\Component\DomCrawler\Field\FileFormField.

            }

            return;
        }

        // Hack to allow multiple files to be attached
        $this->inner->upload(\array_shift($files));

        if (empty($files)) {
            // not multiple files
            return;
        }

        if (!$this->attr('multiple')) {
            throw new \InvalidArgumentException('Cannot attach multiple files to a non-multiple file field.');
        }

        foreach ($files as $file) {
            $field = new FileFormField($this->dom->getNode(0));

It seems like $this->dom->getNode(0) can also be of type Zenstruck\Browser\Dom and null; however, parameter $node of Symfony\Component\DomCrawler\Field\FileFormField::__construct() does only seem to accept DOMElement, maybe add an additional type check?

The method getNode() does not exist on Zenstruck\Browser\Dom. Since you implemented __call, consider adding a @method annotation.

            $field->upload($file);

            $this->form->set($field);
        }
    }
}