From bf913929d38e42aeee4c09211aa387476a4aa1ed Mon Sep 17 00:00:00 2001 From: Andrew Rosiclair Date: Mon, 10 Oct 2022 12:53:42 -0400 Subject: [PATCH] added safety checks for policy API reads --- src/libs/actions/Policy.js | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/src/libs/actions/Policy.js b/src/libs/actions/Policy.js index 6af4d574bcbb..607211baa963 100644 --- a/src/libs/actions/Policy.js +++ b/src/libs/actions/Policy.js @@ -13,6 +13,7 @@ import ROUTES from '../../ROUTES'; import * as OptionsListUtils from '../OptionsListUtils'; import DateUtils from '../DateUtils'; import * as ReportUtils from '../ReportUtils'; +import Log from '../Log'; const allPolicies = {}; Onyx.connect({ @@ -919,10 +920,20 @@ function createWorkspace() { } function openWorkspaceReimburseView(policyID) { + if (!policyID) { + Log.warn('openWorkspaceReimburseView invalid params', {policyID}); + return; + } + API.read('OpenWorkspaceReimburseView', {policyID}); } function openWorkspaceMembersPage(policyID, clientMemberEmails) { + if (!policyID || !clientMemberEmails) { + Log.warn('openWorkspaceMembersPage invalid params', {policyID, clientMemberEmails}); + return; + } + API.read('OpenWorkspaceMembersPage', { policyID, clientMemberEmails: JSON.stringify(clientMemberEmails), @@ -930,6 +941,11 @@ function openWorkspaceMembersPage(policyID, clientMemberEmails) { } function openWorkspaceInvitePage(policyID, clientMemberEmails) { + if (!policyID || !clientMemberEmails) { + Log.warn('openWorkspaceInvitePage invalid params', {policyID, clientMemberEmails}); + return; + } + API.read('OpenWorkspaceInvitePage', { policyID, clientMemberEmails: JSON.stringify(clientMemberEmails),