Merge pull request 'chore (fetch datas) #39 : fermeture connexion sql après login$' (#40) from issue-39 into develop

Reviewed-on: #40

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);
     }

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,