diff options
author | Jeremy Benoist <jeremy.benoist@gmail.com> | 2015-10-01 22:25:23 +0200 |
---|---|---|
committer | Jeremy Benoist <jeremy.benoist@gmail.com> | 2015-10-03 13:30:43 +0200 |
commit | 772d8c4b93adc36baefda93ec37007e4a85321de (patch) | |
tree | c654a9c7b00f2d7d3a5dd20a49120c67e8b8d0a9 /src/Wallabag/CoreBundle/EventListener | |
parent | 2c13918acc3c46120bbef5e6746f3c6dc27be5df (diff) | |
download | wallabag-772d8c4b93adc36baefda93ec37007e4a85321de.tar.gz wallabag-772d8c4b93adc36baefda93ec37007e4a85321de.tar.zst wallabag-772d8c4b93adc36baefda93ec37007e4a85321de.zip |
Add test on RegistrationConfirmedListener
And PLEASE @nicosomb, NEVER EVER inject the whole container inside a service.
Diffstat (limited to 'src/Wallabag/CoreBundle/EventListener')
-rw-r--r-- | src/Wallabag/CoreBundle/EventListener/RegistrationConfirmedListener.php | 24 |
1 files changed, 15 insertions, 9 deletions
diff --git a/src/Wallabag/CoreBundle/EventListener/RegistrationConfirmedListener.php b/src/Wallabag/CoreBundle/EventListener/RegistrationConfirmedListener.php index bcc84923..68c25f1f 100644 --- a/src/Wallabag/CoreBundle/EventListener/RegistrationConfirmedListener.php +++ b/src/Wallabag/CoreBundle/EventListener/RegistrationConfirmedListener.php | |||
@@ -2,22 +2,28 @@ | |||
2 | 2 | ||
3 | namespace Wallabag\CoreBundle\EventListener; | 3 | namespace Wallabag\CoreBundle\EventListener; |
4 | 4 | ||
5 | use FOS\UserBundle\FOSUserEvents; | ||
6 | use Symfony\Component\DependencyInjection\Container; | ||
7 | use Symfony\Component\EventDispatcher\EventDispatcherInterface; | 5 | use Symfony\Component\EventDispatcher\EventDispatcherInterface; |
8 | use Symfony\Component\EventDispatcher\EventSubscriberInterface; | 6 | use Symfony\Component\EventDispatcher\EventSubscriberInterface; |
7 | use Doctrine\ORM\EntityManager; | ||
9 | use FOS\UserBundle\Event\FilterUserResponseEvent; | 8 | use FOS\UserBundle\Event\FilterUserResponseEvent; |
9 | use FOS\UserBundle\FOSUserEvents; | ||
10 | use Wallabag\CoreBundle\Entity\Config; | 10 | use Wallabag\CoreBundle\Entity\Config; |
11 | 11 | ||
12 | class RegistrationConfirmedListener implements EventSubscriberInterface | 12 | class RegistrationConfirmedListener implements EventSubscriberInterface |
13 | { | 13 | { |
14 | private $em; | 14 | private $em; |
15 | private $container; | 15 | private $theme; |
16 | private $itemsOnPage; | ||
17 | private $rssLimit; | ||
18 | private $language; | ||
16 | 19 | ||
17 | public function __construct(Container $container, $em) | 20 | public function __construct(EntityManager $em, $theme, $itemsOnPage, $rssLimit, $language) |
18 | { | 21 | { |
19 | $this->container = $container; | ||
20 | $this->em = $em; | 22 | $this->em = $em; |
23 | $this->theme = $theme; | ||
24 | $this->itemsOnPage = $itemsOnPage; | ||
25 | $this->rssLimit = $rssLimit; | ||
26 | $this->language = $language; | ||
21 | } | 27 | } |
22 | 28 | ||
23 | public static function getSubscribedEvents() | 29 | public static function getSubscribedEvents() |
@@ -34,10 +40,10 @@ class RegistrationConfirmedListener implements EventSubscriberInterface | |||
34 | } | 40 | } |
35 | 41 | ||
36 | $config = new Config($event->getUser()); | 42 | $config = new Config($event->getUser()); |
37 | $config->setTheme($this->container->getParameter('theme')); | 43 | $config->setTheme($this->theme); |
38 | $config->setItemsPerPage($this->container->getParameter('items_on_page')); | 44 | $config->setItemsPerPage($this->itemsOnPage); |
39 | $config->setRssLimit($this->container->getParameter('rss_limit')); | 45 | $config->setRssLimit($this->rssLimit); |
40 | $config->setLanguage($this->container->getParameter('language')); | 46 | $config->setLanguage($this->language); |
41 | $this->em->persist($config); | 47 | $this->em->persist($config); |
42 | $this->em->flush(); | 48 | $this->em->flush(); |
43 | } | 49 | } |