Skip to content

Commit

Permalink
Merge pull request #47342 from daledah/fix/46877
Browse files Browse the repository at this point in the history
fix: copied content is different from system message
  • Loading branch information
cristipaval authored Aug 19, 2024
2 parents d43f6e1 + 39a1b67 commit fc401c9
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 7 deletions.
9 changes: 9 additions & 0 deletions src/libs/ReportActionsUtils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1594,6 +1594,14 @@ function getUpdateRoomDescriptionMessage(reportAction: ReportAction): string {
return Localize.translateLocal('roomChangeLog.clearRoomDescription');
}

function getRenamedAction(reportAction: OnyxEntry<ReportAction<typeof CONST.REPORT.ACTIONS.TYPE.RENAMED>>) {
const initialMessage = getOriginalMessage(reportAction);
return Localize.translateLocal('newRoomPage.renamedRoomAction', {
oldName: initialMessage?.oldName ?? '',
newName: initialMessage?.newName ?? '',
});
}

export {
doesReportHaveVisibleActions,
extractLinksFromMessageHtml,
Expand Down Expand Up @@ -1688,6 +1696,7 @@ export {
getExportIntegrationMessageHTML,
getUpdateRoomDescriptionMessage,
didMessageMentionCurrentUser,
getRenamedAction,
};

export type {LastVisibleMessage};
3 changes: 1 addition & 2 deletions src/libs/SidebarUtils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -395,8 +395,7 @@ function getOptionData({
const lastActionName = lastAction?.actionName ?? report.lastActionType;

if (ReportActionsUtils.isRenamedAction(lastAction)) {
const newName = ReportActionsUtils.getOriginalMessage(lastAction)?.newName ?? '';
result.alternateText = Localize.translate(preferredLocale, 'newRoomPage.roomRenamedTo', {newName});
result.alternateText = ReportActionsUtils.getRenamedAction(lastAction);
} else if (ReportActionsUtils.isTaskAction(lastAction)) {
result.alternateText = ReportUtils.formatReportLastMessageText(TaskUtils.getTaskReportActionMessage(lastAction).text);
} else if (ReportActionsUtils.isInviteOrRemovedAction(lastAction)) {
Expand Down
2 changes: 2 additions & 0 deletions src/pages/home/report/ContextMenu/ContextMenuActions.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -435,6 +435,8 @@ const ContextMenuActions: ContextMenuAction[] = [
setClipboardMessage(mentionWhisperMessage);
} else if (ReportActionsUtils.isActionableTrackExpense(reportAction)) {
setClipboardMessage(CONST.ACTIONABLE_TRACK_EXPENSE_WHISPER_MESSAGE);
} else if (ReportActionsUtils.isRenamedAction(reportAction)) {
setClipboardMessage(ReportActionsUtils.getRenamedAction(reportAction));
} else if (reportAction?.actionName === CONST.REPORT.ACTIONS.TYPE.SUBMITTED) {
const displayMessage = ReportUtils.getIOUSubmittedMessage(reportID);
Clipboard.setString(displayMessage);
Expand Down
6 changes: 1 addition & 5 deletions src/pages/home/report/ReportActionItem.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -660,11 +660,7 @@ function ReportActionItem({
} else if (ReportActionsUtils.isActionOfType(action, CONST.REPORT.ACTIONS.TYPE.EXPORTED_TO_INTEGRATION)) {
children = <ExportIntegration action={action} />;
} else if (ReportActionsUtils.isRenamedAction(action)) {
const initialMessage = ReportActionsUtils.getOriginalMessage(action);
const message = translate('newRoomPage.renamedRoomAction', {
oldName: initialMessage?.oldName ?? '',
newName: initialMessage?.newName ?? '',
});
const message = ReportActionsUtils.getRenamedAction(action);
children = <ReportActionItemBasicMessage message={message} />;
} else {
const hasBeenFlagged =
Expand Down

0 comments on commit fc401c9

Please sign in to comment.