From 3b6ca6fdb87e2ab5433ca7e185d4e366423a215d Mon Sep 17 00:00:00 2001
From: Mattermost Build <build@mattermost.com>
Date: Mon, 26 Aug 2019 16:17:54 +0200
Subject: [PATCH] MM-17792 Do not show download option for files uploaded
 locally (#3192)

---
 .../__snapshots__/image_preview.test.js.snap      |  2 ++
 app/screens/image_preview/image_preview.js        |  5 +++++
 app/screens/image_preview/image_preview.test.js   | 15 +++++++++++++++
 3 files changed, 22 insertions(+)

diff --git a/app/screens/image_preview/__snapshots__/image_preview.test.js.snap b/app/screens/image_preview/__snapshots__/image_preview.test.js.snap
index 115d4db87..fb5dbeee2 100644
--- a/app/screens/image_preview/__snapshots__/image_preview.test.js.snap
+++ b/app/screens/image_preview/__snapshots__/image_preview.test.js.snap
@@ -234,6 +234,8 @@ exports[`ImagePreview should match snapshot 1`] = `
 </AnimatedComponent>
 `;
 
+exports[`ImagePreview should match snapshot and not renderDownloadButton for local files 1`] = `null`;
+
 exports[`ImagePreview should match snapshot, renderDownloadButton 1`] = `
 <TouchableOpacity
   activeOpacity={0.2}
diff --git a/app/screens/image_preview/image_preview.js b/app/screens/image_preview/image_preview.js
index 80dc568a8..68fe38aa9 100644
--- a/app/screens/image_preview/image_preview.js
+++ b/app/screens/image_preview/image_preview.js
@@ -227,6 +227,11 @@ export default class ImagePreview extends PureComponent {
         const {canDownloadFiles} = this.props;
         const file = this.getCurrentFile();
 
+        if (file?.data?.localPath) {
+            // we already have the file locally we don't need to download it
+            return null;
+        }
+
         if (file) {
             let icon;
             let action = emptyFunction;
diff --git a/app/screens/image_preview/image_preview.test.js b/app/screens/image_preview/image_preview.test.js
index 922e26551..90ac21ef9 100644
--- a/app/screens/image_preview/image_preview.test.js
+++ b/app/screens/image_preview/image_preview.test.js
@@ -67,6 +67,21 @@ describe('ImagePreview', () => {
         expect(wrapper.instance().renderDownloadButton()).toMatchSnapshot();
     });
 
+    test('should match snapshot and not renderDownloadButton for local files', () => {
+        const props = {
+            ...baseProps,
+            files: [{caption: 'Caption 1', source: 'source', data: {localPath: 'path'}}],
+        };
+
+        const wrapper = shallow(
+            <ImagePreview {...props}/>,
+            {context: {intl: {formatMessage: jest.fn()}}},
+        );
+
+        expect(wrapper.instance().renderDownloadButton()).toMatchSnapshot();
+        expect(wrapper.instance().renderDownloadButton()).toBeNull();
+    });
+
     test('should match state on handleChangeImage', () => {
         const wrapper = shallow(
             <ImagePreview {...baseProps}/>,