Help::displayHelpForCommand() - Code Metrics - spatie/laravel-slack-slash-command - Measure and Improve Code Quality continuously with Scrutinizer

Help::displayHelpForCommand() A
last analyzed 2020-10-12 09:04 UTC

↳ Parent: Help

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	19
rs	9.6333
c	0
b	0
f	0
cc	1
nc	1
nop	2

<?php

namespace Spatie\SlashCommand\Handlers;

use Illuminate\Support\Collection;
use Illuminate\Support\Str;
use Spatie\SlashCommand\Attachment;
use Spatie\SlashCommand\AttachmentField;
use Spatie\SlashCommand\HandlesSlashCommand;
use Spatie\SlashCommand\Request;
use Spatie\SlashCommand\Response;

class Help extends SignatureHandler
{
    protected $signature = '* help {command? : The command you want information about}';

    protected $description = 'List all commands or provide information about all commands';

    /**
     * Handle the given request.
     *
     * @param \Spatie\SlashCommand\Request $request
     *
     * @return \Spatie\SlashCommand\Response
     */
    public function handle(Request $request): Response
    {
        $handlers = $this->findAvailableHandlers();

        if ($command = $this->getArgument('command')) {
            return $this->displayHelpForCommand($handlers, $command);
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
        }

        return $this->displayListOfAllCommands($handlers);
/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}
    }

    /**
     * Find all handlers that are available for the current SlashCommand
     * and have a signature.
     *
     * @return Collection|SignatureHandler[]
     */
    protected function findAvailableHandlers(): Collection
    {
        return collect(config('laravel-slack-slash-command.handlers'))
            ->map(function (string $handlerClassName) {
                return new $handlerClassName($this->request);
            })
            ->filter(function (HandlesSlashCommand $handler) {
                return $handler instanceof SignatureHandler;
            })
            ->filter(function (SignatureHandler $handler) {
                $signatureParts = new SignatureParts($handler->getSignature());

                return Str::is($signatureParts->getSlashCommandName(), $this->request->command);
            });
    }

    /**
     * Show the help information for a single SignatureHandler.
     *
     * @param  Collection $handlers
     * @param  string $command
     * @return Response
     */
    protected function displayHelpForCommand(Collection $handlers, string $command): Response
    {
        $helpRequest = clone $this->request;

        $helpRequest->text = $command;

        /** @var \Spatie\SlashCommand\Handlers $handler */
        $handler = $handlers->filter(function (HandlesSlashCommand $handler) use ($helpRequest) {
            return $handler->canHandle($helpRequest);
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        })
            ->first();

        $field = AttachmentField::create($handler->getFullCommand(), $handler->getHelpDescription());

        return $this->respondToSlack('')
            ->withAttachment(
                Attachment::create()->addField($field)
            );
    }

    /**
     * Show a list of all available handlers.
     *
     * @param  Collection $handlers
     * @return Response
     */
    protected function displayListOfAllCommands(Collection $handlers): Response
    {
        $attachmentFields = $handlers
            ->sort(function (SignatureHandler $handlerA, SignatureHandler $handlerB) {
                return strcmp($handlerA->getFullCommand(), $handlerB->getFullCommand());
            })
            ->map(function (SignatureHandler $handler) {
                return AttachmentField::create($handler->getFullCommand(), $handler->getDescription());
            })
            ->all();

        return $this->respondToSlack('Available commands:')
            ->withAttachment(
                Attachment::create()
                    ->setColor('good')
                    ->setFields($attachmentFields)
            );
    }
}


