From 79844c0d6673dd2709a0fe996038b5e9ed14769e Mon Sep 17 00:00:00 2001 From: Tristan VALCKE Date: Mon, 21 Oct 2019 14:18:57 +0200 Subject: [PATCH] fix(tabstractconvertermanager): fix missing return statement on next usage --- sources/core/converters/TAbstractConverterManager.js | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/sources/core/converters/TAbstractConverterManager.js b/sources/core/converters/TAbstractConverterManager.js index c5dedfc..fdb6547 100644 --- a/sources/core/converters/TAbstractConverterManager.js +++ b/sources/core/converters/TAbstractConverterManager.js @@ -384,14 +384,15 @@ class TAbstractConverterManager { processFiles ( request, response, next ) { - const files = TAbstractConverterManager._convertFilesObjectToArray( request.files ) - const numberOfFiles = files.length + const files = TAbstractConverterManager._convertFilesObjectToArray( request.files ) + const numberOfFiles = files.length if ( numberOfFiles === 0 ) { if ( this._useNext ) { - next( `Aucun fichier à traiter !` ); + next( `Aucun fichier à traiter !` ) + return } else { - TAbstractConverterManager.returnError( `Aucun fichier à traiter !`, response ); + TAbstractConverterManager.returnError( `Aucun fichier à traiter !`, response ) } } @@ -408,6 +409,7 @@ class TAbstractConverterManager { if ( this._useNext ) { next( `Le fichier ${file.name} à déjà été inséré.` ) + return } else { TAbstractConverterManager.returnError( `Le fichier ${file.name} à déjà été inséré.`, response ) } @@ -424,6 +426,7 @@ class TAbstractConverterManager { if ( this._useNext ) { next( `Impossible d'analyser ${availableFilesNumber} fichiers associatifs simultanément !` ) + return } else { TAbstractConverterManager.returnError( `Impossible d'analyser ${availableFilesNumber} fichiers associatifs simultanément !`, response ) }