Simon

Results 11 comments of Simon

One issue with this fix is that it currently does not handle shortened versions of console commands such as `php artisan tin`. However; I would argue that this is an...

@jasonvarga Sorry to be a bother, but have you had a chance to review this? Please let me know if you would like more detail on the issue and/or the...

@jasonvarga Yes, this approach was disgusting, but it's an issue that really should be resolved somehow.

A potential fix would be to update `Illuminate\Foundation\Application::booted($callback)` with the following: ```php public function booted($callback) { if ($this->isBooted()) { $callback($this); } else { $this->bootedCallbacks[] = $callback; } } ``` I...

Do you require me to provide you with a detailed use-case in order for this bug to be fixed? Because I can do that, but I just don't see it...

1. Certainly not as early as 5.1, I think that you might find that is the offending commit https://github.com/laravel/framework/commit/9eadb7f25db5b0a1aa78470c864a17eb815781f8 2. Perhaps Laravel itself may not have this direct use-case, but...

@crynobone Maybe the `needs more info` label could be removed now?

@crynobone Did you not notice the previous comment indicating that this bug was introduced in this commit https://github.com/laravel/framework/commit/9eadb7f25db5b0a1aa78470c864a17eb815781f8 which was released in version `8.65.0`? This is obviously buggy and undesired...

@crynobone PR created https://github.com/laravel/framework/pull/53683

Here's an alternative implementation for consideration https://github.com/laravel/framework/pull/53695