Skip to content

[DependencyInjection] Added support for variadics in named arguments #24937

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.

Sign up for GitHub

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

Closed
wants to merge 20 commits into from
Closed

[DependencyInjection] Added support for variadics in named arguments #24937

Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
1e0d7e4
[DependencyInjection] Added support for variadics in named arguments
Nov 12, 2017
1859650
[DependencyInjection] Added support for variadics in named arguments
Nov 12, 2017
31b8935
Removed variadic argument from NamedArgumentsDummy
Nov 12, 2017
625b3d3
Introduce NamedArgumentsVariadicsDummy to test variadics named argume…
Nov 12, 2017
3b55332
Test variadics named arguments only on PHP 5.6 and above, also don't …
Nov 12, 2017
f0f4c22
Removed blank line.
Nov 12, 2017
9dbaa09
Removed "strict_types".
Nov 13, 2017
e2c3e22
Hook into $parameters foreach loop instead of "last argument" enforcing.
Nov 13, 2017
28305f2
Removed unused "use".
Nov 13, 2017
25cb9bb
CS fixes.
Nov 14, 2017
62db734
Do not require PHP 5.6 version.
Nov 15, 2017
9961ea5
[DependencyInjection] Added support for variadics in named arguments
Nov 12, 2017
c849e5a
[DependencyInjection] Added support for variadics in named arguments
Nov 12, 2017
44f71c8
Removed variadic argument from NamedArgumentsDummy
Nov 12, 2017
8e55fb2
Removed unused "use".
Nov 13, 2017
6c0a574
CS fixes.
Nov 14, 2017
c749c0c
Do not require PHP 5.6 version.
Nov 15, 2017
2e8ebf7
Added another constructor parameter to improve test.
Nov 19, 2017
86c2db2
CS fix.
PabloKowalczyk Nov 19, 2017
ca5453e
Removed PHP requirement.
PabloKowalczyk Nov 20, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Introduce NamedArgumentsVariadicsDummy to test variadics named argume… 
…nts.
  • Loading branch information
@PabloKowalczyk
pkowalczyk authored and PabloKowalczyk committed Nov 20, 2017
commit 625b3d3033c84cd69a84976c430d0d9bbd41f5d4
13 changes: 13 additions & 0 deletions src/Symfony/Component/DependencyInjection/Tests/Fixtures/NamedArgumentsVariadicsDummy.php
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
<?php

declare(strict_types=1);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be removed


namespace Symfony\Component\DependencyInjection\Tests\Fixtures;

class NamedArgumentsVariadicsDummy
{
public function __construct(...$variadics)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for the tests, could you add another parameter before the variadic one so we show it works?

{

}
}