[DependencyInjection] Forbid container cloning

jakzal · jakzal · commit ba129041ba19 · 2015-07-16T11:10:53.000+01:00
diff --git a/src/Symfony/Component/DependencyInjection/Container.php b/src/Symfony/Component/DependencyInjection/Container.php
@@ -604,4 +604,8 @@ public static function underscore($id)
     {
         return strtolower(preg_replace(array('/([A-Z]+)([A-Z][a-z])/', '/([a-z\d])([A-Z])/'), array('\\1_\\2', '\\1_\\2'), strtr($id, '_', '.')));
     }
+
+    private function __clone()
+    {
+    }
 }
diff --git a/src/Symfony/Component/DependencyInjection/Tests/ContainerTest.php b/src/Symfony/Component/DependencyInjection/Tests/ContainerTest.php
@@ -662,6 +662,16 @@ public function testAlias()
         $this->assertTrue($c->has('alias'));
         $this->assertSame($c->get('alias'), $c->get('bar'));
     }
+
+    public function testThatCloningIsNotSupported()
+    {
+        $class = new \ReflectionClass('Symfony\Component\DependencyInjection\Container');
+        $clone = $class->getMethod('__clone');
+        if (PHP_VERSION_ID >= 540000) {
+            $this->assertFalse($class->isCloneable());
+        }
+        $this->assertTrue($clone->isPrivate());
+    }
 }
 
 class ProjectServiceContainer extends Container

Original file line number	Diff line number	Diff line change
`@@ -604,4 +604,8 @@ public static function underscore($id)`
`604`	`604`	`{`
`605`	`605`	`return strtolower(preg_replace(array('/([A-Z]+)([A-Z][a-z])/', '/([a-z\d])([A-Z])/'), array('\\1_\\2', '\\1_\\2'), strtr($id, '_', '.')));`
`606`	`606`	`}`
	`607`	`+`
	`608`	`+ private function __clone()`
	`609`	`+ {`
	`610`	`+ }`
`607`	`611`	`}`