From 7de7fcf56147a2a125b009984e2b399b8c3cacde Mon Sep 17 00:00:00 2001
From: Daniel Gasienica <daniel@signal.org>
Date: Tue, 27 Mar 2018 12:13:25 -0400
Subject: [PATCH] Avoid `no-param-reassign` violation

---
 js/modules/messages_data_migrator.js | 12 +++---------
 1 file changed, 3 insertions(+), 9 deletions(-)

diff --git a/js/modules/messages_data_migrator.js b/js/modules/messages_data_migrator.js
index 1ba211e0f..7b07ad03c 100644
--- a/js/modules/messages_data_migrator.js
+++ b/js/modules/messages_data_migrator.js
@@ -311,13 +311,7 @@ const setItem = (connection, key, value) => {
 
 const completeTransaction = transaction =>
   new Promise((resolve, reject) => {
-    // eslint-disable-next-line no-param-reassign
-    transaction.onabort = event =>
-      reject(event.target.error);
-    // eslint-disable-next-line no-param-reassign
-    transaction.onerror = event =>
-      reject(event.target.error);
-    // eslint-disable-next-line no-param-reassign
-    transaction.oncomplete = () =>
-      resolve();
+    transaction.addEventListener('abort', event => reject(event.target.error));
+    transaction.addEventListener('error', event => reject(event.target.error));
+    transaction.addEventListener('complete', () => resolve());
   });