problème si les fichiers recu sur Helios sont plusieurs fois en erreur #321 ~Helios

CHANGELOG.md

@@ -12,7 +12,7 @@ et adhère aux principes du [Semantic Versioning](http://semver.org/).
 - problème de nommage de fichier avec pas assez d'entropie et pouvant envoyer de mauvaise information lors de la création des PES via l'APIs
 - Augmentation de la longueur du type de fichier inclu dans actes de 64 à 512 car les réponses de la préfecture peuvent inclure des fichier avec des types très long #320 ~Actes
 - vérification de la balise de scellement sur les annexes (tdt-lib-actes) #319 ~Actes
- 
+- problème si les fichiers recu sur Helios sont plusieurs fois en erreur #321 ~Helios 
 
 ## [3.0.0] - 2017-12-15 
 

class/helios/HeliosAnalyseFichierRecu.class.php

@@ -75,9 +75,13 @@ class HeliosAnalyseFichierRecu {
 
 		foreach($erreur_list as $file => $message){
 			if (file_exists($helios_responses_error_path."/".$file)){
-				$this->log("[ERREUR] Impossible de dplacer le fichier $file dans le rpertoire des fichiers en erreur : le fichier existe dj");
-				$this->sendMailToAdmin("[S2low][Helios] Impossible de dplacer un fichier dans le rpertoire des fichiers en erreur","Le fichier $file existe dj");
-				continue;
+                $i = 0;
+                do {
+                    $i++;
+                    $file_num = "$helios_responses_error_path/$file.$i";
+                } while (file_exists($file_num));
+                $this->log("[WARNING] Le fichier $file existe dj dans le rpertoire des fichiers en erreur : renomm en *.$i");
+                rename($helios_responses_error_path."/".$file,$file_num);
 			}
 			rename($helios_ftp_response_tmp_local_path."/".$file,$helios_responses_error_path."/".$file);
 		}

test/PHPUnit/helios/HeliosAnalyseFichierRecuTest.php

@@ -73,9 +73,10 @@ class HeliosAnalyseFichierRecuTest extends S2lowTestCase {
 	
 	public function testAnalyseDeplacementErreurImpossible(){
 		copy(__DIR__."/fixtures/pes_retour_nonabonne.xml", $this->helios_responses_error_path."/pes_retour_nonabonne.xml");
-		$this->expectOutputRegex("#Impossible de déplacer le fichier pes_retour_nonabonne.xml dans le répertoire des fichiers en erreur : le fichier existe déjà#");
+		//$this->expectOutputRegex("#Impossible de déplacer le fichier pes_retour_nonabonne.xml dans le répertoire des fichiers en erreur : le fichier existe déjà#");
 		$this->analysePesRetour(__DIR__."/fixtures/pes_retour_nonabonne.xml");
-		$this->assertFalse(file_exists($this->helios_response_root."/pes_retour_nonabonne.xml"));
+		$this->assertTrue(file_exists($this->helios_responses_error_path."/pes_retour_nonabonne.xml"));
+        $this->assertTrue(file_exists($this->helios_responses_error_path."/pes_retour_nonabonne.xml.1"));
 	}
 
 	public function testAnalysePesRetourDoubleSire(){