Skip to content

Add missing chain method to interface#58429

Merged
taylorotwell merged 3 commits intolaravel:masterfrom
Jeroen-G:patch-1
Jan 21, 2026
Merged

Add missing chain method to interface#58429
taylorotwell merged 3 commits intolaravel:masterfrom
Jeroen-G:patch-1

Conversation

@Jeroen-G
Copy link
Contributor

I was missing the chain method in the Dispatcher interface :)

@taylorotwell taylorotwell marked this pull request as draft January 19, 2026 21:15
@taylorotwell
Copy link
Member

Can you place it in the correct order?

@Jeroen-G
Copy link
Contributor Author

It is now alphabetically, what would be the correct order? :)

@rodrigopedra
Copy link
Contributor

what would be the correct order?

I guess the same as the concrete class.

In this case, right before the Dispatcher@hasCommandHandler() method.

/**
* Determine if the given command has a handler.
*
* @param mixed $command
* @return bool
*/
public function hasCommandHandler($command);

@Jeroen-G
Copy link
Contributor Author

Done!

@Jeroen-G Jeroen-G marked this pull request as ready for review January 21, 2026 08:09
@taylorotwell taylorotwell merged commit c00f2a8 into laravel:master Jan 21, 2026
60 of 64 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants