yuanxing008 / laravel-aliyun-amqp

src/Command/BasePublisherCommand.php

<?php
/**
 * Author: Joker
 * Date: 2020-05-08 13:57
 */

namespace JokerProject\LaravelAliyunAmqp\Command;

use Illuminate\Console\Command;
use Monolog\Handler\StreamHandler;
use Monolog\Logger;
use JokerProject\LaravelAliyunAmqp\ConsumerInterface;
use JokerProject\LaravelAliyunAmqp\Container;
use JokerProject\LaravelAliyunAmqp\PublisherInterface;
use Psr\Log\LoggerAwareInterface;
use Psr\Log\LoggerInterface;
use Symfony\Component\Console\Output\OutputInterface;

/**
 * Class BasePublisherCommand
 *
 * @package JokerProject\LaravelAliyunAmqp\Command
 */
class BasePublisherCommand extends Command
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'rabbitmq:publish {publisher} {message}';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Publish one message';

    /**
     * @var null|Container
     */
    private $container = null;

    /**
     * BasePublisherCommand constructor.
     *
     * @param Container $container
     */
    public function __construct(Container $container)
    {
        $this->container = $container;
        parent::__construct();
    }

    /**
     * @param string $publisherAliasName
     * @return PublisherInterface
     */
    protected function getPublisher(string $publisherAliasName): PublisherInterface
    {
        return $this->container->getPublisher($publisherAliasName);

The method getPublisher() does not exist on null.

    }

    /**
     * Execute the console command.
     * @return int
     */
    public function handle()
    {
        $this->getPublisher($this->input->getArgument('publisher'))

It seems like $this->input->getArgument('publisher') can also be of type null and string[]; however, parameter $publisherAliasName of JokerProject\LaravelAliyunAmqp\Command\BasePublisherCommand::getPublisher() does only seem to accept string, maybe add an additional type check?

            ->publish($this->input->getArgument('message'));

It seems like $this->input->getArgument('message') can also be of type null and string[]; however, parameter $message of JokerProject\LaravelAliyunAmqp\PublisherInterface::publish() does only seem to accept string, maybe add an additional type check?

    }
}