diff --git a/src/SQLLogin/SQLLoginConnect.php b/src/SQLLogin/SQLLoginConnect.php index 86b55a6..c2a7cfe 100644 --- a/src/SQLLogin/SQLLoginConnect.php +++ b/src/SQLLogin/SQLLoginConnect.php @@ -13,9 +13,8 @@ class SQLLoginConnect extends AbstractController * Méthode qui crée l'unique instance de la classe * si elle n'existe pas encore puis la retourne. * - * @return SQLLoginConnect */ - public static function getInstance() + public static function getInstance(): SQLLoginConnect { if (is_null(self::$_instance)) { self::$_instance = new SQLLoginConnect(); @@ -24,7 +23,7 @@ class SQLLoginConnect extends AbstractController return self::$_instance; } - public function connect($urlDatabase, $dbUser, $dbPassword) + public function connect($urlDatabase, $dbUser, $dbPassword): PDO { return new PDO($urlDatabase, $dbUser, $dbPassword); } diff --git a/src/Service/SQLLoginService.php b/src/Service/SQLLoginService.php index 2011d60..46a23ab 100644 --- a/src/Service/SQLLoginService.php +++ b/src/Service/SQLLoginService.php @@ -26,6 +26,9 @@ class SQLLoginService extends AbstractController public function fetchDatas(string $login): array { + if (empty($login)) { + throw new Exception('Connexion échouée, le login ne peut pas être vide'); + } try { $dbh = $this->getConnection(); } catch (PDOException $e) { @@ -44,11 +47,15 @@ class SQLLoginService extends AbstractController $query = $dbh->prepare($request); $query->execute([$this->sqlLoginRequest->getLoginColumnName() => $login]); $datas = $query->fetch(PDO::FETCH_ASSOC); + $query->closeCursor(); } catch (PDOException $e) { $this->loggerInterface->critical($e->getMessage()); throw new DataToFetchConfigurationException($e->getMessage()); } + if (false === $datas) { + throw new Exception(sprintf('La requête sql "%s" a renvoyé un résultat incorrect.', $request)); + } return $datas; } @@ -68,6 +75,7 @@ class SQLLoginService extends AbstractController $query = $dbh->prepare($request); $query->execute([$this->sqlLoginRequest->getLoginColumnName() => $login]); $password = $query->fetch(PDO::FETCH_ASSOC); + $query->closeCursor(); } catch (PDOException $e) { $this->loggerInterface->critical($e->getMessage()); throw new LoginElementsConfigurationException($e->getMessage()); @@ -75,7 +83,6 @@ class SQLLoginService extends AbstractController if (!$password) { throw new Exception('Une erreur est survenue lors de la récupération des données'); } - return [ $password[$this->sqlLoginRequest->getPasswordColumnName()], isset($password[$this->sqlLoginRequest->getSaltColumnName()]) ? $password[$this->sqlLoginRequest->getSaltColumnName()] : null,