Issue 133: Use get_attributes() instead of ->getAttributeInstances().

Author: donquixote

src/Compiler/InsertParametricArgumentsPlaceholdersPass.php

@@ -14,6 +14,7 @@
 use Symfony\Component\DependencyInjection\ChildDefinition;
 use Symfony\Component\DependencyInjection\Compiler\AbstractRecursivePass;
 use Symfony\Component\DependencyInjection\Definition;
+use function Ock\ClassDiscovery\get_attributes;
 
 /**
  * Inserts placeholders for parametric arguments.
@@ -78,10 +79,10 @@ protected function processDefinition(Definition $definition): void {
    * @return \Ock\DependencyInjection\Parametric\PlaceholderInterface|null
    */
   protected function createArgumentPlaceholder(ParameterReflection $parameter): ?PlaceholderInterface {
-    foreach ($parameter->getAttributeInstances(GetParametricArgument::class) as $attribute) {
+    foreach (get_attributes($parameter, GetParametricArgument::class) as $attribute) {
       return $this->createParametricArgumentPlaceholder($attribute, $parameter);
     }
-    foreach ($parameter->getAttributeInstances(GetParametricService::class) as $attribute) {
+    foreach (get_attributes($parameter, GetParametricService::class) as $attribute) {
       return $this->createParametricServicePlaceholder($attribute, $parameter);
     }
     return NULL;

src/Compiler/ServiceModifierAttributePass.php

@@ -6,6 +6,7 @@
 use Ock\DependencyInjection\ServiceDefinitionUtil;
 use Symfony\Component\DependencyInjection\Compiler\AbstractRecursivePass;
 use Symfony\Component\DependencyInjection\Definition;
+use function Ock\ClassDiscovery\get_attributes;
 
 /**
  * @see \Symfony\Component\DependencyInjection\Compiler\AttributeAutoconfigurationPass
@@ -37,7 +38,7 @@ protected function processDefinition(Definition $definition): void {
     if ($reflector === null) {
       return;
     }
-    $attributes = $reflector->getAttributeInstances(ServiceModifierInterface::class);
+    $attributes = get_attributes($reflector, ServiceModifierInterface::class);
     if (!$attributes) {
       return;
     }

src/Inspector/ClassInspector_ConditionDecorator.php

@@ -8,6 +8,7 @@
 use Ock\ClassDiscovery\Reflection\ClassReflection;
 use Ock\DependencyInjection\Attribute\ServiceConditionAttributeInterface;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
+use function Ock\ClassDiscovery\get_attributes;
 
 /**
  * Registers services for classes and methods with #[Service] attribute.
@@ -40,7 +41,7 @@ public static function create(ClassInspectorInterface $decorated): static {
    * {@inheritdoc}
    */
   public function findInClass(ClassReflection $classReflection): \Iterator {
-    $attributes = $classReflection->getAttributeInstances(ServiceConditionAttributeInterface::class);
+    $attributes = get_attributes($classReflection, ServiceConditionAttributeInterface::class);
     if ($attributes) {
       $result = null;
       yield 'check condition' => static function (ContainerBuilder $container) use (&$result, $attributes) {

src/Inspector/ClassInspector_SymfonyAsAliasAttributeDecorator.php

@@ -11,6 +11,7 @@
 use Symfony\Component\DependencyInjection\Attribute\AsAlias;
 use Symfony\Component\DependencyInjection\Definition;
 use Symfony\Component\DependencyInjection\Exception\LogicException;
+use function Ock\ClassDiscovery\get_attributes;
 
 /**
  * Decorator to support the AsAlias attribute.
@@ -43,7 +44,7 @@ public static function create(ClassInspectorInterface $decorated): self {
    * {@inheritdoc}
    */
   public function findInClass(ClassReflection $classReflection): \Iterator {
-    $attributes = $classReflection->getAttributeInstances(AsAlias::class);
+    $attributes = get_attributes($classReflection, AsAlias::class);
     if (!$attributes) {
       yield from $this->decorated->findInClass($classReflection);
       return;

src/Inspector/FactoryInspector_ConditionDecorator.php

@@ -9,6 +9,7 @@
 use Ock\ClassDiscovery\Reflection\MethodReflection;
 use Ock\DependencyInjection\Attribute\ServiceConditionAttributeInterface;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
+use function Ock\ClassDiscovery\get_attributes;
 
 /**
  * Registers services for classes and methods with #[Service] attribute.
@@ -41,7 +42,7 @@ public static function create(FactoryInspectorInterface $decorated): static {
    * {@inheritdoc}
    */
   public function findInFactory(ClassReflection|MethodReflection $reflector): \Iterator {
-    $attributes = $reflector->getAttributeInstances(ServiceConditionAttributeInterface::class);
+    $attributes = get_attributes($reflector, ServiceConditionAttributeInterface::class);
     if ($attributes) {
       $result = null;
       yield 'check condition' => static function (ContainerBuilder $container) use (&$result, $attributes) {

src/Inspector/FactoryInspector_ServiceAttribute.php

@@ -9,6 +9,7 @@
 use Ock\ClassDiscovery\Reflection\FactoryReflectionInterface;
 use Ock\DependencyInjection\Attribute\ServiceBase;
 use Symfony\Component\DependencyInjection\Reference;
+use function Ock\ClassDiscovery\get_attributes;
 
 /**
  * Registers services for classes and methods with #[Service] attribute.
@@ -23,7 +24,7 @@ class FactoryInspector_ServiceAttribute implements FactoryInspectorInterface {
    * {@inheritdoc}
    */
   public function findInFactory(FactoryReflectionInterface $reflector): \Iterator {
-    $attributes = $reflector->getAttributeInstances(ServiceBase::class);
+    $attributes = get_attributes($reflector, ServiceBase::class);
     if (!$attributes) {
       return;
     }