Wrong annotation in _toOptionArray - magento/framework/Data/Collection/AbstractDb.php

[BrocksiNet]

The annotation for valueField should be @param null|string $valueField at the moment php phan thorws this as an error. And i think phan is right.

See here:
src/vendor/magento/framework/Data/Collection/AbstractDb.php

    /**
     * Overridden to use _idFieldName by default.
     *
     * @param null $valueField
     * @param string $labelField
     * @param array $additional
     * @return array
     */
    protected function _toOptionArray($valueField = null, $labelField = 'name', $additional = [])
    {
        if ($valueField === null) {
            $valueField = $this->getIdFieldName();
        }
        return parent::_toOptionArray($valueField, $labelField, $additional);
    }

[magento-engcom-team]

@bjoern-flagbit, thank you for your report.
We've acknowledged the issue and added to our backlog.

[sanjay-wagento]

I am working on this at #dmcdindia

[magento-engcom-team]

@bjoern-flagbit Looks like the issue was fixed in the commit 67b5fe7 by @sanjay-wagento

Thank you for collaboration!