feature #60844 [DependencyInjection] Remove deprecated ContainerBuilder::getAutoconfiguredAttributes() (GromNaN)

nicolas-grekas · nicolas-grekas · commit 722b523b7124 · 2025-06-20T18:08:16.000+02:00
This PR was merged into the 8.0 branch. Discussion ---------- [DependencyInjection] Remove deprecated `ContainerBuilder::getAutoconfiguredAttributes()` | Q | A | ------------- | --- | Branch? | 8.0 | Bug fix? | no | New feature? | yes | Deprecations? | no | Issues | #60011 | License | MIT In #60011, the method `ContainerBuilder::getAutoconfiguredAttributes()` has been deprecated and replaced by `getAttributeAutoconfigurators()` to support multiple callbacks per attribute class. Commits ------- 7323657 Remove deprecated ContainerBuilder::getAutoconfiguredAttributes()
diff --git a/UPGRADE-8.0.md b/UPGRADE-8.0.md
@@ -109,6 +109,8 @@ DependencyInjection
     }
     ```
 
+ * Remove the `ContainerBuilder::getAutoconfiguredAttributes()` method, use `getAttributeAutoconfigurators()` instead to retrieve all the callbacks for a specific attribute class
+
 DoctrineBridge
 --------------
 
diff --git a/src/Symfony/Component/DependencyInjection/CHANGELOG.md b/src/Symfony/Component/DependencyInjection/CHANGELOG.md
@@ -5,6 +5,7 @@ CHANGELOG
 ---
 
  * Remove `#[TaggedIterator]` and `#[TaggedLocator]` attributes, replaced by `#[AutowireLocator]` and `#[AutowireIterator]`
+ * Remove `ContainerBuilder::getAutoconfiguredAttributes()`, replaced by `ContainerBuilder::getAttributeAutoconfigurators()`
 
 7.4
 ---
diff --git a/src/Symfony/Component/DependencyInjection/ContainerBuilder.php b/src/Symfony/Component/DependencyInjection/ContainerBuilder.php
@@ -1488,27 +1488,6 @@ public function getAutoconfiguredInstanceof(): array
         return $this->autoconfiguredInstanceof;
     }
 
-    /**
-     * @return array<class-string, callable>
-     *
-     * @deprecated Use {@see getAttributeAutoconfigurators()} instead
-     */
-    public function getAutoconfiguredAttributes(): array
-    {
-        trigger_deprecation('symfony/dependency-injection', '7.3', 'The "%s()" method is deprecated, use "getAttributeAutoconfigurators()" instead.', __METHOD__);
-
-        $autoconfiguredAttributes = [];
-        foreach ($this->autoconfiguredAttributes as $attribute => $configurators) {
-            if (count($configurators) > 1) {
-                throw new LogicException(\sprintf('The "%s" attribute has %d configurators. Use "getAttributeAutoconfigurators()" to get all of them.', $attribute, count($configurators)));
-            }
-
-            $autoconfiguredAttributes[$attribute] = $configurators[0];
-        }
-
-        return $autoconfiguredAttributes;
-    }
-
     /**
      * @return array<class-string, callable[]>
      */
diff --git a/src/Symfony/Component/DependencyInjection/Tests/ContainerBuilderTest.php b/src/Symfony/Component/DependencyInjection/Tests/ContainerBuilderTest.php
@@ -842,25 +842,6 @@ public function testMergeAttributeAutoconfiguration()
         $this->assertSame([AsTaggedItem::class => [$c1, $c2]], $container->getAttributeAutoconfigurators());
     }
 
-    /**
-     * @group legacy
-     */
-    public function testGetAutoconfiguredAttributes()
-    {
-        $container = new ContainerBuilder();
-        $container->registerAttributeForAutoconfiguration(AsTaggedItem::class, $c = static function () {});
-
-        $this->expectUserDeprecationMessage('Since symfony/dependency-injection 7.3: The "Symfony\Component\DependencyInjection\ContainerBuilder::getAutoconfiguredAttributes()" method is deprecated, use "getAttributeAutoconfigurators()" instead.');
-        $configurators = $container->getAutoconfiguredAttributes();
-        $this->assertSame($c, $configurators[AsTaggedItem::class]);
-
-        // Method call fails with more than one configurator for a given attribute
-        $container->registerAttributeForAutoconfiguration(AsTaggedItem::class, $c = static function () {});
-
-        $this->expectException(LogicException::class);
-        $container->getAutoconfiguredAttributes();
-    }
-
     public function testResolveEnvValues()
     {
         $_ENV['DUMMY_ENV_VAR'] = 'du%%y';

Original file line number	Diff line number	Diff line change
`@@ -109,6 +109,8 @@ DependencyInjection`
`109`	`109`	`}`
`110`	`110`	```
`111`	`111`
	`112`	+ * Remove the `ContainerBuilder::getAutoconfiguredAttributes()` method, use `getAttributeAutoconfigurators()` instead to retrieve all the callbacks for a specific attribute class
	`113`	`+`
`112`	`114`	`DoctrineBridge`
`113`	`115`	`--------------`
`114`	`116`