Skip to content

Commit ec82d88

Browse files
committed
Try catch flush
1 parent 6e60770 commit ec82d88

File tree

2 files changed

+20
-14
lines changed

2 files changed

+20
-14
lines changed

EventListener/LoginListener.php

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
use DateTime;
77
use Doctrine\ORM\EntityManagerInterface;
88
use Symfony\Component\Security\Http\Event\InteractiveLoginEvent;
9+
use Exception;
910

1011
class LoginListener
1112
{
@@ -24,8 +25,10 @@ public function onSecurityInteractiveLogin(InteractiveLoginEvent $event)
2425
/** @var User $user */
2526
$user = $event->getAuthenticationToken()->getUser();
2627

27-
$user->setLastConnexion(new DateTime());
28-
$this->em->persist($user);
29-
$this->em->flush();
28+
try {
29+
$user->setLastConnexion(new DateTime());
30+
$this->em->persist($user);
31+
$this->em->flush();
32+
} catch (Exception $e) {}
3033
}
3134
}

EventSubscriber/UserStatsSubscriber.php

Lines changed: 14 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
1111
use Symfony\Component\HttpKernel\Event\RequestEvent;
1212
use Symfony\Component\Security\Core\Security;
13+
use Exception;
1314

1415
class UserStatsSubscriber implements EventSubscriberInterface
1516
{
@@ -52,20 +53,22 @@ public function onKernelRequest(RequestEvent $event)
5253
return;
5354
}
5455

55-
$user->setLastVisited(new DateTime());
56-
$user->setNbPageViews($user->getNbPageViews() + 1);
56+
try {
57+
$user->setLastVisited(new DateTime());
58+
$user->setNbPageViews($user->getNbPageViews() + 1);
5759

58-
$userStatsLines = new UserStatsLines();
59-
$userStatsLines->setUser($user);
60-
$userStatsLines->setUrl($event->getRequest()->getRequestUri());
61-
$userStatsLines->setRoute($event->getRequest()->get('_route'));
62-
$userStatsLines->setSessionId(session_id());
63-
$userStatsLines->setBrowser($event->getRequest()->server->get('HTTP_USER_AGENT'));
60+
$userStatsLines = new UserStatsLines();
61+
$userStatsLines->setUser($user);
62+
$userStatsLines->setUrl($event->getRequest()->getRequestUri());
63+
$userStatsLines->setRoute($event->getRequest()->get('_route'));
64+
$userStatsLines->setSessionId(session_id());
65+
$userStatsLines->setBrowser($event->getRequest()->server->get('HTTP_USER_AGENT'));
6466

65-
$this->em->persist($user);
66-
$this->em->persist($userStatsLines);
67+
$this->em->persist($user);
68+
$this->em->persist($userStatsLines);
6769

68-
$this->em->flush();
70+
$this->em->flush();
71+
} catch (Exception $e) {}
6972
}
7073
}
7174
}

0 commit comments

Comments
 (0)