From 6c4b16c53bde279a938c5622a92fcd967d534df3 Mon Sep 17 00:00:00 2001 From: Harsohail Brar Date: Sun, 22 May 2022 15:26:08 -0600 Subject: [PATCH] fix import js to only run when import elements are present --- app/javascript/src/import.js | 22 +++++++++++++--------- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/app/javascript/src/import.js b/app/javascript/src/import.js index 15dd38700f..e33358a30e 100644 --- a/app/javascript/src/import.js +++ b/app/javascript/src/import.js @@ -55,17 +55,21 @@ function populateFileInput (inputId) { $('document').ready(() => { ['volunteer', 'supervisor'].forEach((importType) => { const inputFileElementId = `${importType}-file` + const inputFileElement = $(`#${inputFileElementId}`)[0] + const importButtonElement = $(`#${importType}-import-button`)[0] - document.getElementById(inputFileElementId).addEventListener('change', function (event) { - document.getElementById(`${importType}-import-button`).disabled = event.target.value === '' - const file = document.getElementById(inputFileElementId).files[0] - storeCSVFile(file, inputFileElementId) - }) + if (inputFileElement && importButtonElement) { + inputFileElement.addEventListener('change', function (event) { + importButtonElement.disabled = event.target.value === '' + const file = inputFileElement.files[0] + storeCSVFile(file, inputFileElementId) + }) - if (document.getElementById('smsOptIn') == null) { - delete localStorage[inputFileElementId] - } else { - populateFileInput(inputFileElementId) + if ($('#smsOptIn') == null) { + delete localStorage[inputFileElementId] + } else { + populateFileInput(inputFileElementId) + } } }) })