Fixing missing abstract attribute in XmlDumper

weaverryan · weaverryan · commit ad81c6e6deeb · 2017-05-25T08:51:52.000-04:00
Caused mis-reporting of abstract key (always no) in debug:container
diff --git a/src/Symfony/Component/DependencyInjection/Dumper/XmlDumper.php b/src/Symfony/Component/DependencyInjection/Dumper/XmlDumper.php
@@ -191,6 +191,10 @@ private function addService($definition, $id, \DOMElement $parent)
             $service->appendChild($factory);
         }
 
+        if ($definition->isAbstract()) {
+            $service->setAttribute('abstract', 'true');
+        }
+
         if ($callable = $definition->getConfigurator()) {
             $configurator = $this->document->createElement('configurator');
 
diff --git a/src/Symfony/Component/DependencyInjection/Tests/Dumper/XmlDumperTest.php b/src/Symfony/Component/DependencyInjection/Tests/Dumper/XmlDumperTest.php
@@ -184,4 +184,12 @@ public function testDumpInlinedServices()
 
         $this->assertEquals(file_get_contents(self::$fixturesPath.'/xml/services21.xml'), $dumper->dump());
     }
+
+    public function testDumpAbstractServices()
+    {
+        $container = include self::$fixturesPath.'/containers/container-abstract.php';
+        $dumper = new XmlDumper($container);
+
+        $this->assertEquals(file_get_contents(self::$fixturesPath.'/xml/services-abstract.xml'), $dumper->dump());
+    }
 }
diff --git a/src/Symfony/Component/DependencyInjection/Tests/Fixtures/containers/container-abstract.php b/src/Symfony/Component/DependencyInjection/Tests/Fixtures/containers/container-abstract.php
@@ -0,0 +1,12 @@
+<?php
+
+use Symfony\Component\DependencyInjection\ContainerBuilder;
+
+$container = new ContainerBuilder();
+
+$container
+    ->register('foo', 'Foo')
+    ->setAbstract(true)
+;
+
+return $container;
diff --git a/src/Symfony/Component/DependencyInjection/Tests/Fixtures/xml/services-abstract.xml b/src/Symfony/Component/DependencyInjection/Tests/Fixtures/xml/services-abstract.xml
@@ -0,0 +1,6 @@
+<?xml version="1.0" encoding="utf-8"?>
+<container xmlns="http://symfony.com/schema/dic/services" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://symfony.com/schema/dic/services http://symfony.com/schema/dic/services/services-1.0.xsd">
+  <services>
+    <service id="foo" class="Foo" abstract="true"/>
+  </services>
+</container>

Original file line number	Diff line number	Diff line change
`@@ -191,6 +191,10 @@ private function addService($definition, $id, \DOMElement $parent)`
`191`	`191`	`$service->appendChild($factory);`
`192`	`192`	`}`
`193`	`193`
	`194`	`+ if ($definition->isAbstract()) {`
	`195`	`+ $service->setAttribute('abstract', 'true');`
	`196`	`+ }`
	`197`	`+`
`194`	`198`	`if ($callable = $definition->getConfigurator()) {`
`195`	`199`	`$configurator = $this->document->createElement('configurator');`
`196`	`200`
Original file line number	Diff line number	Diff line change
`@@ -184,4 +184,12 @@ public function testDumpInlinedServices()`
`184`	`184`
`185`	`185`	`$this->assertEquals(file_get_contents(self::$fixturesPath.'/xml/services21.xml'), $dumper->dump());`
`186`	`186`	`}`
	`187`	`+`
	`188`	`+ public function testDumpAbstractServices()`
	`189`	`+ {`
	`190`	`+ $container = include self::$fixturesPath.'/containers/container-abstract.php';`
	`191`	`+ $dumper = new XmlDumper($container);`
	`192`	`+`
	`193`	`+ $this->assertEquals(file_get_contents(self::$fixturesPath.'/xml/services-abstract.xml'), $dumper->dump());`
	`194`	`+ }`
`187`	`195`	`}`