[TASK] Take container.excluded tag into account

Author: sabbelasichon

src/Service/XmlServiceMapFactory.php

@@ -43,6 +43,12 @@ public function create(): ServiceMap
 				continue;
 			}
 
+			$tags = $this->createTags($def);
+
+			if (in_array('container.excluded', $tags, true)) {
+				continue;
+			}
+
 			$serviceDefinition = new ServiceDefinition(
 				strpos((string) $attrs->id, '.') === 0 ? substr((string) $attrs->id, 1) : (string) $attrs->id,
 				isset($attrs->class) ? (string) $attrs->class : null,
@@ -75,4 +81,26 @@ public function create(): ServiceMap
 		return new DefaultServiceMap($serviceDefinitions);
 	}
 
+	/**
+	 * @return string[]
+	 */
+	private function createTags(?SimpleXMLElement $def): array
+	{
+		if (!isset($def->tag)) {
+			return [];
+		}
+
+		$tagNames = [];
+
+		foreach ($def->tag as $tag) {
+			$attributes = $tag->attributes();
+			if (!isset($attributes->name)) {
+				continue;
+			}
+			$tagNames[] = (string) $attributes->name;
+		}
+
+		return $tagNames;
+	}
+
 }

tests/Unit/Fixtures/container.xml

@@ -8,7 +8,13 @@
 		<service id="publicNotTrue" class="Foo" public="false"></service>
 		<service id="public" class="Foo" public="true"></service>
 		<service id="synthetic" class="Foo" synthetic="true"></service>
-		<service id="alias" class="Bar" alias="withClass"></service>
+		<service id="alias" class="Bar" alias="withClass">
+			<tag name="event.listener"/>
+			<tag name="foo"/>
+		</service>
 		<service id="private" class="Foo" public="false"></service>
+		<service id="excluded" class="Foo" public="false">
+			<tag name="container.excluded"/>
+		</service>
 	</services>
 </container>

tests/Unit/Service/XmlServiceMapFactoryTest.php

@@ -38,6 +38,9 @@ public function testThatServiceDefinitionsAreConstructedSuccessfully(): void
 
 		$serviceDefinition = $serviceMap->getServiceDefinitionById('public');
 		self::assertNotNull($serviceDefinition);
+
+		$serviceDefinitionExcluded = $serviceMap->getServiceDefinitionById('excluded');
+		self::assertNull($serviceDefinitionExcluded);
 	}
 
 	private function createServiceMap(?string $containerXmlPath): ServiceMap