Zenstruck\Browser\Dom\Form\Field\FileField::upload()

Complexity

Conditions	9
Paths	14

Size

Total Lines	39
Code Lines	19

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    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);
        }