libretic/etherpad-lite
6
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

LibreOffice: Use the async-provided callback to signal errors

Browse source 
This avoids having two callbacks, which improves readability.
This commit is contained in:
Richard Hansen 2021-03-17 19:14:36 -04:00 committed by John McLear
parent b914a46a87
commit b321267e66
1 changed files with 4 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
src/node/utils/LibreOffice.js
View file

@ -73,10 +73,7 @@ const doConvertTask = (task, callback) => {
libreOfficeLogger.debug(`Renaming ${sourcePath} to ${task.destFile}`);
fs.rename(sourcePath, task.destFile, callback);
},
], (err) => {
callback();
task.callback(err);
});
], callback);
};
// Conversion tasks will be queued up, so we don't overload the system
@ -116,17 +113,15 @@ exports.convertFile = (srcFile, destFile, type, callback) => {
destFile: intermediateFile,
type: 'odt',
fileExtension: 'odt',
callback,
}),
}, callback),
(callback) => queue.push({
srcFile: intermediateFile,
destFile,
type,
callback,
fileExtension,
}),
}, callback),
], callback);
} else {
queue.push({srcFile, destFile, type, callback, fileExtension});
queue.push({srcFile, destFile, type, fileExtension}, callback);
}
};