Skip to content

Commit 3db1407

Browse files
Merge branch '2.7' into 2.8
* 2.7: [Console] Add missing `@require` annotation in test Fix merge [appveyor] Fix failure reporting [#17634] move DebugBundle license file backport GlobTest from 2.7 branch Move licenses according to new best practices [FrameworkBundle] Remove unused code in test [2.3] Fixed an undefined variable in Glob::toRegex simplified a test fix container cache key generation [Form] fix option name in changelog [Translation] Add resources from fallback locale [DependencyInjection] enforce tags to have a name [YAML] Refine the return value of Yaml::parse() Conflicts: src/Symfony/Component/DependencyInjection/Tests/Loader/YamlFileLoaderTest.php
2 parents 380a0f1 + bf36e67 commit 3db1407

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed
File renamed without changes.

Tests/DependencyInjection/FrameworkExtensionTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -513,7 +513,7 @@ protected function createContainer(array $data = array())
513513

514514
protected function createContainerFromFile($file, $data = array())
515515
{
516-
$cacheKey = md5($file.serialize($data));
516+
$cacheKey = md5(get_class($this).$file.serialize($data));
517517
if (isset(self::$containerCache[$cacheKey])) {
518518
return self::$containerCache[$cacheKey];
519519
}

Tests/DependencyInjection/PhpFrameworkExtensionTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ protected function loadFromFile(ContainerBuilder $container, $file)
2828
*/
2929
public function testAssetsCannotHavePathAndUrl()
3030
{
31-
$container = $this->createContainerFromClosure(function ($container) {
31+
$this->createContainerFromClosure(function ($container) {
3232
$container->loadFromExtension('framework', array(
3333
'assets' => array(
3434
'base_urls' => 'http://cdn.example.com',
@@ -43,7 +43,7 @@ public function testAssetsCannotHavePathAndUrl()
4343
*/
4444
public function testAssetPackageCannotHavePathAndUrl()
4545
{
46-
$container = $this->createContainerFromClosure(function ($container) {
46+
$this->createContainerFromClosure(function ($container) {
4747
$container->loadFromExtension('framework', array(
4848
'assets' => array(
4949
'packages' => array(

0 commit comments

Comments
 (0)