Skip to content

Commit abe2a23

Browse files
Merge branch '4.0' into 4.1
* 4.0: [HttpKernel] Fix merging bindings for controllers' locators bumped Symfony version to 4.0.14 updated VERSION for 4.0.13 updated CHANGELOG for 4.0.13 bumped Symfony version to 3.4.14 updated VERSION for 3.4.13 updated CHANGELOG for 3.4.13 bumped Symfony version to 2.8.44
2 parents cdc4f7d + 097ac83 commit abe2a23

File tree

3 files changed

+59
-1
lines changed

3 files changed

+59
-1
lines changed

CHANGELOG-4.0.md

Lines changed: 34 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,40 @@ in 4.0 minor versions.
77
To get the diff for a specific change, go to https://github.com/symfony/symfony/commit/XXX where XXX is the change hash
88
To get the diff between two versions, go to https://github.com/symfony/symfony/compare/v4.0.0...v4.0.1
99

10+
* 4.0.13 (2018-07-23)
11+
12+
* bug #28005 [HttpKernel] Fixed templateExists on parse error of the template name (yceruto)
13+
* bug #27997 Serbo-Croatian has Serbian plural rule (kylekatarnls)
14+
* bug #26193 Fix false-positive deprecation notices for TranslationLoader and WriteCheckSessionHandler (iquito)
15+
* bug #27941 [WebProfilerBundle] Fixed icon alignment issue using Bootstrap 4.1.2 (jmsche)
16+
* bug #27937 [HttpFoundation] reset callback on StreamedResponse when setNotModified() is called (rubencm)
17+
* bug #27927 [HttpFoundation] Suppress side effects in 'get' and 'has' methods of NamespacedAttributeBag (webnet-fr)
18+
* bug #27923 [Form/Profiler] Massively reducing memory footprint of form profiling pages... (VincentChalnot)
19+
* bug #27918 [Console] correctly return parameter's default value on "--" (seschwar)
20+
* bug #27904 [Filesystem] fix lock file permissions (fritzmg)
21+
* bug #27903 [Lock] fix lock file permissions (fritzmg)
22+
* bug #27889 [Form] Replace .initialism with .text-uppercase. (vudaltsov)
23+
* bug #27902 Fix the detection of the Process new argument (stof)
24+
* bug #27885 [HttpFoundation] don't encode cookie name for BC (nicolas-grekas)
25+
* bug #27782 [DI] Fix dumping ignore-on-uninitialized references to synthetic services (nicolas-grekas)
26+
* bug #27435 [OptionResolver] resolve arrays (Doctrs)
27+
* bug #27728 [TwigBridge] Fix missing path and separators in loader paths list on debug:twig output (yceruto)
28+
* bug #27837 [PropertyInfo] Fix dock block lookup fallback loop (DerManoMann)
29+
* bug #27758 [WebProfilerBundle] Prevent toolbar links color override by css (alcalyn)
30+
* bug #27847 [Security] Fix accepting null as $uidKey in LdapUserProvider (louhde)
31+
* bug #27834 [DI] Don't show internal service id on binding errors (nicolas-grekas)
32+
* bug #27831 Check for Hyper terminal on all operating systems. (azjezz)
33+
* bug #27794 Add color support for Hyper terminal . (azjezz)
34+
* bug #27809 [HttpFoundation] Fix tests: new message for status 425 (dunglas)
35+
* bug #27618 [PropertyInfo] added handling of nullable types in PhpDoc (oxan)
36+
* bug #27659 [HttpKernel] Make AbstractTestSessionListener compatible with CookieClearingLogoutHandler (thewilkybarkid)
37+
* bug #27752 [Cache] provider does not respect option maxIdLength with versioning enabled (Constantine Shtompel)
38+
* bug #27776 [ProxyManagerBridge] Fix support of private services (bis) (nicolas-grekas)
39+
* bug #27714 [HttpFoundation] fix session tracking counter (nicolas-grekas, dmaicher)
40+
* bug #27747 [HttpFoundation] fix registration of session proxies (nicolas-grekas)
41+
* bug #27722 Redesign the Debug error page in prod (javiereguiluz)
42+
* bug #27716 [DI] fix dumping deprecated service in yaml (nicolas-grekas)
43+
1044
* 4.0.12 (2018-06-25)
1145

1246
* bug #27626 [TwigBundle][DX] Only add the Twig WebLinkExtension if the WebLink component is enabled (thewilkybarkid)

src/Symfony/Component/HttpKernel/DependencyInjection/RegisterControllerArgumentLocatorsPass.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ public function process(ContainerBuilder $container)
6262
while ($def instanceof ChildDefinition) {
6363
$def = $container->findDefinition($def->getParent());
6464
$class = $class ?: $def->getClass();
65-
$bindings = $def->getBindings();
65+
$bindings += $def->getBindings();
6666
}
6767
$class = $parameterBag->resolveValue($class);
6868

src/Symfony/Component/HttpKernel/Tests/DependencyInjection/RegisterControllerArgumentLocatorsPassTest.php

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313

1414
use PHPUnit\Framework\TestCase;
1515
use Symfony\Component\DependencyInjection\Argument\ServiceClosureArgument;
16+
use Symfony\Component\DependencyInjection\ChildDefinition;
1617
use Symfony\Component\DependencyInjection\ContainerAwareInterface;
1718
use Symfony\Component\DependencyInjection\ContainerAwareTrait;
1819
use Symfony\Component\DependencyInjection\ContainerBuilder;
@@ -339,6 +340,29 @@ public function testBindScalarValueToControllerArgument()
339340
$this->assertTrue($container->has((string) $reference));
340341
$this->assertSame('foo_val', $container->get((string) $reference));
341342
}
343+
344+
public function testBindingsOnChildDefinitions()
345+
{
346+
$container = new ContainerBuilder();
347+
$resolver = $container->register('argument_resolver.service')->addArgument(array());
348+
349+
$container->register('parent', ArgumentWithoutTypeController::class);
350+
351+
$container->setDefinition('child', (new ChildDefinition('parent'))
352+
->setBindings(array('$someArg' => new Reference('parent')))
353+
->addTag('controller.service_arguments')
354+
);
355+
356+
$pass = new RegisterControllerArgumentLocatorsPass();
357+
$pass->process($container);
358+
359+
$locator = $container->getDefinition((string) $resolver->getArgument(0))->getArgument(0);
360+
$this->assertInstanceOf(ServiceClosureArgument::class, $locator['child:fooAction']);
361+
362+
$locator = $container->getDefinition((string) $locator['child:fooAction']->getValues()[0])->getArgument(0);
363+
$this->assertInstanceOf(ServiceClosureArgument::class, $locator['someArg']);
364+
$this->assertEquals(new Reference('parent'), $locator['someArg']->getValues()[0]);
365+
}
342366
}
343367

344368
class RegisterTestController

0 commit comments

Comments
 (0)