zer0/harmony
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Fix Client#connect()'s logic to prioritize the token parameter over Client#token

Browse source 
When Client#token was set and the token parameter is passed to Client#connect(), the module would throw an 'No Token Provided' error. This change makes it prioritize the parameter over Client#token.
This commit is contained in:
Martín (Netux) Rodríguez 2021-02-03 01:17:01 -03:00
parent 6a8e4ddc43
commit 7af4553fbe
1 changed files with 3 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/models/client.ts
View file

@ -291,10 +291,9 @@ export class Client extends HarmonyEventEmitter<ClientEvents> {
* @param intents Gateway intents in array. This is required if not given in ClientOptions.
*/
async connect(token?: string, intents?: GatewayIntents[]): Promise<Client> {
if (token === undefined && this.token !== undefined) token = this.token
else if (this.token === undefined && token !== undefined) {
this.token = token
} else throw new Error('No Token Provided')
token ??= this.token
if (token === undefined) throw new Error('No Token Provided')
this.token = token
if (intents !== undefined && this.intents !== undefined) {
this.debug(
'client',