mirrors/wallabag
1
0
Fork 0
mirror of https://github.com/wallabag/wallabag.git synced 2024-12-22 23:56:29 +00:00
Code Issues Projects Releases Wiki Activity

Create config even if user is disabled

Browse source 
When a user register itself AND the wallabag instance is configured to send a confirmation email, the user is disabled when the listener (which create the config) receive the event.
There were a check (don't know why) if the user is enabled we create the config. But the user is disabled when confirmation email is actived.
This commit is contained in:
Jeremy Benoist 2016-10-04 10:42:45 +02:00
parent f0b2d57154
commit 755ff9e835
No known key found for this signature in database
GPG key ID: BCA73962457ACC3C
2 changed files with 0 additions and 24 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/Wallabag/UserBundle/EventListener/CreateConfigListener.php
View file

@ -45,10 +45,6 @@ class CreateConfigListener implements EventSubscriberInterface
public function createConfig(UserEvent $event, $eventName = null, EventDispatcherInterface $eventDispatcher = null)
{
if (!$event->getUser()->isEnabled()) {
return;
}
$config = new Config($event->getUser());
$config->setTheme($this->theme);
$config->setItemsPerPage($this->itemsOnPage);

20
tests/Wallabag/UserBundle/EventListener/CreateConfigListenerTest.php
View file

@ -41,26 +41,6 @@ class CreateConfigListenerTest extends \PHPUnit_Framework_TestCase
$this->response = Response::create();
}
public function testWithInvalidUser()
{
$user = new User();
$user->setEnabled(false);
$event = new FilterUserResponseEvent(
$user,
$this->request,
$this->response
);
$this->em->expects($this->never())->method('persist');
$this->em->expects($this->never())->method('flush');
$this->dispatcher->dispatch(
FOSUserEvents::REGISTRATION_COMPLETED,
$event
);
}
public function testWithValidUser()
{
$user = new User();