services:manager:class:UserManagerarguments:          -truecalls:          -method:setLoggerarguments:- @logger          -method:setClassarguments:              -Usertags:          -name:manageralias:user

Copy link

Member

nicolas-grekas commentedMar 18, 2015

👍

Copy link

Member

stof commentedMar 18, 2015

This implementatio currently supports mixing numeric keys and name keys for the method and the arguments, which can be confusing

[DependencyInjection] Added support for keys "method" and "arguments"…

0eb34f3

… in "calls" statement for yaml format

hason force-pushed thedi_yaml branch from7549dd8 to0eb34f3Compare

March 18, 2015 11:12

Copy link

ContributorAuthor

hason commentedMar 18, 2015

@stof Fixed

Copy link

Member

fabpot commentedMar 19, 2015

Looks good to me 👍

aitboudad added the Yaml label

Mar 20, 2015

Copy link

Member

fabpot commentedMar 23, 2015

@hason Can you make a PR on the docs to document this new possibility? Thanks.

Copy link

Member

fabpot commentedMar 23, 2015

Thank you@hason.

fabpot merged commit0eb34f3 intosymfony:2.7

Mar 23, 2015

fabpot added a commit that referenced this pull request

Mar 23, 2015

feature#13892[DependencyInjection] Improved yaml syntax (hason)

89cbafd

This PR was merged into the 2.7 branch.Discussion----------[DependencyInjection] Improved yaml syntax| Q             | A| ------------- | ---| Bug fix?      | no| New feature?  | yes| BC breaks?    | no| Deprecations? | no| Tests pass?   | yes| Fixed tickets || License       | MIT| Doc PR        |This PR adds support for this:```yamlservices:    manager:        class: UserManager        arguments:          - true        calls:          - method: setLogger            arguments:              -@logger          - method: setClass            arguments:              - User        tags:          - name: manager            alias: user```Commits-------0eb34f3 [DependencyInjection] Added support for keys "method" and "arguments" in "calls" statement for yaml format