1			<?php
2
3			namespace Spatie\SlashCommand\Handlers;
4
5			use Illuminate\Support\Collection;
6			use Illuminate\Support\Str;
7			use Spatie\SlashCommand\Attachment;
8			use Spatie\SlashCommand\AttachmentField;
9			use Spatie\SlashCommand\HandlesSlashCommand;
10			use Spatie\SlashCommand\Request;
11			use Spatie\SlashCommand\Response;
12
13			class Help extends SignatureHandler
14			{
15			protected $signature = '* help {command? : The command you want information about}';
16
17			protected $description = 'List all commands or provide information about all commands';
18
19			/**
20			* Handle the given request.
21			*
22			* @param \Spatie\SlashCommand\Request $request
23			*
24			* @return \Spatie\SlashCommand\Response
25			*/
26			public function handle(Request $request): Response
27			{
28			$handlers = $this->findAvailableHandlers();
29
30			if ($command = $this->getArgument('command')) {
31			return $this->displayHelpForCommand($handlers, $command);
			0 ignored issues – show Bug introduced 2016-10-04 08:05 UTC by Report Bug Copy Issue Report It seems like `$handlers` defined by `$this->findAvailableHandlers()` on line `28` can also be of type `array<integer,object<Spa...lers\SignatureHandler>>`; however, `Spatie\SlashCommand\Hand...displayHelpForCommand()` does only seem to accept `object<Illuminate\Support\Collection>`, maybe add an additional type check? If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check: /** * @return array\|string / function returnsDifferentValues($x) { if ($x) { return 'foo'; } return array(); } $x = returnsDifferentValues($y); if (is_array($x)) { // $x is an array. } If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue. Loading history... Bug introduced 2019-01-15 16:52 UTC by Report Bug Copy Issue Report It seems like `$command` defined by `$this->getArgument('command')` on line `30` can also be of type `array<integer,string>`; however, `Spatie\SlashCommand\Hand...displayHelpForCommand()` does only seem to accept `string`, maybe add an additional type check? If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check: /* * @return array\|string */ function returnsDifferentValues($x) { if ($x) { return 'foo'; } return array(); } $x = returnsDifferentValues($y); if (is_array($x)) { // $x is an array. } If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue. Loading history...
32			}
33
34			return $this->displayListOfAllCommands($handlers);
			0 ignored issues – show Bug introduced 2016-10-04 17:55 UTC by Report Bug Copy Issue Report It seems like `$handlers` defined by `$this->findAvailableHandlers()` on line `28` can also be of type `array<integer,object<Spa...lers\SignatureHandler>>`; however, `Spatie\SlashCommand\Hand...playListOfAllCommands()` does only seem to accept `object<Illuminate\Support\Collection>`, maybe add an additional type check? If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check: /** * @return array\|string */ function returnsDifferentValues($x) { if ($x) { return 'foo'; } return array(); } $x = returnsDifferentValues($y); if (is_array($x)) { // $x is an array. } If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue. Loading history...
35			}
36
37			/**
38			* Find all handlers that are available for the current SlashCommand
39			* and have a signature.
40			*
41			* @return Collection\|SignatureHandler[]
42			*/
43			protected function findAvailableHandlers(): Collection
44			{
45			return collect(config('laravel-slack-slash-command.handlers'))
46			->map(function (string $handlerClassName) {
47			return new $handlerClassName($this->request);
48			})
49			->filter(function (HandlesSlashCommand $handler) {
50			return $handler instanceof SignatureHandler;
51			})
52			->filter(function (SignatureHandler $handler) {
53			$signatureParts = new SignatureParts($handler->getSignature());
54
55			return Str::is($signatureParts->getSlashCommandName(), $this->request->command);
56			});
57			}
58
59			/**
60			* Show the help information for a single SignatureHandler.
61			*
62			* @param Collection $handlers
63			* @param string $command
64			* @return Response
65			*/
66			protected function displayHelpForCommand(Collection $handlers, string $command): Response
67			{
68			$helpRequest = clone $this->request;
69
70			$helpRequest->text = $command;
71
72			/** @var \Spatie\SlashCommand\Handlers $handler */
73			$handler = $handlers->filter(function (HandlesSlashCommand $handler) use ($helpRequest) {
74			return $handler->canHandle($helpRequest);
			0 ignored issues – show Bug introduced 2016-10-03 09:53 UTC by Report Bug Copy Issue Report It seems like you code against a concrete implementation and not the interface `Spatie\SlashCommand\HandlesSlashCommand` as the method `canHandle()` does only exist in the following implementations of said interface: `Spatie\SlashCommand\Handlers\BaseHandler`, `Spatie\SlashCommand\Handlers\CatchAll`, `Spatie\SlashCommand\Handlers\Help`, `Spatie\SlashCommand\Handlers\SignatureHandler`. Let’s take a look at an example: interface User { /** @return string / public function getPassword(); } class MyUser implements User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the interface: interface User { /** @return string / public function getPassword(); /* @return string */ public function getDisplayName(); } Loading history...
75			})
76			->first();
77
78			$field = AttachmentField::create($handler->getFullCommand(), $handler->getHelpDescription());
79
80			return $this->respondToSlack('')
81			->withAttachment(
82			Attachment::create()->addField($field)
83			);
84			}
85
86			/**
87			* Show a list of all available handlers.
88			*
89			* @param Collection $handlers
90			* @return Response
91			*/
92			protected function displayListOfAllCommands(Collection $handlers): Response
93			{
94			$attachmentFields = $handlers
95			->sort(function (SignatureHandler $handlerA, SignatureHandler $handlerB) {
96			return strcmp($handlerA->getFullCommand(), $handlerB->getFullCommand());
97			})
98			->map(function (SignatureHandler $handler) {
99			return AttachmentField::create($handler->getFullCommand(), $handler->getDescription());
100			})
101			->all();
102
103			return $this->respondToSlack('Available commands:')
104			->withAttachment(
105			Attachment::create()
106			->setColor('good')
107			->setFields($attachmentFields)
108			);
109			}
110			}
111

spatie / laravel-slack-slash-command

GitHub Access Token became invalid

Help::displayHelpForCommand() A last analyzed 2020-10-12 09:04 UTC

Complexity

Size

Duplication

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like

Help::displayHelpForCommand() A
last analyzed 2020-10-12 09:04 UTC