8000 Ignore invalid reference arguments in collections · Issue #6550 · symfony/symfony-docs · GitHub
[go: up one dir, main page]

Skip to content

Ignore invalid reference arguments in collections #6550

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
xabbuh opened this issue May 9, 2016 · 2 comments
Closed

Ignore invalid reference arguments in collections #6550

xabbuh opened this issue May 9, 2016 · 2 comments
Labels
DependencyInjection hasPR A Pull Request has already been submitted for this issue.
Milestone

Comments

@xabbuh
Copy link
Member
xabbuh commented May 9, 2016

#6511 adds a description for the DependencyInjection component's feature to ignore missing arguments in method calls. symfony/symfony#17132 expanded this to also drop invalid references when being passed as part of a collection argument.

@wouterj
Copy link
Member
wouterj commented May 14, 2016

Hmm, I'm wondering if it makes sense to document this as it seems very detailed and not really usefull to 99% of the readers.

@xabbuh
Copy link
Member Author
xabbuh commented May 14, 2016

I think we can simply add a short sentence to the recently added documentation.

@javiereguiluz javiereguiluz added hasPR A Pull Request has already been submitted for this issue. Missing Documentation labels Apr 24, 2017
xabbuh added a commit that referenced this issue May 12, 2017
… in collection arguments (javiereguiluz)

This PR was submitted for the 3.1 branch but it was merged into the 3.2 branch instead (closes #7834).

Discussion
----------

Added a note about ignoring invalid reference arguments in collection arguments

This fixes #6550.

Commits
-------

0b2d9ba Added a note about ignoring invalid reference arguments in colleciton arguments
@xabbuh xabbuh closed this as completed May 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DependencyInjection hasPR A Pull Request has already been submitted for this issue.
Projects
None yet
Development

No branches or pull requests

3 participants
0