Skip to content

Commit

Permalink
refactor the existing createOptimistic... functions to buildOptimisti…
Browse files Browse the repository at this point in the history
…c...
  • Loading branch information
Julesssss committed Sep 7, 2022
1 parent 4147659 commit 22c5612
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 15 deletions.
2 changes: 1 addition & 1 deletion src/libs/actions/Policy.js
Original file line number Diff line number Diff line change
Expand Up @@ -810,7 +810,7 @@ function createWorkspace() {
expenseChatReportID,
expenseChatData,
expenseReportActionData,
} = Report.createOptimisticWorkspaceChats(policyID, workspaceName);
} = Report.buildOptimisticWorkspaceChats(policyID, workspaceName);

// We need to use makeRequestWithSideEffects as we try to redirect to the policy right after creation
// The policy hasn't been merged in Onyx data at this point, leading to an intermittent Not Found screen
Expand Down
28 changes: 14 additions & 14 deletions src/libs/actions/Report.js
Original file line number Diff line number Diff line change
Expand Up @@ -586,7 +586,7 @@ function fetchAllReports(
}

/**
* Creates an optimistic chat report with a randomly generated reportID and as much information as we currently have
* Builds an optimistic chat report with a randomly generated reportID and as much information as we currently have
*
* @param {Array} participantList
* @param {String} reportName
Expand All @@ -597,7 +597,7 @@ function fetchAllReports(
* @param {String} oldPolicyName
* @returns {Object}
*/
function createOptimisticChatReport(
function buildOptimisticChatReport(
participantList,
reportName = 'Chat Report',
chatType = '',
Expand Down Expand Up @@ -632,7 +632,7 @@ function createOptimisticChatReport(
}

/*
* Creates an optimistic IOU report with a randomly generated reportID
* Builds an optimistic IOU report with a randomly generated reportID
*/
function buildOptimisticIOUReport(total, chatReportID, currency, locale) {
const formattedTotal = NumberFormatUtils.format(locale,
Expand All @@ -659,7 +659,7 @@ function buildOptimisticIOUReport(total, chatReportID, currency, locale) {
* @param {String} ownerEmail
* @returns {Object}
*/
function createOptimisticCreatedReportAction(ownerEmail) {
function buildOptimisticCreatedReportAction(ownerEmail) {
return {
0: {
actionName: CONST.REPORT.ACTIONS.TYPE.CREATED,
Expand Down Expand Up @@ -699,8 +699,8 @@ function createOptimisticCreatedReportAction(ownerEmail) {
* @param {String} policyName
* @returns {Object}
*/
function createOptimisticWorkspaceChats(policyID, policyName) {
const announceChatData = createOptimisticChatReport(
function buildOptimisticWorkspaceChats(policyID, policyName) {
const announceChatData = buildOptimisticChatReport(
[currentUserEmail],
CONST.REPORT.WORKSPACE_CHAT_ROOMS.ANNOUNCE,
CONST.REPORT.CHAT_TYPE.POLICY_ANNOUNCE,
Expand All @@ -710,15 +710,15 @@ function createOptimisticWorkspaceChats(policyID, policyName) {
policyName,
);
const announceChatReportID = announceChatData.reportID;
const announceReportActionData = createOptimisticCreatedReportAction(announceChatData.ownerEmail);
const announceReportActionData = buildOptimisticCreatedReportAction(announceChatData.ownerEmail);

const adminsChatData = createOptimisticChatReport([currentUserEmail], CONST.REPORT.WORKSPACE_CHAT_ROOMS.ADMINS, CONST.REPORT.CHAT_TYPE.POLICY_ADMINS, policyID, null, false, policyName);
const adminsChatData = buildOptimisticChatReport([currentUserEmail], CONST.REPORT.WORKSPACE_CHAT_ROOMS.ADMINS, CONST.REPORT.CHAT_TYPE.POLICY_ADMINS, policyID, null, false, policyName);
const adminsChatReportID = adminsChatData.reportID;
const adminsReportActionData = createOptimisticCreatedReportAction(adminsChatData.ownerEmail);
const adminsReportActionData = buildOptimisticCreatedReportAction(adminsChatData.ownerEmail);

const expenseChatData = createOptimisticChatReport([currentUserEmail], '', CONST.REPORT.CHAT_TYPE.POLICY_EXPENSE_CHAT, policyID, currentUserEmail, true, policyName);
const expenseChatData = buildOptimisticChatReport([currentUserEmail], '', CONST.REPORT.CHAT_TYPE.POLICY_EXPENSE_CHAT, policyID, currentUserEmail, true, policyName);
const expenseChatReportID = expenseChatData.reportID;
const expenseReportActionData = createOptimisticCreatedReportAction(expenseChatData.ownerEmail);
const expenseReportActionData = buildOptimisticCreatedReportAction(expenseChatData.ownerEmail);

return {
announceChatReportID,
Expand Down Expand Up @@ -1655,10 +1655,10 @@ export {
readOldestAction,
openReport,
openPaymentDetailsPage,
createOptimisticWorkspaceChats,
createOptimisticChatReport,
buildOptimisticWorkspaceChats,
buildOptimisticChatReport,
buildOptimisticIOUReport,
createOptimisticCreatedReportAction,
buildOptimisticCreatedReportAction,
updatePolicyRoomName,
clearPolicyRoomNameErrors,
};

0 comments on commit 22c5612

Please sign in to comment.