From 22229d6b4312108144e1f132272dd9e9d848fdf7 Mon Sep 17 00:00:00 2001
From: Christian Schneppe <christian@pix-art.de>
Date: Thu, 13 Apr 2017 21:24:52 +0200
Subject: treat URL as file if URL is in oob or contains key

---
 .../de/pixart/messenger/ui/ConversationFragment.java   | 18 ++----------------
 1 file changed, 2 insertions(+), 16 deletions(-)

(limited to 'src/main/java/de/pixart/messenger/ui/ConversationFragment.java')

diff --git a/src/main/java/de/pixart/messenger/ui/ConversationFragment.java b/src/main/java/de/pixart/messenger/ui/ConversationFragment.java
index fc735785c..7324f27b0 100644
--- a/src/main/java/de/pixart/messenger/ui/ConversationFragment.java
+++ b/src/main/java/de/pixart/messenger/ui/ConversationFragment.java
@@ -627,14 +627,13 @@ public class ConversationFragment extends Fragment implements EditMessage.Keyboa
             MenuItem shareWith = menu.findItem(R.id.share_with);
             MenuItem sendAgain = menu.findItem(R.id.send_again);
             MenuItem copyUrl = menu.findItem(R.id.copy_url);
-            MenuItem downloadFile = menu.findItem(R.id.download_file);
             MenuItem cancelTransmission = menu.findItem(R.id.cancel_transmission);
             MenuItem deleteFile = menu.findItem(R.id.delete_file);
             MenuItem showErrorMessage = menu.findItem(R.id.show_error_message);
             if (!treatAsFile
                     && !GeoHelper.isGeoUri(m.getBody())
                     && !XmppUri.isXmppUri(m.getBody())
-                    && m.treatAsDownloadable() != Message.Decision.MUST) {
+                    && !m.treatAsDownloadable()) {
                 copyText.setVisible(true);
                 selectText.setVisible(ListSelectionManager.isSupported());
             }
@@ -662,15 +661,10 @@ public class ConversationFragment extends Fragment implements EditMessage.Keyboa
             if (m.hasFileOnRemoteHost()
                     || GeoHelper.isGeoUri(m.getBody())
                     || XmppUri.isXmppUri(m.getBody())
-                    || m.treatAsDownloadable() == Message.Decision.MUST
+                    || m.treatAsDownloadable()
                     || (t != null && t instanceof HttpDownloadConnection)) {
                 copyUrl.setVisible(true);
             }
-            if ((m.getType() == Message.TYPE_TEXT && t == null && m.treatAsDownloadable() != Message.Decision.NEVER)
-                    || (m.isFileOrImage() && t instanceof TransferablePlaceholder && m.hasFileOnRemoteHost())) {
-                downloadFile.setVisible(true);
-                downloadFile.setTitle(activity.getString(R.string.download_x_file, UIHelper.getFileDescriptionString(activity, m)));
-            }
             boolean waitingOfferedSending = m.getStatus() == Message.STATUS_WAITING
                     || m.getStatus() == Message.STATUS_UNSEND
                     || m.getStatus() == Message.STATUS_OFFERED;
@@ -712,9 +706,6 @@ public class ConversationFragment extends Fragment implements EditMessage.Keyboa
             case R.id.copy_url:
                 copyUrl(selectedMessage);
                 return true;
-            case R.id.download_file:
-                downloadFile(selectedMessage);
-                return true;
             case R.id.cancel_transmission:
                 cancelTransmission(selectedMessage);
                 return true;
@@ -839,11 +830,6 @@ public class ConversationFragment extends Fragment implements EditMessage.Keyboa
         }
     }
 
-    private void downloadFile(Message message) {
-        activity.xmppConnectionService.getHttpConnectionManager()
-                .createNewDownloadConnection(message, true);
-    }
-
     private void cancelTransmission(Message message) {
         Transferable transferable = message.getTransferable();
         if (transferable != null) {
-- 
cgit v1.2.3