Remove dependency on symfony/config

It's worth a shot. If it can be done without the use of symfony/config, I'm all in favor of dropping the dependency and closing the core issue.

I'll dig into this for a day and see what I can do. If we can pull it off, then that basically nullifies a blocking core issue, and that's certainly worth something.

OK, took some real tweaking around with iterators, but I think this should do the trick.

Crediting @effulgentsia for this idea.

Reviewed. IMHO this is missing a comment to ensure we remove the new logic/complexity whenever Drupal core does adopt symfony/config, which seems likely to happen at some point.

This does avoid the dependency on symfony/config, but confirms the observation I posted at #3321933-11: Remove dependency on symfony/finder → which was the reason for opening 📌 Add symfony/config to core's dependencies for package_manager Closed: duplicate :

→ I don't see how we can avoid using symfony/config if we don't want to make \Drupal\package_manager\PackageManagerServiceProvider::register() a lot more complicated.

Once a @todo has been added to document that this non-trivial complexity can be removed altogether if we adopt a widely used Symfony package that will remove the need for lots of contrib modules to do similar things to make autowiring work as expected in Drupal (see #3111008-2: Use native Symfony YamlLoader + Config → ), I will RTBC.

Applied your suggested comment and returning to you for review, Wim.

Thanks for that context, @phenaproxima! :D I bet it was @gabesullice who knew about static anonymous functions 🤯, not me! 😅

Automatically closed - issue fixed for 2 weeks with no activity.