From 78ade1d99d0d29acb29c39d87a23d60c900ae47e Mon Sep 17 00:00:00 2001 From: mabashian Date: Mon, 1 Jun 2020 15:55:18 -0400 Subject: [PATCH] Build inputSourcesMap using an array reduce method instead of forEach with a local variable. --- .../Credential/CredentialEdit/CredentialEdit.jsx | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/awx/ui_next/src/screens/Credential/CredentialEdit/CredentialEdit.jsx b/awx/ui_next/src/screens/Credential/CredentialEdit/CredentialEdit.jsx index 09efb36790..766541533c 100644 --- a/awx/ui_next/src/screens/Credential/CredentialEdit/CredentialEdit.jsx +++ b/awx/ui_next/src/screens/Credential/CredentialEdit/CredentialEdit.jsx @@ -37,11 +37,12 @@ function CredentialEdit({ credential, me }) { CredentialsAPI.readInputSources(credential.id, { page_size: 200 }), ]); setCredentialTypes(loadedCredentialTypes); - const inputSourcesMap = {}; - loadedInputSources.forEach(inputSource => { - inputSourcesMap[inputSource.input_field_name] = inputSource; - }); - setInputSources(inputSourcesMap); + setInputSources( + loadedInputSources.reduce((inputSourcesMap, inputSource) => { + inputSourcesMap[inputSource.input_field_name] = inputSource; + return inputSourcesMap; + }, {}) + ); } catch (err) { setError(err); } finally {