Zenstruck\Browser\Dom\Form\Field\ChoiceField::setValue()

Complexity

Conditions	10
Paths	16

Size

Total Lines	28
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function setValue($value): void
    {
        if ($this->is('radio')) {
            $value = $value ?? $this->attr('value');
        }

        if (null === $value) {
            throw new \InvalidArgumentException('Value required for select form fields.');
        }

        if (\is_array($value) && !$this->isMultiple()) {

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

            throw new \InvalidArgumentException('Value for a single select form field cannot be an array.');
        }

        if (\is_array($value) && $this->isMultiple() && $this->dom->inner() instanceof PantherCrawler) {

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

            // todo have a separate PantherCrawler to avoid the "inner" check above
            // with panther, unselect all before setting (otherwise existing selections will remain)
            (new WebDriverSelect($this->dom->getElement(0)))->deselectAll();

It seems like $this->dom->getElement(0) can also be of type Zenstruck\Browser\Dom; however, parameter $element of Facebook\WebDriver\WebDriverSelect::__construct() does only seem to accept Facebook\WebDriver\WebDriverElement, maybe add an additional type check?

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

        }

        try {
            $this->inner->setValue($value);
        } catch (NoSuchElementException $e) {
            // try selecting by visible text
            $select = new WebDriverSelect($this->dom->getElement(0));

            foreach ((array) $value as $item) {
                $select->selectByVisibleText($item);
            }