improved the performance of the Twig loader by not registering a chain loader if not needed
merged branch fabpot/twig-perf (PR #635)

This PR was merged into the master branch.

Commits
-------

7e554db improved the performance of the Twig loader by not registering a chain loader if not needed

Discussion
----------

improved the performance of the Twig loader by not registering a chain loader if not needed

When `twig.templates` is empty, there is no need to register the Twig array loader as it will never be used anyway.

---------------------------------------------------------------------------

by lyrixx at 2013-02-18T09:50:07Z

totally :+1: More over, I think this feature was not documented.

---------------------------------------------------------------------------

by GromNaN at 2013-02-18T10:04:32Z

Nice tip. How much is it more performant ?

@lyrixx it is documented here : http://silex.sensiolabs.org/doc/providers/twig.html#parameters

---------------------------------------------------------------------------

by fabpot at 2013-02-18T10:06:17Z

It's not really about how much but just keeping the array loader around is a waste of time.

---------------------------------------------------------------------------

by igorw at 2013-02-18T13:37:58Z

:-1: This prevents extending the `twig.loader` service and also feels like a micro-optimization to me.

---------------------------------------------------------------------------

by fabpot at 2013-02-18T13:57:13Z

I've removed the optimization but I've kept the different order in registration as it makes more sense.

---------------------------------------------------------------------------

by igorw at 2013-02-18T13:59:55Z

:+1:
Updated docs to include _method and enableHttpMethodParameterOverride()
merged branch robdudley/master (PR #636)

This PR was squashed before being merged into the master branch (closes #636).

Commits
-------

abdc1fc Updated docs to include _method and enableHttpMethodParameterOverride()

Discussion
----------

Updated docs to include _method and enableHttpMethodParameterOverride()

Should be self explanatory. Small addition to the docs to explain about the _method form value and also to advise abut the explicit enableHttpMethodParameterOverride() call required with Symfony 2.2.0 and later.

---------------------------------------------------------------------------

by igorw at 2013-02-18T13:57:37Z

Please wrap the docs at 80 chars.

:+1:

---------------------------------------------------------------------------

by davedevelopment at 2013-02-18T14:02:08Z

:+1:

Only one thing, do we need to explicitly state to do this before the request is handled? No-brainer to us, but what about others?

Or should I say, how likely are people to try and do something like:

``` php

$app->put("/some/resource", function(Request $request) {
    Request::enableHttpMethodParameterOverride();
});

```

---------------------------------------------------------------------------

by robdudley at 2013-02-18T14:11:15Z

I've used a similar example to the reverse proxy tip which I hope would make the context clear. Not sure how else to word it without becoming unnecessarily verbose. Also I'm assuming that the setting is going to be rolled into core at some point as it has pretty big backwards compatibility implications.

---------------------------------------------------------------------------

by davedevelopment at 2013-02-18T14:14:00Z

:shipit:
tweaked previous merge