Minor tweaks
This commit is contained in:
parent
783abab7ae
commit
80ed2c5173
@ -36,7 +36,6 @@ class CachePoolPass implements CompilerPassInterface
|
||||
}
|
||||
$seed .= '.'.$container->getParameter('kernel.name').'.'.$container->getParameter('kernel.environment');
|
||||
|
||||
$aliases = $container->getAliases();
|
||||
$attributes = array(
|
||||
'provider',
|
||||
'namespace',
|
||||
@ -57,9 +56,9 @@ class CachePoolPass implements CompilerPassInterface
|
||||
$tags[0]['namespace'] = $this->getNamespace($seed, $id);
|
||||
}
|
||||
if (isset($tags[0]['clearer'])) {
|
||||
$clearer = strtolower($tags[0]['clearer']);
|
||||
while (isset($aliases[$clearer])) {
|
||||
$clearer = (string) $aliases[$clearer];
|
||||
$clearer = $tags[0]['clearer'];
|
||||
while ($container->hasAlias($clearer)) {
|
||||
$clearer = (string) $container->getAlias($clearer);
|
||||
}
|
||||
} else {
|
||||
$clearer = null;
|
||||
|
@ -317,26 +317,26 @@ abstract class FrameworkExtensionTest extends TestCase
|
||||
$packages = $container->getDefinition('assets.packages');
|
||||
|
||||
// default package
|
||||
$defaultPackage = $container->getDefinition($packages->getArgument(0));
|
||||
$defaultPackage = $container->getDefinition((string) $packages->getArgument(0));
|
||||
$this->assertUrlPackage($container, $defaultPackage, array('http://cdn.example.com'), 'SomeVersionScheme', '%%s?version=%%s');
|
||||
|
||||
// packages
|
||||
$packages = $packages->getArgument(1);
|
||||
$this->assertCount(5, $packages);
|
||||
|
||||
$package = $container->getDefinition($packages['images_path']);
|
||||
$package = $container->getDefinition((string) $packages['images_path']);
|
||||
$this->assertPathPackage($container, $package, '/foo', 'SomeVersionScheme', '%%s?version=%%s');
|
||||
|
||||
$package = $container->getDefinition($packages['images']);
|
||||
$package = $container->getDefinition((string) $packages['images']);
|
||||
$this->assertUrlPackage($container, $package, array('http://images1.example.com', 'http://images2.example.com'), '1.0.0', '%%s?version=%%s');
|
||||
|
||||
$package = $container->getDefinition($packages['foo']);
|
||||
$package = $container->getDefinition((string) $packages['foo']);
|
||||
$this->assertPathPackage($container, $package, '', '1.0.0', '%%s-%%s');
|
||||
|
||||
$package = $container->getDefinition($packages['bar']);
|
||||
$package = $container->getDefinition((string) $packages['bar']);
|
||||
$this->assertUrlPackage($container, $package, array('https://bar2.example.com'), 'SomeVersionScheme', '%%s?version=%%s');
|
||||
|
||||
$package = $container->getDefinition($packages['bar_version_strategy']);
|
||||
$package = $container->getDefinition((string) $packages['bar_version_strategy']);
|
||||
$this->assertEquals('assets.custom_version_strategy', (string) $package->getArgument(1));
|
||||
}
|
||||
|
||||
@ -346,7 +346,7 @@ abstract class FrameworkExtensionTest extends TestCase
|
||||
$packages = $container->getDefinition('assets.packages');
|
||||
|
||||
// default package
|
||||
$defaultPackage = $container->getDefinition($packages->getArgument(0));
|
||||
$defaultPackage = $container->getDefinition((string) $packages->getArgument(0));
|
||||
$this->assertEquals('assets.custom_version_strategy', (string) $defaultPackage->getArgument(1));
|
||||
}
|
||||
|
||||
|
@ -403,7 +403,7 @@ class SecurityExtension extends Extension
|
||||
$config->replaceArgument(8, isset($firewall['access_denied_handler']) ? $firewall['access_denied_handler'] : null);
|
||||
$config->replaceArgument(9, isset($firewall['access_denied_url']) ? $firewall['access_denied_url'] : null);
|
||||
|
||||
$container->setAlias(new Alias('security.user_checker.'.$id, false), $firewall['user_checker']);
|
||||
$container->setAlias('security.user_checker.'.$id, new Alias($firewall['user_checker'], false));
|
||||
|
||||
foreach ($this->factories as $position) {
|
||||
foreach ($position as $factory) {
|
||||
|
@ -73,7 +73,7 @@ abstract class CompleteConfigurationTest extends \PHPUnit_Framework_TestCase
|
||||
$arguments = $contextDef->getArguments();
|
||||
$listeners[] = array_map(function ($ref) { return (string) $ref; }, $arguments['index_0']);
|
||||
|
||||
$configDef = $container->getDefinition($arguments['index_2']);
|
||||
$configDef = $container->getDefinition((string) $arguments['index_2']);
|
||||
$configs[] = array_values($configDef->getArguments());
|
||||
}
|
||||
|
||||
|
@ -287,7 +287,7 @@ class PhpDumperTest extends \PHPUnit_Framework_TestCase
|
||||
$container = include self::$fixturesPath.'/containers/container24.php';
|
||||
$dumper = new PhpDumper($container);
|
||||
|
||||
$this->assertEquals(file_get_contents(self::$fixturesPath.'/php/services24.php'), $dumper->dump());
|
||||
$this->assertStringEqualsFile(self::$fixturesPath.'/php/services24.php', $dumper->dump());
|
||||
}
|
||||
|
||||
public function testEnvParameter()
|
||||
|
Reference in New Issue
Block a user