BGC-189

src/Phoenix/Controller/LearningSpace/LoginController.php

@@ -15,6 +15,7 @@ use Logipro\Phoenix\Form\CatalogType\CatalogDefinePasswordType;
 use Logipro\Phoenix\Entity\Account\Constraints\Password\PasswordValidator;
 use Symfony\Component\Security\Http\Event\InteractiveLoginEvent;
 use Logipro\Phoenix\Service\LoginService;
+use Symfony\Component\HttpFoundation\Response;
 
 class LoginController extends LearningSpaceController
 {
@@ -66,7 +67,21 @@ class LoginController extends LearningSpaceController
 				$manager = $this->getDoctrine()->getManager();
 				$repository = $manager->getRepository(LearnerAccount::class);
 
-				$learnerAccount = $repository->loadUserByUsername($email);
+				try {
+					$learnerAccount = $repository->loadUserByUsername($email);
+				} catch (\Exception $exp) {
+
+					$response = new Response('', 409);
+
+					// token expiré
+					$parameters = array(
+						'text' => $exp->getMessage(),
+						'path' => 'ls_password_recover',
+						'go' => $this->translate('catalog_form_catalog_password_title')
+					);
+					return $this->phxRender('bundles/TwigBundle/Exception/definePassword.html.twig', $parameters, $response);
+				}
+
 				if ($learnerAccount) {
 					//On déclenche l'event si l'adresse email est précisée
 					if ($learnerAccount->getEmail() != "") {

src/Phoenix/Repository/LearningSpace/LearnerAccountRepository.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace Logipro\Phoenix\Repository\LearningSpace;
 
 use Doctrine\Bundle\DoctrineBundle\Repository\ServiceEntityRepository;
@@ -25,43 +26,39 @@ class LearnerAccountRepository extends ServiceEntityRepository implements UserLo
 	}
 
 	public function loadUserByUsername($username)
-    {
+	{
 		// recherche l'utilisateur par identifiant
 		$query = $this->createQueryBuilder('u')
-		->where('u.username = :username')
-		->setParameter('username', $username)
-		->getQuery();
+			->where('u.username = :username')
+			->setParameter('username', $username)
+			->getQuery();
 
 		// si 1 seul utilisateur trouvé => on le retourne
 		// si aucun ou plusieur, on regarde l'email
 		$users = $query->execute();
-		if ($users)
-		{
+		if ($users) {
 			// multi utilisateur = erreur
-			if (count($users) == 1)
-			{
+			if (count($users) == 1) {
 				return reset($users);
 			}
 		}
 
 		// recherche l'utilisateur par email
- 		$query = $this->createQueryBuilder('u')
-		->where('u.email = :email')
-		->setParameter('email', $username)
-		->getQuery();
+		$query = $this->createQueryBuilder('u')
+			->where('u.email = :email')
+			->setParameter('email', $username)
+			->getQuery();
 
 		$users = $query->execute();
-		if (!$users)
-		{
+		if (!$users) {
 			return null;
 		}
 
 		// multi utilisateur = erreur
-		if (count($users) > 1)
-		{
-            throw new \Exception('Cette adresse email correspond à plusieurs utilisateurs, veuillez saisir votre identifiant.');
+		if (count($users) > 1) {
+			throw new \Exception('Cette adresse email correspond à plusieurs utilisateurs, veuillez saisir votre identifiant.');
 		}
 
 		return reset($users);
-    }
+	}
 }

templates/bundles/TwigBundle/Exception/error404.html.twig

@@ -15,10 +15,9 @@
 			
 			{% block content %}
 				{% set text = exception.message %}
-				{% if text == '' %}
-					{% set text = 'error_404_funny_text'|trans %}
+				{% if text != '' %}
+					<p class="h3">{{ text }}</p>
 				{% endif %}
-				<p class="h3">{{ text }}</p>
 
 				<p class="">
 					<a href="{{ path('bo_homepage') }}">{{ 'action_go_back_home'|trans }}</a>