Fixes a TODO in Laravel 8 set. Apparently, it's the only time we need to use ArgumentAdderWithoutDefaultValue.
Replaces ArgumentAdderWithoutDefaultValue with ArgumentAdder in Laravel 9 set, while setting the scope to only the ClassMethod. This satisfies both conditions of modifying the class extensions where appropriate (#239) and keeping method/static calls intact (https://github.com/driftingly/rector-laravel/issues/220).
Removes the change for FailedJobProviderInterface entirely, since an argument rename is needed in that case, but we don't have a rule in core Rector to only rename arguments, yet. I've kept the old test and improved the stub for future changes.
Adds missing stubs for Container and Application, reflecting the current implementations in Laravel.
Removes tests/Sets/Laravel90/Source/Collection since we already have a global Collection stub.
Fixes https://github.com/driftingly/rector-laravel/issues/239.
ArgumentAdderWithoutDefaultValue
.ArgumentAdderWithoutDefaultValue
withArgumentAdder
in Laravel 9 set, while setting the scope to only theClassMethod
. This satisfies both conditions of modifying the class extensions where appropriate (#239) and keeping method/static calls intact (https://github.com/driftingly/rector-laravel/issues/220).FailedJobProviderInterface
entirely, since an argument rename is needed in that case, but we don't have a rule in core Rector to only rename arguments, yet. I've kept the old test and improved the stub for future changes.tests/Sets/Laravel90/Source/Collection
since we already have a global Collection stub.