Documented the repository_method option

Closes issue #198
merged branch Gregoire-M/patch-1 (PR #199)

This PR was merged into the master branch.

Commits
-------

fa31299 Documented the repository_method option

Discussion
----------

Documented the repository_method option

Closes issue #198
[Route] Ensure @Route.service is not defined on methods (fix #192)
merged branch vicb/route/service (PR #200)

This PR was merged into the master branch.

Commits
-------

3e6f536 [Route] Ensure @Route.service is not defined on methods (fix #192)

Discussion
----------

[Route] Ensure @Route.service is not defined on methods (fix #192)

| Q             | A
| ------------- | ---
| Bug fix?      | yes/no, throw an exception rather than silently ignore the service option
| New feature?  | yes, throw an exception rather than silently ignore the service option
| BC breaks?    | kind of if the configuration was wrong
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #192
| License       | MIT
| Doc PR        | -
DoctrineParamConverter should allow 0 as valid identifier
merged branch borisguery/id-0-should-not-throw-an-exception (PR #183)

This PR was submitted for the master branch but it was merged into the 2.1 branch instead (closes #183).

Commits
-------

9c6b671 DoctrineParamConverter should allow 0 as valid identifier

Discussion
----------

DoctrineParamConverter should allow 0 as valid identifier

Use case:

* I've a route like `/items/{item}` where item is an unique number.

* If I try to fetch the item identified by a `0` it throws

 `\LogicException('Unable to guess how to get a Doctrine instance from the request information.');`

This is because in the `find()` method it "validates" the identifier as being not false but without strict type check resulting in `0` being evaluated to `false`, which is, strictly speaking, wrong.

[DoctrineParamConverter.php::find()](https://github.com/sensio/SensioFrameworkExtraBundle/blob/946f74bd4aa78b311c08b14f8a934ec30f20defd/Request/ParamConverter/DoctrineParamConverter.php#L71)

I wrote a test case which fails when using a 0 as an identifier and a fix using a strict type check to get ride of this.
Merge branch '2.1'

* 2.1:
  DoctrineParamConverter should allow 0 as valid identifier
made doctrine conversion optional depending on the parameter default value
apply method relies on some methods not present in the interface contract
merged branch bamarni/default-null (PR #201)

This PR was merged into the master branch.

Commits
-------

f5d9270 apply method relies on some methods not present in the interface contract
4bfdad6 made doctrine conversion optional depending on the parameter default value

Discussion
----------

made doctrine conversion optional

cc @fabpot @borisguery

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

by bamarni at 2013-02-21T11:25:22Z

ouch I can see it has been merged in 2.1, I'll rebase it and retarget as soon as i can.

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

by bamarni at 2013-02-21T20:34:11Z

cf #203
Update composer.json

Bump dependency of doctrine common for symfony 2.1 users.
merged branch guilhermeblanco/patch-3 (PR #202)

This PR was submitted for the master branch but it was merged into the 2.1 branch instead (closes #202).

Commits
-------

3fa2c5c Update composer.json

Discussion
----------

Update composer.json

Bump dependency of doctrine common for symfony 2.1 users.
[ParamConverter] Improving documentation
merged branch gnugat/doc-converters (PR #205)

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

Commits
-------

96944d1 [ParamConverter] Improving documentation

Discussion
----------

[ParamConverter] Improving documentation

This PR provides a `yml` configuration example for the service registration in the documentation, and some PHPdoc to the interface, the DoctrineParamConverter and the DataTimeParamConverter.
Added classes to cache
merged branch dlsniper/classes-cache (PR #197)

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

Commits
-------

23daf05 Added classes to cache

Discussion
----------

Added classes to cache

Hello,

This commit adds some used classes to Symfony2 cache.
 See the talk in sensio/SensioDistributionBundle#76

Let me know if it needs anything else/if I've did it against the wrong branch (it should go into 2.2 distro).

Thanks.

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

by dlsniper at 2013-02-28T07:43:38Z

When will this be merged? ping :)

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

by fabpot at 2013-02-28T07:56:06Z

We should add the classes only if we are sure that they will be used. So, they should be added only when the related configuration setting is enabled.

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

by dlsniper at 2013-02-28T12:43:05Z

I think this should be a little bit better now.
partially reverted a previous merge (closes #206)
Added ability to ignore null request attributes
updated deps
changed listeners to become subscriber, move the priority of some listeners (closes #57)