Fix: Can delete attachments in archived workspace

src/components/AnchorForAttachmentsOnly/BaseAnchorForAttachmentsOnly.js

@@ -13,6 +13,7 @@ import * as Download from '../../libs/actions/Download';
 import fileDownload from '../../libs/fileDownload';
 import addEncryptedAuthTokenToURL from '../../libs/addEncryptedAuthTokenToURL';
 import {ShowContextMenuContext, showContextMenuForReport} from '../ShowContextMenuContext';
+import * as ReportUtils from '../../libs/ReportUtils';
 
 const propTypes = {
     /** Press in handler for the link */
@@ -48,7 +49,7 @@ const BaseAnchorForAttachmentsOnly = (props) => {
         <ShowContextMenuContext.Consumer>
             {({
                 anchor,
-                reportID,
+                report,
                 action,
                 checkIfContextMenuActive,
             }) => (
@@ -66,9 +67,10 @@ const BaseAnchorForAttachmentsOnly = (props) => {
                     onLongPress={event => showContextMenuForReport(
                         event,
                         anchor,
-                        reportID,
+                        report.reportID,
                         action,
                         checkIfContextMenuActive,
+                        ReportUtils.isArchivedRoom(report),
                     )}
                 >
                     <AttachmentView

src/components/HTMLEngineProvider/HTMLRenderers/ImageRenderer.js

@@ -7,6 +7,7 @@ import PressableWithoutFocus from '../../PressableWithoutFocus';
 import CONST from '../../../CONST';
 import {ShowContextMenuContext, showContextMenuForReport} from '../../ShowContextMenuContext';
 import tryResolveUrlFromApiRoot from '../../../libs/tryResolveUrlFromApiRoot';
+import * as ReportUtils from '../../../libs/ReportUtils';
 
 const ImageRenderer = (props) => {
     const htmlAttribs = props.tnode.attributes;
@@ -53,13 +54,13 @@ const ImageRenderer = (props) => {
         <ShowContextMenuContext.Consumer>
             {({
                 anchor,
-                reportID,
+                report,
                 action,
                 checkIfContextMenuActive,
             }) => (
                 <AttachmentModal
                     allowDownload
-                    reportID={reportID}
+                    reportID={report.reportID}
                     source={source}
                     isAuthTokenRequired={isAttachment}
                     originalFileName={originalFileName}
@@ -68,7 +69,7 @@ const ImageRenderer = (props) => {
                         <PressableWithoutFocus
                             style={styles.noOutline}
                             onPress={show}
-                            onLongPress={event => showContextMenuForReport(event, anchor, reportID, action, checkIfContextMenuActive)}
+                            onLongPress={event => showContextMenuForReport(event, anchor, report.reportID, action, checkIfContextMenuActive, ReportUtils.isArchivedRoom(report))}
                         >
                             <ThumbnailImage
                                 previewSourceURL={previewSource}

src/components/HTMLEngineProvider/HTMLRenderers/PreRenderer/BasePreRenderer.js

@@ -7,6 +7,7 @@ import htmlRendererPropTypes from '../htmlRendererPropTypes';
 import withLocalize from '../../../withLocalize';
 import {ShowContextMenuContext, showContextMenuForReport} from '../../../ShowContextMenuContext';
 import styles from '../../../../styles/styles';
+import * as ReportUtils from '../../../../libs/ReportUtils';
 
 const propTypes = {
     /** Press in handler for the code block */
@@ -36,14 +37,14 @@ const BasePreRenderer = forwardRef((props, ref) => {
             <ShowContextMenuContext.Consumer>
                 {({
                     anchor,
-                    reportID,
+                    report,
                     action,
                     checkIfContextMenuActive,
                 }) => (
                     <TouchableWithoutFeedback
                         onPressIn={props.onPressIn}
                         onPressOut={props.onPressOut}
-                        onLongPress={event => showContextMenuForReport(event, anchor, reportID, action, checkIfContextMenuActive)}
+                        onLongPress={event => showContextMenuForReport(event, anchor, report.reportID, action, checkIfContextMenuActive, ReportUtils.isArchivedRoom(report))}
                     >
                         <View>
                             {/* eslint-disable-next-line react/jsx-props-no-spreading */}

src/components/ShowContextMenuContext.js

@@ -5,7 +5,7 @@ import * as DeviceCapabilities from '../libs/DeviceCapabilities';
 
 const ShowContextMenuContext = React.createContext({
     anchor: null,
-    reportID: null,
+    report: null,
     action: undefined,
     checkIfContextMenuActive: () => {},
 });
@@ -20,12 +20,12 @@ ShowContextMenuContext.displayName = 'ShowContextMenuContext';
  * @param {String} reportID - Active Report ID
  * @param {Object} action - ReportAction for ContextMenu
  * @param {Function} checkIfContextMenuActive Callback to update context menu active state
+ * @param {Boolean} [isArchivedRoom=false] - Is the report an archived room
  */
-function showContextMenuForReport(event, anchor, reportID, action, checkIfContextMenuActive) {
+function showContextMenuForReport(event, anchor, reportID, action, checkIfContextMenuActive, isArchivedRoom = false) {
     if (!DeviceCapabilities.canUseTouchScreen()) {
         return;
     }
-
     ReportActionContextMenu.showContextMenu(
         ContextMenuActions.CONTEXT_MENU_TYPES.REPORT_ACTION,
         event,
@@ -36,6 +36,7 @@ function showContextMenuForReport(event, anchor, reportID, action, checkIfContex
         '',
         checkIfContextMenuActive,
         checkIfContextMenuActive,
+        isArchivedRoom,
     );
 }
 

src/pages/home/report/ReportActionItem.js

@@ -193,7 +193,7 @@ class ReportActionItem extends Component {
                 <ShowContextMenuContext.Provider
                     value={{
                         anchor: this.popoverAnchor,
-                        reportID: this.props.report.reportID,
+                        report: this.props.report,
                         action: this.props.action,
                         checkIfContextMenuActive: this.checkIfContextMenuActive,
                     }}

src/pages/home/report/ReportActionsList.js

@@ -132,7 +132,10 @@ const ReportActionsList = (props) => {
 
     // Native mobile does not render updates flatlist the changes even though component did update called.
     // To notify there something changes we can use extraData prop to flatlist
-    const extraData = (!props.isDrawerOpen && props.isSmallScreenWidth) ? props.newMarkerReportActionID : undefined;
+    const extraData = [
+        (!props.isDrawerOpen && props.isSmallScreenWidth) ? props.newMarkerReportActionID : undefined,
+        ReportUtils.isArchivedRoom(props.report),
+    ];
     const shouldShowReportRecipientLocalTime = ReportUtils.canShowReportRecipientLocalTime(props.personalDetails, props.report);
     return (
         <Animated.View style={[animatedStyles, styles.flex1]}>