Skip to content

Commit

Permalink
fix(tabstractconvertermanager): fix missing return statement on next …
Browse files Browse the repository at this point in the history
…usage
  • Loading branch information
Itee committed Oct 21, 2019
1 parent 77a41b9 commit 79844c0
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions sources/core/converters/TAbstractConverterManager.js
Original file line number Diff line number Diff line change
Expand Up @@ -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 )
}

}
Expand All @@ -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 )
}
Expand All @@ -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 )
}
Expand Down

0 comments on commit 79844c0

Please sign in to comment.