Skip to content

Commit

Permalink
Added config for user_to_username_transformer
Browse files Browse the repository at this point in the history
  • Loading branch information
Nyholm committed Oct 8, 2013
1 parent 0891050 commit c26f00c
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 5 deletions.
8 changes: 3 additions & 5 deletions DataTransformer/RecipientsDataTransformer.php
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@
use Doctrine\Common\Persistence\ObjectManager;
use Doctrine\Common\Collections\Collection;
use Doctrine\Common\Collections\ArrayCollection;
use FOS\UserBundle\Form\DataTransformer\UserToUsernameTransformer;
/**
* Transforms collection of UserInterface into strings separated with coma
*
Expand All @@ -17,14 +16,14 @@
class RecipientsDataTransformer implements DataTransformerInterface
{
/**
* @var UserToUsernameTransformer
* @var DataTransformerInterface
*/
private $userToUsernameTransformer;

/**
* @param UserToUsernameTransformer $userToUsernameTransformer
* @param DataTransformerInterface $userToUsernameTransformer
*/
public function __construct(UserToUsernameTransformer $userToUsernameTransformer)
public function __construct(DataTransformerInterface $userToUsernameTransformer)
{
$this->userToUsernameTransformer = $userToUsernameTransformer;
}
Expand Down Expand Up @@ -71,7 +70,6 @@ public function reverseTransform($usernames)
}

$recipients = new ArrayCollection();
$transformer = $this->userToUsernameTransformer;
$recipientsNames = array_filter(explode(',', $usernames));

foreach ($recipientsNames as $username) {
Expand Down
1 change: 1 addition & 0 deletions DependencyInjection/Configuration.php
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@ public function getConfigTreeBuilder()
->scalarNode('deleter')->defaultValue('fos_message.deleter.default')->cannotBeEmpty()->end()
->scalarNode('spam_detector')->defaultValue('fos_message.noop_spam_detector')->cannotBeEmpty()->end()
->scalarNode('twig_extension')->defaultValue('fos_message.twig_extension.default')->cannotBeEmpty()->end()
->scalarNode('user_transformer')->defaultValue('fos_user.user_to_username_transformer')->cannotBeEmpty()->end()
->arrayNode('search')
->addDefaultsIfNotSet()
->children()
Expand Down
4 changes: 4 additions & 0 deletions DependencyInjection/FOSMessageExtension.php
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
namespace FOS\MessageBundle\DependencyInjection;

use Symfony\Component\Config\Definition\Processor;
use Symfony\Component\DependencyInjection\Reference;
use Symfony\Component\HttpKernel\DependencyInjection\Extension;
use Symfony\Component\DependencyInjection\Loader\XmlFileLoader;
use Symfony\Component\DependencyInjection\ContainerBuilder;
Expand Down Expand Up @@ -61,5 +62,8 @@ public function load(array $configs, ContainerBuilder $container)
$container->setAlias('fos_message.search_finder', $config['search']['finder']);
$container->getDefinition('fos_message.search_query_factory.default')
->replaceArgument(1, $config['search']['query_parameter']);

$container->getDefinition('fos_message.recipients_data_transformer')
->replaceArgument(0, new Reference($config['user_transformer']));
}
}
1 change: 1 addition & 0 deletions Resources/doc/index.rst
Original file line number Diff line number Diff line change
Expand Up @@ -349,6 +349,7 @@ All configuration options are listed below::
deleter: fos_message.deleter # See Deleter\DeleterInterface
spam_detector: fos_message.noop_spam_detector # See SpamDetection\SpamDetectorInterface
twig_extension: fos_message.twig_extension # See Twig\Extension\MessageExtension
user_transformer: fos_user.user_transformer # See Symfony\Component\Form\DataTransformerInterface
search:
finder: fos_message.search_finder # See Finder\FinderInterface
query_factory: fos_message.search_query_factory # See Finder\QueryFactoryInterface
Expand Down

0 comments on commit c26f00c

Please sign in to comment.