Merge pull request #41 from jdeniau/patch-1

Fix issue while loading a container

Author: lookyman

src/Symfony/XmlServiceMapFactory.php

@@ -2,7 +2,7 @@
 
 namespace PHPStan\Symfony;
 
-use function simplexml_load_file;
+use function simplexml_load_string;
 use function sprintf;
 use function strpos;
 use function substr;
@@ -20,7 +20,12 @@ public function __construct(string $containerXml)
 
 	public function create(): ServiceMap
 	{
-		$xml = @simplexml_load_file($this->containerXml);
+		$fileContents = file_get_contents($this->containerXml);
+		if ($fileContents === false) {
+			throw new XmlContainerNotExistsException(sprintf('Container %s does not exist or cannot be parsed', $this->containerXml));
+		}
+
+		$xml = @simplexml_load_string($fileContents);
 		if ($xml === false) {
 			throw new XmlContainerNotExistsException(sprintf('Container %s does not exist or cannot be parsed', $this->containerXml));
 		}

tests/Symfony/ServiceMapTest.php

@@ -17,6 +17,14 @@ public function testGetService(string $id, callable $validator): void
 		$validator($factory->create()->getService($id));
 	}
 
+	public function testGetContainerEscapedPath(): void
+	{
+		$factory = new XmlServiceMapFactory(__DIR__ . '/containers/bugfix%2Fcontainer.xml');
+		$serviceMap = $factory->create();
+
+		self::assertNotNull($serviceMap->getService('withClass'));
+	}
+
 	public function getServiceProvider(): Iterator
 	{
 		yield [

tests/Symfony/containers/bugfix%2Fcontainer.xml

@@ -0,0 +1,13 @@
+<?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></service><!-- without id -->
+    <service id="withoutClass"></service>
+    <service id="withClass" class="Foo"></service>
+    <service id="withoutPublic" class="Foo"></service>
+    <service id="publicNotFalse" class="Foo" public="true"></service>
+    <service id="private" class="Foo" public="false"></service>
+    <service id="synthetic" class="Foo" synthetic="true"></service>
+    <service id="alias" class="Bar" alias="withClass"></service>
+  </services>
+</container>