We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ArrayExtension
The ArrayExtension extension class' behavior changed in 0f9ad94, when the resolution of dependencies was extracted into a trait.
Previously, the resolution involved a simple array_map (source here).
array_map
After 0f9ad94, ArrayExtension used the newly extracted resolveKeys() trait method, which specifically only returns array values (source here)
resolveKeys()
Therefore, in order to preserve the previous functionality, ArrayExtension should therefore use resolveDeps() to preserve the dependency array keys.
resolveDeps()
The text was updated successfully, but these errors were encountered:
Correction: resolveDeps() does not preserve the keys in the $keys argument. So a new method must be introduced in the ResolveKeysCapableTrait ... sigh
$keys
ResolveKeysCapableTrait
Is this really better than just array_map()?
array_map()
Sorry, something went wrong.
ArrayExtension preserves keys
b257372
Fixes #6
mecha
Successfully merging a pull request may close this issue.
The
ArrayExtension
extension class' behavior changed in 0f9ad94, when the resolution of dependencies was extracted into a trait.Previously, the resolution involved a simple
array_map
(source here).After 0f9ad94,
ArrayExtension
used the newly extractedresolveKeys()
trait method, which specifically only returns array values (source here)Therefore, in order to preserve the previous functionality,
ArrayExtension
should therefore useresolveDeps()
to preserve the dependency array keys.The text was updated successfully, but these errors were encountered: