[DependencyInjection] Remove TaggedIterator and TaggedLocator attributes#21104

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Open

javiereguiluz wants to merge1 commit intosymfony:8.0

base:8.0

Choose a base branch

fromjaviereguiluz:fix_21098

Open

[DependencyInjection] Remove TaggedIterator and TaggedLocator attributes#21104

javiereguiluz wants to merge1 commit intosymfony:8.0fromjaviereguiluz:fix_21098

Conversation

Copy link

Member

javiereguiluz commentedJun 20, 2025

Fixes#21098.

Inservice_container/tags.rst there are some occurrences of a similar but different class:

use Symfony\Component\DependencyInjection\Argument\TaggedIteratorArgument;

It's needed because i's used by thetagged_iterator() method (which is not deprecated).

[DependencyInjection] Remove TaggedIterator and TaggedLocator attributes

db1b4e7

javiereguiluz added this to the8.0 milestone

Jun 20, 2025

javiereguiluz added the DependencyInjection label

Jun 20, 2025

carsonbot added the Status: Needs Review label

Jun 20, 2025

Copy link

Member

GromNaN commentedJun 20, 2025

Inservice_container/tags.rst there are some occurrences of a similar but different class:
use Symfony\Component\DependencyInjection\Argument\TaggedIteratorArgument;
It's needed because it's used by thetagged_iterator() method (which is not deprecated).

I think it's fine to remove it as the class is not actually used in the code example.

It seems you haven't seen my PR:#21100.