feature #37198 [FrameworkBundle] Add support for tagged_iterator/tagged_locator in unused tags util (fabpot)

fabpot · fabpot · commit 88a06c936c75 · 2020-06-11T11:10:19.000+02:00
This PR was merged into the 5.2-dev branch. Discussion ---------- [FrameworkBundle] Add support for tagged_iterator/tagged_locator in unused tags util | Q | A | ------------- | --- | Branch? | master  | Bug fix? | yes | New feature? | no  | Deprecations? | no  | Tickets | n/a  | License | MIT | Doc PR | n/a /cc @stof :) Commits ------- 164ca90 [FrameworkBundle] Add support for tagged_iterator/tagged_locator in unused tags util
diff --git a/src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Compiler/UnusedTagsPassUtils.php b/src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Compiler/UnusedTagsPassUtils.php
@@ -46,6 +46,11 @@ public static function getDefinedTags(): array
                     $tags[$match] = true;
                 }
             }
+            if (preg_match_all("{tagged_(?:locator|iterator)\('([^']+)'}", $contents, $matches)) {
+                foreach ($matches[1] as $match) {
+                    $tags[$match] = true;
+                }
+            }
         }
 
         // get all tags used in findTaggedServiceIds calls()

Original file line number	Diff line number	Diff line change
`@@ -46,6 +46,11 @@ public static function getDefinedTags(): array`
`46`	`46`	`$tags[$match] = true;`
`47`	`47`	`}`
`48`	`48`	`}`
	`49`	`+ if (preg_match_all("{tagged_(?:locator\|iterator)\('([^']+)'}", $contents, $matches)) {`
	`50`	`+ foreach ($matches[1] as $match) {`
	`51`	`+ $tags[$match] = true;`
	`52`	`+ }`
	`53`	`+ }`
`49`	`54`	`}`
`50`	`55`
`51`	`56`	`// get all tags used in findTaggedServiceIds calls()`