Nomchy/discord.js
1
0
Fork 0
mirror of https://github.com/discordjs/discord.js.git synced 2026-03-09 16:13:31 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix(*Collector): always run postCheck, remove 'translatation' of message collector options (#3718)

Browse Source 
* fix(*Collector): always run postCheck, correctly 'translate' message collector options

* fix(MessageCollector): remove translation, fix postCheck conditions
This commit is contained in:
SpaceEEC
2020-01-24 16:56:04 +01:00
committed by GitHub
parent 30adb378fc
commit 099a1a47e8
2 changed files with 12 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/structures/MessageCollector.js
View File

@@ -36,9 +36,6 @@ class MessageCollector extends Collector {
if (this.client.getMaxListeners() !== 0) this.client.setMaxListeners(this.client.getMaxListeners() + 1);
this.client.on('message', this.listener);
// For backwards compatibility (remove in v12)
if (this.options.max) this.options.maxProcessed = this.options.max;
if (this.options.maxMatches) this.options.max = this.options.maxMatches;
this._reEmitter = message => {
/**
* Emitted when the collector receives a message.
@@ -81,8 +78,8 @@ class MessageCollector extends Collector {
*/
postCheck() {
// Consider changing the end reasons for v12
if (this.options.maxMatches && this.collected.size >= this.options.max) return 'matchesLimit';
if (this.options.max && this.received >= this.options.maxProcessed) return 'limit';
if (this.options.maxMatches && this.collected.size >= this.options.maxMatches) return 'matchesLimit';
if (this.options.max && this.received >= this.options.max) return 'limit';
return null;
}

20
src/structures/interfaces/Collector.js
View File

@@ -79,17 +79,17 @@ class Collector extends EventEmitter {
*/
_handle(...args) {
const collect = this.handle(...args);
if (!collect || !this.filter(...args, this.collected)) return;
if (collect && this.filter(...args, this.collected)) {
this.collected.set(collect.key, collect.value);
this.collected.set(collect.key, collect.value);
/**
* Emitted whenever an element is collected.
* @event Collector#collect
* @param {*} element The element that got collected
* @param {Collector} collector The collector
*/
this.emit('collect', collect.value, this);
/**
* Emitted whenever an element is collected.
* @event Collector#collect
* @param {*} element The element that got collected
* @param {Collector} collector The collector
*/
this.emit('collect', collect.value, this);
}
const post = this.postCheck(...args);
if (post) this.stop(post);