nopolabs/yabot

Method __construct has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

        Client $slackClient,
        array $data,
        string $formattedText,
        User $user = null,
        Channel $channel)

Return type of getChannel() is undeclared type \Slack\Channel

Open

    public function getChannel() : Channel

• Exclude checks
  • Disable engine
  • Disable check

Property \Nopolabs\Yabot\Message\Message->channel has undeclared type \Slack\Channel

Open

    private $channel;

• Exclude checks
  • Disable engine
  • Disable check

Assigning ?\Slack\User to property but \Nopolabs\Yabot\Message\Message->user is \Slack\User

Open

        $this->user = $user;

• Exclude checks
  • Disable engine
  • Disable check

Parameter $user has undeclared type ?\Slack\User

Open

    public function __construct(

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property data from undeclared class \Slack\User

Open

            return (bool) $this->user->data['is_bot'];

• Exclude checks
  • Disable engine
  • Disable check

Parameter $channel has undeclared type \Slack\Channel

Open

    public function __construct(

• Exclude checks
  • Disable engine
  • Disable check

Reference to instance property data from undeclared class \Slack\User

Open

        if (isset($this->user->data['is_bot'])) {

• Exclude checks
  • Disable engine
  • Disable check

Call to method getUsername from undeclared class \Slack\User

Open

        return $user ? $user->getUsername() : ($this->isBot() ? 'unknown-bot' : 'unknown-user');

• Exclude checks
  • Disable engine
  • Disable check

Call to method getName from undeclared class \Slack\Channel

Open

        return $this->getChannel()->getName();

• Exclude checks
  • Disable engine
  • Disable check

Property \Nopolabs\Yabot\Message\Message->user has undeclared type \Slack\User

Open

    private $user;

• Exclude checks
  • Disable engine
  • Disable check

Required argument follows optional

Open

    public function __construct(

• Exclude checks
  • Disable engine
  • Disable check

The closing parenthesis of a multi-line function declaration must be on a new line

Open

        Channel $channel)

• Exclude checks
  • Disable engine
  • Disable check

The closing parenthesis and the opening brace of a multi-line function declaration must be on the same line

Open

    {

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 newline at end of file; 0 found

Open

}

• Exclude checks
  • Disable engine
  • Disable check