Fixed instantiating DatabaseAdapter multiple times

markokajzer · Mar 20, 2018 · 822fbdc · 822fbdc
1 parent 91dc80b
commit 822fbdc
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/commands/CommandCollection.ts b/src/commands/CommandCollection.ts
@@ -31,10 +31,10 @@ import AvatarCommand from './soundbot/AvatarCommand';
 export default class CommandCollection extends Collection<string, ICommand> {
   private readonly soundCommand: SoundCommand;
 
-  constructor(queue = new SoundQueue(), db = new DatabaseAdapter()) {
+  constructor(db = new DatabaseAdapter(), queue = new SoundQueue(db)) {
     super();
     this.soundCommand = new SoundCommand(queue);
-    this.initializeCommands(queue, db);
+    this.initializeCommands(db, queue);
   }
 
   public registerUserCommands(user: ClientUser) {
@@ -51,7 +51,7 @@ export default class CommandCollection extends Collection<string, ICommand> {
     this.soundCommand.run(message, params);
   }
 
-  private initializeCommands(queue: SoundQueue, db: DatabaseAdapter) {
+  private initializeCommands(db: DatabaseAdapter, queue: SoundQueue) {
     [
       new AddCommand(),
       new RenameCommand(db),

diff --git a/src/queue/SoundQueue.ts b/src/queue/SoundQueue.ts
@@ -11,7 +11,7 @@ export default class SoundQueue {
   private readonly queue: Array<QueueItem>;
   private current: QueueItem | null;
 
-  constructor(db = new DatabaseAdapter()) {
+  constructor(db: DatabaseAdapter) {
     this.db = db;
     this.queue = [];
     this.current = null;