this automaticaly add class for ParamConverter without class, when a
typehint is provided
merged branch jaugustin/auto-add-class-for-custom-paramconverter (PR #119)

Commits
-------

64aa438 this automaticaly add class for ParamConverter without class, when a typehint is provided

Discussion
----------

this add class for ParamConverter without class

Hi,

This add the possibility to configure the paramConverter without class param, when the parameter has a typehint:

```php
<?php
use Acme\BookBundle\Model\Book;
    /**
     *  @Route("/edit-book/{id}", name="edit_book")
     *  @ParamConverter("book", options={"option1"="foo"})
     *  @Template()
     */
    public function editBookAction(Book $book)
    {
    }
```

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

by jaugustin at 2012-05-22T22:20:11Z

any new on this ?

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

by willdurand at 2012-05-24T09:07:13Z

ping @fabpot

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

by fabpot at 2012-05-25T07:43:13Z

Have you tested your fix? Because I don't see how it can work.

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

by jaugustin at 2012-05-25T08:11:19Z

Yes I tested it, and it work.
if your param converter has no `class` parameter and there is a typehint on the action parameter, class param will be automaticaly set, the same way as when there is no paramconverter at all.

like in the example :

```php
<?php
use Acme\BookBundle\Model\Book;
    /**
     *  @Route("/edit-book/{id}", name="edit_book")
     *  @ParamConverter("book", options={"option1"="foo"})
     *  @Template()
     */
    public function editBookAction(Book $book)
    {
    }
```

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

by beberlei at 2012-07-24T16:51:50Z

+1 on this one
merged branch beberlei/DateTimeConverter (PR #136)

Commits
-------

64a0a69 Fix apply and CS
94fd1bb Allow options to pass format, documented datetime converter
cc0350d Test for DateTimeParamConverter
90605cb DateTimeParamConverter

Discussion
----------

[ParamConverter] DateTime converter

Allow converting route or request parameters into DateTime instances in the method signature of an action.

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

by lsmith77 at 2012-07-21T07:14:15Z

would it be possible to also support ``Y/m/d`` with this converter?

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

by stof at 2012-07-21T11:00:11Z

@lsmith77 I don't why it could not support it

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

by beberlei at 2012-07-22T14:44:35Z

@lsmith77 well not in route attributes, but in get/post it obviously works-
Backwards compatible refactoring that increases the use-cases for DoctrineParamConverter. Support for composite primary keys and for cases where multiple entities use have the same field names
merged branch beberlei/DoctrineParamConverterRefactoring (PR #141)

Commits
-------

ae620b8 Backwards compatible refactoring that increases the use-cases for DoctrineParamConverter. Support for composite primary keys and for cases where multiple entities use have the same field names

Discussion
----------

[ParamConverter] Doctrine Converter Refactoring

Backwards compatible refactoring that increases the use-cases for DoctrineParamConverter.

* Support for composite primary keys
* Allow explicit mapping and excludes in cases where multiple converters clash with the same field names.
* Fix bug where an exception is thrown even when the parameter is optional.
* Increase the code-coverage of the converter, covering find by primary key and by fields values.

This brings DoctrineConverter in line with Propel converter (https://github.com/propelorm/PropelBundle/blob/1.1/Request/ParamConverter/PropelParamConverter.php) which also has 'mapping' and 'exclude' keys.
fixed CS
[ParamConverter][Doctrine] allow to find by single-valued associations.
Merge sensio/master
Add UPGRADE.md notice of possible BC break
Merge branch 'DoctrineFindByAssociations'