fix(Command): Do not attempt to execute application command reloading logic when a command does not have any application commands (#598)

favna · web-flow · commit 61e96d27236d · 2023-02-08T19:19:19.000+01:00
diff --git a/src/lib/structures/Command.ts b/src/lib/structures/Command.ts
@@ -299,6 +299,11 @@ export class Command<PreParseReturn = Args, O extends Command.Options = Command.
 			}
 		}
 
+		// If there are no API calls to execute then exit out early
+		if (!updatedRegistry['apiCalls'].length) {
+			return;
+		}
+
 		// If the default behavior is set to bulk overwrite, handle it as such and return.
 		if (getDefaultBehaviorWhenNotIdentical() === RegisterBehavior.BulkOverwrite) {
 			await handleBulkOverwrite(store, this.container.client.application!.commands);

Original file line number	Diff line number	Diff line change
`@@ -299,6 +299,11 @@ export class Command<PreParseReturn = Args, O extends Command.Options = Command.`
`299`	`299`	`}`
`300`	`300`	`}`
`301`	`301`
	`302`	`+ // If there are no API calls to execute then exit out early`
	`303`	`+ if (!updatedRegistry['apiCalls'].length) {`
	`304`	`+ return;`
	`305`	`+ }`
	`306`	`+`
`302`	`307`	`// If the default behavior is set to bulk overwrite, handle it as such and return.`
`303`	`308`	`if (getDefaultBehaviorWhenNotIdentical() === RegisterBehavior.BulkOverwrite) {`
`304`	`309`	`await handleBulkOverwrite(store, this.container.client.application!.commands);`