Repository method with arguments

Example:

I have set this:

    /**
     * @ParamConverter(
     *    "post",
     *    class="AcmeBlogBundle:Post",
     *    options={
     *        "repository_method" = "findForYmdLink",
     *        "mapping": {"year": "year", "month": "month", "day": "day", "permalink": "permalink"}
     *    }
     * )
     */

My repository method looks like this:

    public function findForYmdLink($year, $month, $day, $permalink)
    {
        $creationDate = \DateTime::createFromFormat('Ymd', $year.$month.$day);

        return $this->findOneBy(array(
            'creationDate' => $creationDate,
            'permalink' => $permalink,
            'published' => true,
        ));
    }

My Post entity doesn't have the fields day, month and year but has creationDate instead (DateTime). The current implementation of the converter doesn't pass the 3 fields because they are not in the metadata. I don't see why they should be if I pass a custom repository_method. This patch changes that behaviour.

With the current implementation, I would also have to change the repository method to this:

    public function findForYmdLink($params)
    {
        list($year, $month, $day, $permalink) = $params;
        ...

This patch changes that behaviour by calling invokeArgs on the reflectionMethod so the exact parameters are in the method signature.
Correct, and I wasn't going to change this code but fabbot complains about the "return null;", instead of changing this to "return;" I liked to keep the null as it's more explicit. Also the 2 return statements are now 1. I heard having less exit points is better anyway.