Optional sendFile name callback and remove extra logic

This commit is contained in:
Nicholas Tay
2016-01-25 10:35:44 +11:00
parent 47f35a942c
commit 3a8f1ddbf9
2 changed files with 11 additions and 3 deletions

View File

@@ -209,7 +209,11 @@ var Client = (function (_EventEmitter) {
Client.prototype.sendFile = function sendFile(where, attachment, name) {
var callback = arguments.length <= 3 || arguments[3] === undefined ? function () /*err, m*/{} : arguments[3];
name = name ? name : require('path').basename(attachment);
if (typeof name === "function") {
// name is the callback
callback = name;
name = undefined; // Will get resolved into original filename in internal
}
return this.internal.sendFile(where, attachment, name).then(dataCallback(callback), errorCallback(callback));
};

View File

@@ -191,8 +191,12 @@ export default class Client extends EventEmitter {
// def sendFile
sendFile(where, attachment, name, callback = (/*err, m*/) => { }) {
name = name ? name : require('path').basename(attachment);
if (typeof name === "function") {
// name is the callback
callback = name;
name = undefined; // Will get resolved into original filename in internal
}
return this.internal.sendFile(where, attachment, name)
.then(dataCallback(callback), errorCallback(callback));
}