This repository has been archived on 2023-08-20. You can view files and clone it, but cannot push or open issues or pull requests.

Fabien Potencier 107e1f1f6d merged branch kufi/master (PR #5032) ... This PR was squashed before being merged into the master branch (closes #5032). Commits ------- afba15f [2.2] Translatable field type for Propel i18n columns Discussion ---------- [2.2] Translatable field type for Propel i18n columns A field type which allows to automatically generate the correct fields for propels i18n behavior. Usage example: $builder->add('pageI18ns', 'translatable_collection', array( 'i18n_class' => '\foo\barBundle\Model\PageI18n', 'languages' => array('de', 'fr'), 'label' => 'Translations', 'columns' => array( 'title' => array( 'label' => 'Custom title', ), 'description' => array( 'type' => 'textarea' ) ) )); With this configuration the field automatically generates the correct fields for the title and description column for the given languages. --------------------------------------------------------------------------- by stof at 2012-07-24T14:37:27Z tests are also missing --------------------------------------------------------------------------- by kufi at 2012-07-27T08:50:05Z Ok. Moved the Listeners into own classes. Changed the names of the types. Fixed the TranslationCollectionType which now is a Subclass of AbstractType and has the parent collection. Edit: The syntax changed slighty for the form: $builder->add('pageI18ns', new \Symfony\Bridge\Propel1\Form\Type\TranslationCollectionType(), array( 'languages' => array('de', 'fr', 'en'), 'label' => 'Translations', 'options' => array( 'data_class' => 'foo\bar\Modell\PageI18n', 'columns' => array( 'title' => array( 'label' => 'Custom title', ), 'description' => array( 'type' => 'textarea' ) ) ) )); --------------------------------------------------------------------------- by stof at 2012-07-27T08:55:07Z tests are still missing, and you have some CS issue (which can probably all be fixed by running the [PHP-CS-Fixer](http://cs.sensiolabs.org/) on your classes) --------------------------------------------------------------------------- by sindro88 at 2012-08-13T13:27:46Z I followed step by step the implementation but the form type return an error "Could not load type propel1_translation". --------------------------------------------------------------------------- by kufi at 2012-08-14T06:21:40Z Could you try again. The problem was that the type propel1_translation_collection relied on a registered form type propel1_translation. I removed this one and replaced it with the actual form class. --------------------------------------------------------------------------- by sindro88 at 2012-08-14T06:35:33Z I replaced with the class and now it work, thank you so much! --------------------------------------------------------------------------- by fabpot at 2012-09-18T16:53:21Z ping @willdurand --------------------------------------------------------------------------- by stof at 2012-10-13T17:56:16Z @willdurand ping --------------------------------------------------------------------------- by willdurand at 2012-10-23T12:03:22Z There are a few comments by @stloyd to fix, but I'm 👍 on this PR. --------------------------------------------------------------------------- by fabpot at 2012-10-23T13:18:59Z @kufi Can you add a note in the CHANGELOG of the Propel bridge before I merge this PR? Thanks. --------------------------------------------------------------------------- by kufi at 2012-10-23T13:32:31Z @fabpot Sure. Does this belong to Version 2.1.0 or the upcoming 2.2.0? --------------------------------------------------------------------------- by fabpot at 2012-10-23T13:59:04Z 2.2		2012-10-27 21:40:56 +02:00
..
ChoiceList	[Form] Fixed failing tests	2012-07-25 22:16:38 +02:00
DataTransformer	[Propel1] Allowed PropelObjectCollection only in CollectionToArrayTransformer	2012-04-20 09:32:01 +02:00
Type	[2.2] Translatable field type for Propel i18n columns	2012-10-27 21:40:54 +02:00
PropelTypeGuesserTest.php	fixed CS	2012-07-09 14:54:20 +02:00