symfony
diff --git a/‎src/Symfony/Component/DependencyInjection/Compiler/PassConfig.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Compiler/PassConfig.php
+1Lines changed: 1 addition & 0 deletions b/‎src/Symfony/Component/DependencyInjection/Compiler/PassConfig.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Compiler/PassConfig.php
+1Lines changed: 1 addition & 0 deletions
diff --git a/‎src/Symfony/Component/DependencyInjection/Tests/Compiler/IntegrationTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Tests/Compiler/IntegrationTest.php
+26Lines changed: 26 additions & 0 deletions b/‎src/Symfony/Component/DependencyInjection/Tests/Compiler/IntegrationTest.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Tests/Compiler/IntegrationTest.php
+26Lines changed: 26 additions & 0 deletions
diff --git a/‎src/Symfony/Component/DependencyInjection/Tests/Fixtures/containers/container9.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Tests/Fixtures/containers/container9.php
-1Lines changed: 0 additions & 1 deletion b/‎src/Symfony/Component/DependencyInjection/Tests/Fixtures/containers/container9.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Tests/Fixtures/containers/container9.php
-1Lines changed: 0 additions & 1 deletion
diff --git a/‎src/Symfony/Component/DependencyInjection/Tests/Fixtures/php/services9.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Tests/Fixtures/php/services9.php
+1-5Lines changed: 1 addition & 5 deletions b/‎src/Symfony/Component/DependencyInjection/Tests/Fixtures/php/services9.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Tests/Fixtures/php/services9.php
+1-5Lines changed: 1 addition & 5 deletions
diff --git a/‎src/Symfony/Component/DependencyInjection/Tests/Fixtures/php/services9_compiled.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Tests/Fixtures/php/services9_compiled.php
+3-8Lines changed: 3 additions & 8 deletions b/‎src/Symfony/Component/DependencyInjection/Tests/Fixtures/php/services9_compiled.php
Copy file name to clipboardExpand all lines: src/Symfony/Component/DependencyInjection/Tests/Fixtures/php/services9_compiled.php
+3-8Lines changed: 3 additions & 8 deletions
@@ -63,6 +63,7 @@ public function __construct()
                 new RemoveUnusedDefinitionsPass(),
             )),
             new CheckExceptionOnInvalidReferenceBehaviorPass(),
+            new CheckCircularReferencesPass(),
         );
     }
 
 
@@ -113,4 +113,30 @@ public function testProcessInlinesWhenThereAreMultipleReferencesButFromTheSameDe
         $this->assertFalse($container->hasDefinition('b'));
         $this->assertFalse($container->hasDefinition('c'), 'Service C was not inlined.');
     }
+
+    /**
+     * @expectedException \Symfony\Component\DependencyInjection\Exception\ServiceCircularReferenceException
+     */
+    public function testCircularReferencesCausedByMethodCallsAreDetectedDuringCompilation()
+    {
+        $container = new ContainerBuilder();
+        $container->setResourceTracking(false);
+
+        $container
+            ->register('foobar', '\stdClass')
+            ->addArgument(new Reference('foo'))
+        ;
+
+        $container
+            ->register('foo', '\stdClass')
+            ->addArgument(new Reference('bar'))
+        ;
+
+        $container
+            ->register('foo', '\stdClass')
+            ->addMethodCall('addFoobar', array(new Reference('foobar')))
+        ;
+
+        $container->compile();
+    }
 }
@@ -64,7 +64,6 @@
 ;
 $container
     ->register('baz', 'Baz')
-    ->addMethodCall('setFoo', array(new Reference('foo_with_inline')))
 ;
 $container
     ->register('request', 'Request')
 
@@ -80,11 +80,7 @@ protected function getBarService()
      */
     protected function getBazService()
     {
-        $this->services['baz'] = $instance = new \Baz();
-
-        $instance->setFoo($this->get('foo_with_inline'));
-
-        return $instance;
+        return $this->services['baz'] = new \Baz();
     }
 
     /**
 
@@ -99,11 +99,7 @@ protected function getBarService()
      */
     protected function getBazService()
     {
-        $this->services['baz'] = $instance = new \Baz();
-
-        $instance->setFoo($this->get('foo_with_inline'));
-
-        return $instance;
+        return $this->services['baz'] = new \Baz();
     }
 
     /**
@@ -227,12 +223,11 @@ protected function getFooBarService()
     protected function getFooWithInlineService()
     {
         $a = new \Bar();
-
-        $this->services['foo_with_inline'] = $instance = new \Foo();
-
         $a->pub = 'pub';
         $a->setBaz($this->get('baz'));
 
+        $this->services['foo_with_inline'] = $instance = new \Foo();
+
         $instance->setBar($a);
 
         return $instance;
Original file line number	Diff line number	Diff line change
`@@ -63,6 +63,7 @@ public function __construct()`
`63`	`63`	`new RemoveUnusedDefinitionsPass(),`
`64`	`64`	`)),`
`65`	`65`	`new CheckExceptionOnInvalidReferenceBehaviorPass(),`
	`66`	`+ new CheckCircularReferencesPass(),`
`66`	`67`	`);`
`67`	`68`	`}`
`68`	`69`
Original file line number	Diff line number	Diff line change
`@@ -64,7 +64,6 @@`
`64`	`64`	`;`
`65`	`65`	`$container`
`66`	`66`	`->register('baz', 'Baz')`
`67`		`- ->addMethodCall('setFoo', array(new Reference('foo_with_inline')))`
`68`	`67`	`;`
`69`	`68`	`$container`
`70`	`69`	`->register('request', 'Request')`
Original file line number	Diff line number	Diff line change
`@@ -80,11 +80,7 @@ protected function getBarService()`
`80`	`80`	`*/`
`81`	`81`	`protected function getBazService()`
`82`	`82`	`{`
`83`		`- $this->services['baz'] = $instance = new \Baz();`
`84`		`-`
`85`		`- $instance->setFoo($this->get('foo_with_inline'));`
`86`		`-`
`87`		`- return $instance;`
	`83`	`+ return $this->services['baz'] = new \Baz();`
`88`	`84`	`}`
`89`	`85`
`90`	`86`	`/**`