Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Resolver: autowire variadic parameters #244

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jiripudil
Copy link
Contributor

  • new feature
  • BC break: no
  • doc PR: todo

Hi there, in addition to autowiring arrays of services via annotations, I think it might be useful to autowire variadic arguments as well – it's a very similar mechanism, but with added type safety on the language level:

final class ApplicationPermissionsConfigurator
{
    /** @var ModulePermissionsConfigurator[] */
    private array $moduleConfigurators;

    public function __construct(ModulePermissionsConfigurator ...$moduleConfigurators) {
        $this->moduleConfigurators = $moduleConfigurators;
    }
}

@dg dg force-pushed the master branch 9 times, most recently from 47688f3 to 4c651dc Compare November 4, 2020 15:16
@dg dg force-pushed the master branch 4 times, most recently from e14a8ec to 5db4398 Compare November 6, 2020 00:54
@dg dg force-pushed the master branch 8 times, most recently from 9b4240b to e639ccf Compare November 25, 2020 22:43
@dg dg force-pushed the master branch 4 times, most recently from 997b4fc to a1e346d Compare December 20, 2020 22:24
@dg dg force-pushed the master branch 2 times, most recently from d09b7b3 to 9a6cd01 Compare January 31, 2021 22:05
@dg dg force-pushed the master branch 3 times, most recently from 680bc12 to 5066242 Compare October 15, 2023 12:34
@dg dg force-pushed the master branch 3 times, most recently from b109822 to 7f11e6e Compare November 3, 2023 00:16
@dg dg force-pushed the master branch 2 times, most recently from ef39d2d to f729b1e Compare December 14, 2023 12:50
@dg dg force-pushed the master branch 9 times, most recently from a43bb2c to 71a91be Compare February 7, 2024 10:11
@dg dg force-pushed the master branch 2 times, most recently from 51cfed7 to 9c4af52 Compare February 12, 2024 17:55
@dg dg force-pushed the master branch 3 times, most recently from 0a1f0ab to 8c392ab Compare April 29, 2024 11:53
@dg dg force-pushed the master branch 2 times, most recently from 96c09bc to 7a40f39 Compare May 16, 2024 13:31
@dg dg force-pushed the master branch 2 times, most recently from 8655bcb to 59cf699 Compare December 2, 2024 05:13
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.

2 participants