Revert "restricted access to CommandManager's list"

This reverts commit 00753477de.
This commit is contained in:
mmacierzanka committed 2021-08-31 19:59:52 +02:00
1 parent 00753477de
commit 6bf657c3d2
3 files changed
+5 -17

No files matched your search

+3 -6
View File
@@ -132,6 +132,7 @@ export class Bot extends EventEmitter {
const helpMsg: Command = new HelpMessage(this.commands, this.messages.help, this.name); const helpMsg: Command = new HelpMessage(this.commands, this.messages.help, this.name);
this.commands.add(helpMsg); this.commands.add(helpMsg);
} }
Object.freeze(this.commands.list);
process.stdout.write("Connecting to Discord... "); process.stdout.write("Connecting to Discord... ");
this.client.login(this.token); this.client.login(this.token);
this.client.on("ready", async () => { this.client.on("ready", async () => {
@@ -247,15 +248,11 @@ export class Bot extends EventEmitter {
process.stdout.write("Registering commands... "); process.stdout.write("Registering commands... ");
await axios.put( await axios.put(
`https://discord.com/api/v8/applications/${this.applicationId}/commands`, `https://discord.com/api/v8/applications/${this.applicationId}/commands`,
this.commands this.commands.list.filter((c) => !Array.isArray(c.guilds) && c.slash).map((c) => c.toCommandObject()),
.getList()
.filter((c) => !Array.isArray(c.guilds) && c.slash)
.map((c) => c.toCommandObject()),
{ headers: { Authorization: `Bot ${this.token}` } } { headers: { Authorization: `Bot ${this.token}` } }
); );
const guilds: any = {}; const guilds: any = {};
await this.commands await this.commands.list
.getList()
.filter((c) => Array.isArray(c.guilds) && c.guilds.length > 0) .filter((c) => Array.isArray(c.guilds) && c.guilds.length > 0)
.map((c) => { .map((c) => {
c.guilds?.map(async (g) => { c.guilds?.map(async (g) => {
+1 -10
View File
@@ -12,7 +12,7 @@ export class CommandManager {
* List of commands registered in the manager * List of commands registered in the manager
* @type {Array} {@link Command} * @type {Array} {@link Command}
*/ */
private readonly list: Command[]; public readonly list: Command[];
/** /**
* Prefix used as a way to trigger the bot using messages * Prefix used as a way to trigger the bot using messages
* @type {string} * @type {string}
@@ -56,15 +56,6 @@ export class CommandManager {
return command; return command;
} }
/**
* Get all commands registered in a manager
* @returns {Command[]} List of registered commands in a manager
*/
getList(): Command[] {
const list = this.list;
return list;
}
/** /**
* Adds the given {@link Command} to the instance manager with initial processing * Adds the given {@link Command} to the instance manager with initial processing
* @param {Command} command - {@link Command} instance object * @param {Command} command - {@link Command} instance object
+1 -1
View File
@@ -61,7 +61,7 @@ export class HelpMessage extends Command {
helpMsg.setTitle(params.title); helpMsg.setTitle(params.title);
helpMsg.setDescription(params.bottomText); helpMsg.setDescription(params.bottomText);
cmdManager.getList().map((c) => { cmdManager.list.map((c) => {
if ( if (
c.visible && c.visible &&
((Array.isArray(c.guilds) && c.guilds.length > 0 && c.guilds.find((g) => i?.guild?.id === g)) || !Array.isArray(c.guilds) || c.guilds.length === 0) ((Array.isArray(c.guilds) && c.guilds.length > 0 && c.guilds.find((g) => i?.guild?.id === g)) || !Array.isArray(c.guilds) || c.guilds.length === 0)