b2pweb / bdf-prime

src/Migration/Console/AbstractCommand.php

<?php

namespace Bdf\Prime\Migration\Console;

use Bdf\Prime\Migration\MigrationManager;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

/**
 * AbstractCommand
 */
abstract class AbstractCommand extends Command
{
    /**
     * @var MigrationManager
     */
    private $manager;

    /**
     * Migration command constructor.
     *
     * @param MigrationManager $manager
     */
    public function __construct(MigrationManager $manager)
    {
        $this->manager = $manager;

        parent::__construct(static::$defaultName);
    }

    /**
     * Add console context on manager
     *
     * @param InputInterface $input
     * @param OutputInterface $output
     *
     * @return MigrationManager
     */
    public function manager(InputInterface $input, OutputInterface $output): MigrationManager
    {
        $this->manager->setOutput($output);
        $this->manager->setInput($input);
        $this->manager->setHelper($this->getHelperSet());

It seems like $this->getHelperSet() can also be of type null; however, parameter $helper of Bdf\Prime\Migration\MigrationManager::setHelper() does only seem to accept Symfony\Component\Console\Helper\HelperSet, maybe add an additional type check?

        return $this->manager;
    }
}