Commit 385efaf
committed
minor symfony#59820 [DependencyInjection] Fix phpdoc for $configurator in Autoconfigure attribute (jfranclin)
This PR was merged into the 7.2 branch.
Discussion
----------
[DependencyInjection] Fix phpdoc for $configurator in Autoconfigure attribute
| Q | A
| ------------- | ---
| Branch? | 7.2
| Bug fix? | yes
| New feature? | no
| Deprecations? | no
| Issues | Fix symfony#59821
| License | MIT
Since SF7.1, in Autoconfigure attribute, the $configurator phpdoc seems wrong :
` * `@param` array<class-string, string>|string|null $configurator A PHP function, reference or an array containing a class/Reference and a method to call after the service is fully initialized`
The doc is asking for a class-string **key** and a string **value**. However, this is not always the case because the first parameter could be a service, like ``@foo`` which is not a class-string. Moreover, the second parameter should not be empty.
Commits
-------
ce03562 [DependencyInjection] Fix phpdoc for $configurator in Autoconfigure attributeFile tree
1 file changed
+1
-1
lines changed- src/Symfony/Component/DependencyInjection/Attribute
1 file changed
+1
-1
lines changedLines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
28 | 28 | | |
29 | 29 | | |
30 | 30 | | |
31 | | - | |
| 31 | + | |
32 | 32 | | |
33 | 33 | | |
34 | 34 | | |
| |||
0 commit comments