Fix bulk deletion error if < 2 messages

Also return error if no server (dm)
This commit is contained in:
Manuel Kraus
2016-05-24 23:44:26 +02:00
parent fae03042cb
commit 8bee117445
3 changed files with 24 additions and 15 deletions

View File

@@ -849,7 +849,7 @@ var InternalClient = (function () {
InternalClient.prototype.deleteMessages = function deleteMessages(_messages) { InternalClient.prototype.deleteMessages = function deleteMessages(_messages) {
if (!_messages instanceof Array) return Promise.reject(new Error("Messages provided must be in an array")); if (!_messages instanceof Array) return Promise.reject(new Error("Messages provided must be in an array"));
if (_messages.length < 1) return Promise.reject(new Error("You must provide at least one message to delete")); if (_messages.length < 2) return Promise.reject(new Error("You must provide at least two messages to delete"));
var messages = []; var messages = [];
var channel; var channel;
@@ -870,6 +870,10 @@ var InternalClient = (function () {
var message = this.resolver.resolveMessage(_message); var message = this.resolver.resolveMessage(_message);
if (!message) return Promise.reject(new Error("Something other than a message could not be resolved in the array...")); if (!message) return Promise.reject(new Error("Something other than a message could not be resolved in the array..."));
if (!message.server) {
return Promise.reject(new Error("You can only bulk delete messages on guild channels"));
}
// ensure same channel // ensure same channel
if (!channel) { if (!channel) {
channel = message.channel; channel = message.channel;

View File

@@ -701,8 +701,9 @@ export default class InternalClient {
deleteMessages(_messages) { deleteMessages(_messages) {
if (!_messages instanceof Array) if (!_messages instanceof Array)
return Promise.reject(new Error("Messages provided must be in an array")); return Promise.reject(new Error("Messages provided must be in an array"));
if (_messages.length < 1) if (_messages.length < 2)
return Promise.reject(new Error("You must provide at least one message to delete")) return Promise.reject(new Error("You must provide at least two messages to delete"));
var messages = []; var messages = [];
var channel; var channel;
@@ -711,6 +712,10 @@ export default class InternalClient {
if (!message) if (!message)
return Promise.reject(new Error("Something other than a message could not be resolved in the array...")); return Promise.reject(new Error("Something other than a message could not be resolved in the array..."));
if(!message.server) {
return Promise.reject(new Error("You can only bulk delete messages on guild channels"));
}
// ensure same channel // ensure same channel
if (!channel) { if (!channel) {
channel = message.channel; channel = message.channel;