Merge pull request #291 from codefori/fix/play_button_state

worksofliam · web-flow · commit 8177f3c90d9d · 2024-10-22T14:01:06.000-04:00
Correctly remove listeners when job is cleaned up
diff --git a/src/connection/sqlJob.ts b/src/connection/sqlJob.ts
@@ -210,5 +210,6 @@ export class OldSQLJob extends SQLJob {
     this.channel.close();
     this.channel = undefined;
     this.status = "ended";
+    this.responseEmitter.removeAllListeners();
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -210,5 +210,6 @@ export class OldSQLJob extends SQLJob {`
`210`	`210`	`this.channel.close();`
`211`	`211`	`this.channel = undefined;`
`212`	`212`	`this.status = "ended";`
	`213`	`+ this.responseEmitter.removeAllListeners();`
`213`	`214`	`}`
`214`	`215`	`}`