darya / framework

PHP Web Application Framework
http://darya.io/
MIT License
2 stars 1 forks source link

Smarter parameter argument resolution for the service container #32

Open hexus opened 9 years ago

hexus commented 9 years ago

Right now, Darya\Service\Container::mergeResolvedParameters() is begging for a smart implementation that merges user-provided arguments with those resolved from the container.

This will likely take into account numeric indexes of user-provided arguments and possibly the types of the resolved arguments.

hexus commented 6 years ago

There is also a bug whereby the order of associative arguments given to newInstanceArgs() or invokeWithArgs() matters (because associate arrays do have an order), and of course the keys must match the parameter names (which they do already).