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

Calling functions and resolving their parameters using the container #162

Closed
wants to merge 17 commits into
base: master
from

Conversation

Projects
None yet
3 participants
@mnapoli
Member

mnapoli commented May 19, 2014

Coverage Status
Scrutinizer Code Quality

$container->call(function (Logger $logger, EntityManager $em) {
    // do stuff
});
interface InvokerInterface
{
    public function call($callable, array $parameters = array());
}

Using defined parameters:

$parameters = [
    'data' => /* some variable */
];

$container->call(function (Logger $logger, $data) {
    // ...
}, $parameters);

mnapoli added some commits May 19, 2014

@mnapoli mnapoli added this to the 4.2 milestone Jun 2, 2014

@mnapoli mnapoli referenced this pull request Jun 3, 2014

Merged

4.2 #165

11 of 11 tasks complete

daniphp and others added some commits Jun 3, 2014

Update definition.md
add a demo for wildcards syntax
Merge pull request #166 from daniphp/patch-1
Update definition.md
Merge pull request #167 from avant1/4.2
 #164 Prototype scope on "factory" definitions

@mnapoli mnapoli added the enhancement label Jun 4, 2014

@mnapoli mnapoli changed the title from WIP Calling functions and resolving their parameters using the container to Calling functions and resolving their parameters using the container Jun 5, 2014

@mnapoli

This comment has been minimized.

Show comment
Hide comment
@mnapoli

mnapoli Jun 5, 2014

Member

Merged in the 4.2 branch.

Member

mnapoli commented Jun 5, 2014

Merged in the 4.2 branch.

@mnapoli mnapoli closed this Jun 5, 2014

@mnapoli mnapoli deleted the feature/CallFunction branch Jun 5, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment