congow/Orient — pull request #128 by nrk (2012-04-29 19:57:03+00:00)
Title: Radically change our approach for bindings.

I reorganized a couple of namespaces and cleaned the code in a few test units while I was at this. There are still a couple of rough edges but overall the code looks decent, it's easier to test and we are not tied anymore to our own HTTP client class. The test suite is obviously all green.

Things I'd change with further commits.

1. Instead of having a list of 5 arguments, use a named array to pass database parameters to `Congow\Orient\Binding\HttpBinding::__construct()`.
2. Related to the first point, it's probably better to have a second optional argument to specify a `Congow\Orient\Contract\Binding\Adapter\HttpClientAdapterInterface` when initializing the HTTP binding.
3. `Congow\Orient\Binding\Adapter\CurlClientAdapterResult` can be definitely improved and made more robust.
4. Can we just drop the `Congow\Orient\Contract` namespace and move interfaces in their respective namespaces? I find the current layout for interfaces highly confusing, having them in a separate namespace is of dubious usefulness. Well maybe not for beta-5, but still...

I know it's a whole lot of changes for the binding stuff but I'd like to have those changes merged with a few adjustments before pushing a beta-5 tag.