From 88b8bda2e0bd4d25ddbe60d075c8ae522692a579 Mon Sep 17 00:00:00 2001
From: Fernando Sanz <fsanz@izertis.com>
Date: Thu, 28 Apr 2022 14:11:20 +0200
Subject: [PATCH 1/9] Buttons created

---
 .../main/res/layout/file_details_fragment.xml | 49 +++++++++++++++++++
 owncloudApp/src/main/res/values/strings.xml   |  2 +
 owncloudApp/src/main/res/values/styles.xml    | 12 ++++-
 3 files changed, 62 insertions(+), 1 deletion(-)

diff --git a/owncloudApp/src/main/res/layout/file_details_fragment.xml b/owncloudApp/src/main/res/layout/file_details_fragment.xml
index 2d170bf6a2d..62a601f7e50 100644
--- a/owncloudApp/src/main/res/layout/file_details_fragment.xml
+++ b/owncloudApp/src/main/res/layout/file_details_fragment.xml
@@ -154,6 +154,55 @@
             app:layout_constraintEnd_toEndOf="parent"
             app:layout_constraintTop_toBottomOf="@id/fdProgressText" />
 
+        <View
+            android:id="@+id/view_file_detail_separator"
+            android:layout_width="0dp"
+            android:layout_height="1dp"
+            android:layout_marginTop="@dimen/standard_padding"
+            android:alpha="0.8"
+            android:background="@color/owncloud_blue"
+            app:layout_constraintLeft_toLeftOf="parent"
+            app:layout_constraintRight_toRightOf="parent"
+            app:layout_constraintTop_toBottomOf="@id/fdProgressBar" />
+
+        <Button
+            android:id="@+id/button_file_detail_download"
+            style="@style/Button.FileDetail"
+            android:layout_width="0dp"
+            android:layout_height="wrap_content"
+            android:drawableStart="@drawable/ic_cloud_download"
+            android:text="@string/filedetails_download"
+            app:layout_constraintLeft_toLeftOf="parent"
+            app:layout_constraintTop_toBottomOf="@id/view_file_detail_separator" />
+
+        <Button
+            android:id="@+id/button_file_detail_open_with"
+            style="@style/Button.FileDetail"
+            android:layout_width="0dp"
+            android:layout_height="wrap_content"
+            android:drawableStart="@drawable/ic_share"
+            android:text="@string/filedetails_open_with"
+            app:layout_constraintLeft_toLeftOf="parent"
+            app:layout_constraintTop_toBottomOf="@id/button_file_detail_download" />
+
+        <Button
+            android:id="@+id/button_file_detail_open_send"
+            style="@style/Button.FileDetail"
+            android:layout_width="0dp"
+            android:layout_height="wrap_content"
+            android:drawableStart="@drawable/ic_send"
+            android:text="@string/filedetails_send"
+            app:layout_constraintLeft_toLeftOf="parent"
+            app:layout_constraintTop_toBottomOf="@id/button_file_detail_open_with"
+            app:layout_constraintRight_toLeftOf="@id/barrier"/>
+
+        <androidx.constraintlayout.widget.Barrier
+            android:id="@+id/barrier"
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            app:barrierDirection="end"
+            app:constraint_referenced_ids="button_file_detail_download,button_file_detail_open_with" />
+
     </androidx.constraintlayout.widget.ConstraintLayout>
 
 </ScrollView>
diff --git a/owncloudApp/src/main/res/values/strings.xml b/owncloudApp/src/main/res/values/strings.xml
index 25682bd2be2..7d2f075f7eb 100644
--- a/owncloudApp/src/main/res/values/strings.xml
+++ b/owncloudApp/src/main/res/values/strings.xml
@@ -168,6 +168,8 @@
     <string name="filedetails_created">Created:</string>
     <string name="filedetails_modified">Modified:</string>
     <string name="filedetails_download">Download</string>
+    <string name="filedetails_open_with">Open with</string>
+    <string name="filedetails_send">Send</string>
     <string name="filedetails_file_image_content_description">File Image</string>
     <string name="filedetails_cancel_btn_content_description">Cancel download button</string>
     <string name="filedetails_sync_file">Synchronize</string>
diff --git a/owncloudApp/src/main/res/values/styles.xml b/owncloudApp/src/main/res/values/styles.xml
index 441d30ebb98..d83e6109672 100644
--- a/owncloudApp/src/main/res/values/styles.xml
+++ b/owncloudApp/src/main/res/values/styles.xml
@@ -16,7 +16,7 @@
   You should have received a copy of the GNU General Public License
   along with this program.  If not, see <http://www.gnu.org/licenses/>.
 -->
-<resources xmlns:android="http://schemas.android.com/apk/res/android">
+<resources xmlns:android="http://schemas.android.com/apk/res/android" xmlns:tools="http://schemas.android.com/tools">
 
     <style name="Animations" />
 
@@ -298,4 +298,14 @@
     <style name="Theme.ownCloud.SortBottomSheetFragment.Item">
         <item name="android:textAppearance">@style/TextAppearance.OwnCloud.SortBottomSheetFragment.Item</item>
     </style>
+
+    <style name="Button.FileDetail" parent="Widget.MaterialComponents.Button.TextButton">
+        <item name="textAllCaps">true</item>
+        <item name="android:textSize">12sp</item>
+        <item name="android:textColor">@color/owncloud_blue</item>
+        <item name="android:gravity">start|center_vertical</item>
+        <item name="android:drawableTint" tools:targetApi="m">@color/owncloud_blue</item>
+        <item name="android:layout_marginTop">@dimen/standard_half_margin</item>
+        <item name="android:drawablePadding">@dimen/standard_margin</item>
+    </style>
 </resources>

From fe057d99ea81d144042712761d147aff1d02a317 Mon Sep 17 00:00:00 2001
From: Fernando Sanz <fsanz@izertis.com>
Date: Fri, 29 Apr 2022 09:44:29 +0200
Subject: [PATCH 2/9] Button functionality added

---
 .../ui/fragment/FileDetailFragment.java       | 77 ++++++++++++++++---
 .../main/res/layout/file_details_fragment.xml | 12 +--
 owncloudApp/src/main/res/values/strings.xml   |  1 +
 3 files changed, 75 insertions(+), 15 deletions(-)

diff --git a/owncloudApp/src/main/java/com/owncloud/android/ui/fragment/FileDetailFragment.java b/owncloudApp/src/main/java/com/owncloud/android/ui/fragment/FileDetailFragment.java
index 5dd81b7baa1..c2b58f0276c 100644
--- a/owncloudApp/src/main/java/com/owncloud/android/ui/fragment/FileDetailFragment.java
+++ b/owncloudApp/src/main/java/com/owncloud/android/ui/fragment/FileDetailFragment.java
@@ -33,6 +33,7 @@
 import android.view.View;
 import android.view.View.OnClickListener;
 import android.view.ViewGroup;
+import android.widget.Button;
 import android.widget.ImageView;
 import android.widget.LinearLayout;
 import android.widget.ProgressBar;
@@ -153,6 +154,7 @@ public View onCreateView(@NotNull LayoutInflater inflater, ViewGroup container,
         }
 
         updateFileDetails(false, false);
+        setButtonsClickListener();
         return mView;
     }
 
@@ -281,7 +283,7 @@ public boolean onOptionsItemSelected(MenuItem item) {
                 return true;
             }
             case R.id.action_open_file_with: {
-                mContainerActivity.getFileOperationsHelper().openFile(getFile());
+                openWith();
                 return true;
             }
             case R.id.action_remove_file: {
@@ -300,17 +302,11 @@ public boolean onOptionsItemSelected(MenuItem item) {
             }
             case R.id.action_download_file:
             case R.id.action_sync_file: {
-                mContainerActivity.getFileOperationsHelper().syncFile(getFile());
+                downloadFile();
                 return true;
             }
             case R.id.action_send_file: {
-                // Obtain the file
-                if (!getFile().isDown()) {  // Download the file                    
-                    Timber.d("%s : File must be downloaded", getFile().getRemotePath());
-                    ((FileDisplayActivity) mContainerActivity).startDownloadForSending(getFile());
-                } else {
-                    mContainerActivity.getFileOperationsHelper().sendDownloadedFile(getFile());
-                }
+                sendFile();
                 return true;
             }
             case R.id.action_set_available_offline: {
@@ -326,6 +322,24 @@ public boolean onOptionsItemSelected(MenuItem item) {
         }
     }
 
+    private void openWith() {
+        mContainerActivity.getFileOperationsHelper().openFile(getFile());
+    }
+
+    private void downloadFile() {
+        mContainerActivity.getFileOperationsHelper().syncFile(getFile());
+    }
+
+    private void sendFile() {
+        // Obtain the file
+        if (!getFile().isDown()) {  // Download the file
+            Timber.d("%s : File must be downloaded", getFile().getRemotePath());
+            ((FileDisplayActivity) mContainerActivity).startDownloadForSending(getFile());
+        } else {
+            mContainerActivity.getFileOperationsHelper().sendDownloadedFile(getFile());
+        }
+    }
+
     @Override
     public void onClick(View v) {
         switch (v.getId()) {
@@ -335,6 +349,25 @@ public void onClick(View v) {
             }
             case R.id.fdIcon: {
                 displayFile(getFile());
+
+                break;
+            }
+            case R.id.button_file_detail_download: {
+                OCFile file = getFile();
+                if (file.isDown()) {
+
+                } else {
+                    downloadFile();
+                }
+                break;
+            }
+            case R.id.button_file_detail_open_with: {
+                openWith();
+                break;
+            }
+            case R.id.button_file_detail_send: {
+                sendFile();
+                break;
             }
             default:
                 Timber.e("Incorrect view clicked!");
@@ -426,6 +459,7 @@ private void updateFileDetails(boolean forcedTransferring, boolean refresh) {
                 setButtonsForRemote();
             }
         }
+        setOpenOrDownloadButtonAction();
         getView().invalidate();
     }
 
@@ -575,4 +609,29 @@ private void setButtonsForRemote() {
         }
     }
 
+    private void setButtonsClickListener() {
+        Button buttonDownload = getView().findViewById(R.id.button_file_detail_download);
+        Button buttonOpenWith = getView().findViewById(R.id.button_file_detail_open_with);
+        Button buttonSend = getView().findViewById(R.id.button_file_detail_send);
+
+        buttonDownload.setOnClickListener(this);
+        buttonOpenWith.setOnClickListener(this);
+        buttonSend.setOnClickListener(this);
+
+        setOpenOrDownloadButtonAction();
+    }
+
+    private void setOpenOrDownloadButtonAction() {
+        OCFile file = getFile();
+        Button button = getView().findViewById(R.id.button_file_detail_download);
+
+        if (file.isDown()) {
+            button.setText(getString(R.string.filedetails_open));
+            button.setCompoundDrawablesWithIntrinsicBounds(R.drawable.ic_open_in_new, 0, 0, 0);
+        } else {
+            button.setText(getString(R.string.filedetails_download));
+            button.setCompoundDrawablesWithIntrinsicBounds(R.drawable.ic_cloud_download, 0, 0, 0);
+        }
+    }
+
 }
diff --git a/owncloudApp/src/main/res/layout/file_details_fragment.xml b/owncloudApp/src/main/res/layout/file_details_fragment.xml
index 62a601f7e50..144a1fc00cd 100644
--- a/owncloudApp/src/main/res/layout/file_details_fragment.xml
+++ b/owncloudApp/src/main/res/layout/file_details_fragment.xml
@@ -170,10 +170,10 @@
             style="@style/Button.FileDetail"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
-            android:drawableStart="@drawable/ic_cloud_download"
-            android:text="@string/filedetails_download"
             app:layout_constraintLeft_toLeftOf="parent"
-            app:layout_constraintTop_toBottomOf="@id/view_file_detail_separator" />
+            app:layout_constraintTop_toBottomOf="@id/view_file_detail_separator"
+            tools:drawableStart="@drawable/ic_cloud_download"
+            tools:text="@string/filedetails_download" />
 
         <Button
             android:id="@+id/button_file_detail_open_with"
@@ -186,15 +186,15 @@
             app:layout_constraintTop_toBottomOf="@id/button_file_detail_download" />
 
         <Button
-            android:id="@+id/button_file_detail_open_send"
+            android:id="@+id/button_file_detail_send"
             style="@style/Button.FileDetail"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
             android:drawableStart="@drawable/ic_send"
             android:text="@string/filedetails_send"
             app:layout_constraintLeft_toLeftOf="parent"
-            app:layout_constraintTop_toBottomOf="@id/button_file_detail_open_with"
-            app:layout_constraintRight_toLeftOf="@id/barrier"/>
+            app:layout_constraintRight_toLeftOf="@id/barrier"
+            app:layout_constraintTop_toBottomOf="@id/button_file_detail_open_with" />
 
         <androidx.constraintlayout.widget.Barrier
             android:id="@+id/barrier"
diff --git a/owncloudApp/src/main/res/values/strings.xml b/owncloudApp/src/main/res/values/strings.xml
index 7d2f075f7eb..3d06214ac59 100644
--- a/owncloudApp/src/main/res/values/strings.xml
+++ b/owncloudApp/src/main/res/values/strings.xml
@@ -169,6 +169,7 @@
     <string name="filedetails_modified">Modified:</string>
     <string name="filedetails_download">Download</string>
     <string name="filedetails_open_with">Open with</string>
+    <string name="filedetails_open">Open</string>
     <string name="filedetails_send">Send</string>
     <string name="filedetails_file_image_content_description">File Image</string>
     <string name="filedetails_cancel_btn_content_description">Cancel download button</string>

From 537af37683022e0a7f1e68782f5c561e9d063dcf Mon Sep 17 00:00:00 2001
From: Fernando Sanz <fsanz@izertis.com>
Date: Fri, 29 Apr 2022 09:55:22 +0200
Subject: [PATCH 3/9] Added icons

---
 owncloudApp/src/main/res/drawable/ic_cloud_download.xml | 5 +++++
 owncloudApp/src/main/res/drawable/ic_open_in_new.xml    | 5 +++++
 owncloudApp/src/main/res/drawable/ic_send.xml           | 5 +++++
 owncloudApp/src/main/res/drawable/ic_share.xml          | 5 +++++
 4 files changed, 20 insertions(+)
 create mode 100644 owncloudApp/src/main/res/drawable/ic_cloud_download.xml
 create mode 100644 owncloudApp/src/main/res/drawable/ic_open_in_new.xml
 create mode 100644 owncloudApp/src/main/res/drawable/ic_send.xml
 create mode 100644 owncloudApp/src/main/res/drawable/ic_share.xml

diff --git a/owncloudApp/src/main/res/drawable/ic_cloud_download.xml b/owncloudApp/src/main/res/drawable/ic_cloud_download.xml
new file mode 100644
index 00000000000..164165211c7
--- /dev/null
+++ b/owncloudApp/src/main/res/drawable/ic_cloud_download.xml
@@ -0,0 +1,5 @@
+<vector android:height="24dp" android:tint="#7C7C7D"
+    android:viewportHeight="24" android:viewportWidth="24"
+    android:width="24dp" xmlns:android="http://schemas.android.com/apk/res/android">
+    <path android:fillColor="@android:color/white" android:pathData="M19.35,10.04C18.67,6.59 15.64,4 12,4 9.11,4 6.6,5.64 5.35,8.04 2.34,8.36 0,10.91 0,14c0,3.31 2.69,6 6,6h13c2.76,0 5,-2.24 5,-5 0,-2.64 -2.05,-4.78 -4.65,-4.96zM17,13l-5,5 -5,-5h3V9h4v4h3z"/>
+</vector>
diff --git a/owncloudApp/src/main/res/drawable/ic_open_in_new.xml b/owncloudApp/src/main/res/drawable/ic_open_in_new.xml
new file mode 100644
index 00000000000..54837f97419
--- /dev/null
+++ b/owncloudApp/src/main/res/drawable/ic_open_in_new.xml
@@ -0,0 +1,5 @@
+<vector android:height="24dp" android:tint="#7C7C7D"
+    android:viewportHeight="24" android:viewportWidth="24"
+    android:width="24dp" xmlns:android="http://schemas.android.com/apk/res/android">
+    <path android:fillColor="@android:color/white" android:pathData="M19,19H5V5h7V3H5c-1.11,0 -2,0.9 -2,2v14c0,1.1 0.89,2 2,2h14c1.1,0 2,-0.9 2,-2v-7h-2v7zM14,3v2h3.59l-9.83,9.83 1.41,1.41L19,6.41V10h2V3h-7z"/>
+</vector>
diff --git a/owncloudApp/src/main/res/drawable/ic_send.xml b/owncloudApp/src/main/res/drawable/ic_send.xml
new file mode 100644
index 00000000000..35452a519fb
--- /dev/null
+++ b/owncloudApp/src/main/res/drawable/ic_send.xml
@@ -0,0 +1,5 @@
+<vector android:autoMirrored="true" android:height="24dp"
+    android:tint="#7C7C7D" android:viewportHeight="24"
+    android:viewportWidth="24" android:width="24dp" xmlns:android="http://schemas.android.com/apk/res/android">
+    <path android:fillColor="@android:color/white" android:pathData="M2.01,21L23,12 2.01,3 2,10l15,2 -15,2z"/>
+</vector>
diff --git a/owncloudApp/src/main/res/drawable/ic_share.xml b/owncloudApp/src/main/res/drawable/ic_share.xml
new file mode 100644
index 00000000000..a25f642592b
--- /dev/null
+++ b/owncloudApp/src/main/res/drawable/ic_share.xml
@@ -0,0 +1,5 @@
+<vector android:height="24dp" android:tint="#7C7C7D"
+    android:viewportHeight="24" android:viewportWidth="24"
+    android:width="24dp" xmlns:android="http://schemas.android.com/apk/res/android">
+    <path android:fillColor="@android:color/white" android:pathData="M18,16.08c-0.76,0 -1.44,0.3 -1.96,0.77L8.91,12.7c0.05,-0.23 0.09,-0.46 0.09,-0.7s-0.04,-0.47 -0.09,-0.7l7.05,-4.11c0.54,0.5 1.25,0.81 2.04,0.81 1.66,0 3,-1.34 3,-3s-1.34,-3 -3,-3 -3,1.34 -3,3c0,0.24 0.04,0.47 0.09,0.7L8.04,9.81C7.5,9.31 6.79,9 6,9c-1.66,0 -3,1.34 -3,3s1.34,3 3,3c0.79,0 1.5,-0.31 2.04,-0.81l7.12,4.16c-0.05,0.21 -0.08,0.43 -0.08,0.65 0,1.61 1.31,2.92 2.92,2.92 1.61,0 2.92,-1.31 2.92,-2.92s-1.31,-2.92 -2.92,-2.92z"/>
+</vector>

From 1db994c551e67a9cb990e59bddf2ccf846b8caab Mon Sep 17 00:00:00 2001
From: Fernando Sanz <fsanz@izertis.com>
Date: Wed, 4 May 2022 11:51:08 +0200
Subject: [PATCH 4/9] The menu button options have been removed

---
 .../owncloud/android/files/FileMenuFilter.java    | 15 ++++++++-------
 .../android/ui/fragment/FileDetailFragment.java   |  2 +-
 .../android/ui/fragment/OCFileListFragment.java   |  2 +-
 .../android/ui/preview/PreviewAudioFragment.java  |  2 +-
 .../android/ui/preview/PreviewImageFragment.kt    |  2 +-
 .../android/ui/preview/PreviewTextFragment.java   |  2 +-
 .../android/ui/preview/PreviewVideoFragment.java  |  2 +-
 7 files changed, 14 insertions(+), 13 deletions(-)

diff --git a/owncloudApp/src/main/java/com/owncloud/android/files/FileMenuFilter.java b/owncloudApp/src/main/java/com/owncloud/android/files/FileMenuFilter.java
index 65d5a0ec31c..b61c1ae75e5 100644
--- a/owncloudApp/src/main/java/com/owncloud/android/files/FileMenuFilter.java
+++ b/owncloudApp/src/main/java/com/owncloud/android/files/FileMenuFilter.java
@@ -93,7 +93,7 @@ public FileMenuFilter(OCFile targetFile, Account account, ComponentsGetter cg,
      * @param menu Options or context menu to filter.
      */
     public void filter(Menu menu, boolean displaySelectAll, boolean displaySelectInverse,
-                       boolean onlyAvailableOffline, boolean sharedByLinkFiles) {
+                       boolean onlyAvailableOffline, boolean sharedByLinkFiles, boolean isFileDetailFragment) {
         if (mFiles == null || mFiles.size() <= 0) {
             hideAll(menu);
 
@@ -101,7 +101,7 @@ public void filter(Menu menu, boolean displaySelectAll, boolean displaySelectInv
             List<Integer> toShow = new ArrayList<>();
             List<Integer> toHide = new ArrayList<>();
 
-            filter(toShow, toHide, displaySelectAll, displaySelectInverse, onlyAvailableOffline, sharedByLinkFiles);
+            filter(toShow, toHide, displaySelectAll, displaySelectInverse, onlyAvailableOffline, sharedByLinkFiles, isFileDetailFragment);
 
             MenuItem item;
             for (int i : toShow) {
@@ -140,8 +140,8 @@ private void hideAll(Menu menu) {
      * @param toHide List to save the options that must be shown in the menu.
      */
 
-    private void filter(List<Integer> toShow, List<Integer> toHide, boolean displaySelectAll,
-                        boolean displaySelectInverse, boolean onlyAvailableOffline, boolean sharedByLinkFiles) {
+    private void filter(List<Integer> toShow, List<Integer> toHide, boolean displaySelectAll, boolean displaySelectInverse,
+                        boolean onlyAvailableOffline, boolean sharedByLinkFiles, boolean isFileDetailFragment) {
 
         boolean synchronizing = anyFileSynchronizing();
 
@@ -164,7 +164,7 @@ private void filter(List<Integer> toShow, List<Integer> toHide, boolean displayS
 
         // DOWNLOAD
         if (mFiles.isEmpty() || containsFolder() || anyFileDown() || synchronizing || videoPreviewing ||
-                onlyAvailableOffline || sharedByLinkFiles) {
+                onlyAvailableOffline || sharedByLinkFiles || isFileDetailFragment) {
             toHide.add(R.id.action_download_file);
 
         } else {
@@ -198,7 +198,7 @@ private void filter(List<Integer> toShow, List<Integer> toHide, boolean displayS
         }
 
         // OPEN WITH (different to preview!)
-        if (!isSingleFile() || !anyFileDown() || synchronizing) {
+        if (!isSingleFile() || !anyFileDown() || synchronizing || isFileDetailFragment) {
             toHide.add(R.id.action_open_file_with);
 
         } else {
@@ -249,7 +249,8 @@ private void filter(List<Integer> toShow, List<Integer> toHide, boolean displayS
         // SEND
         boolean sendAllowed = (mContext != null &&
                 mContext.getString(R.string.send_files_to_other_apps).equalsIgnoreCase("on"));
-        if (containsFolder() || (!areDownloaded() && !isSingleFile()) || !sendAllowed || synchronizing || videoStreaming || onlyAvailableOffline) {
+
+        if (containsFolder() || (!areDownloaded() && !isSingleFile()) || !sendAllowed || synchronizing || videoStreaming || onlyAvailableOffline || isFileDetailFragment) {
             toHide.add(R.id.action_send_file);
         } else {
             toShow.add(R.id.action_send_file);
diff --git a/owncloudApp/src/main/java/com/owncloud/android/ui/fragment/FileDetailFragment.java b/owncloudApp/src/main/java/com/owncloud/android/ui/fragment/FileDetailFragment.java
index c2b58f0276c..3b71a3d76bc 100644
--- a/owncloudApp/src/main/java/com/owncloud/android/ui/fragment/FileDetailFragment.java
+++ b/owncloudApp/src/main/java/com/owncloud/android/ui/fragment/FileDetailFragment.java
@@ -241,7 +241,7 @@ public void onPrepareOptionsMenu(Menu menu) {
                     mContainerActivity,
                     getActivity()
             );
-            mf.filter(menu, false, false, false, false);
+            mf.filter(menu, false, false, false, false,true);
         }
 
         // additional restriction for this fragment 
diff --git a/owncloudApp/src/main/java/com/owncloud/android/ui/fragment/OCFileListFragment.java b/owncloudApp/src/main/java/com/owncloud/android/ui/fragment/OCFileListFragment.java
index d02e0eae38a..2a9b1dfb830 100644
--- a/owncloudApp/src/main/java/com/owncloud/android/ui/fragment/OCFileListFragment.java
+++ b/owncloudApp/src/main/java/com/owncloud/android/ui/fragment/OCFileListFragment.java
@@ -636,7 +636,7 @@ public boolean onPrepareActionMode(ActionMode mode, Menu menu) {
                     getActivity()
             );
             mf.filter(menu, mEnableSelectAll, true, mFileListOption.isAvailableOffline(),
-                    mFileListOption.isSharedByLink());
+                    mFileListOption.isSharedByLink(),false);
             return true;
         }
 
diff --git a/owncloudApp/src/main/java/com/owncloud/android/ui/preview/PreviewAudioFragment.java b/owncloudApp/src/main/java/com/owncloud/android/ui/preview/PreviewAudioFragment.java
index bce42db4532..c64b800f635 100644
--- a/owncloudApp/src/main/java/com/owncloud/android/ui/preview/PreviewAudioFragment.java
+++ b/owncloudApp/src/main/java/com/owncloud/android/ui/preview/PreviewAudioFragment.java
@@ -315,7 +315,7 @@ public void onPrepareOptionsMenu(Menu menu) {
                 mContainerActivity,
                 getActivity()
         );
-        mf.filter(menu, false, false, false, false);
+        mf.filter(menu, false, false, false, false,false);
 
         // additional restriction for this fragment 
         // TODO allow renaming in PreviewAudioFragment
diff --git a/owncloudApp/src/main/java/com/owncloud/android/ui/preview/PreviewImageFragment.kt b/owncloudApp/src/main/java/com/owncloud/android/ui/preview/PreviewImageFragment.kt
index ce613c0e776..e26fd371603 100644
--- a/owncloudApp/src/main/java/com/owncloud/android/ui/preview/PreviewImageFragment.kt
+++ b/owncloudApp/src/main/java/com/owncloud/android/ui/preview/PreviewImageFragment.kt
@@ -199,7 +199,7 @@ class PreviewImageFragment : FileFragment() {
                 mContainerActivity,
                 activity
             )
-            fileMenuFilter.filter(menu, false, false, false, false)
+            fileMenuFilter.filter(menu, false, false, false, false,false)
         }
 
         // additional restriction for this fragment
diff --git a/owncloudApp/src/main/java/com/owncloud/android/ui/preview/PreviewTextFragment.java b/owncloudApp/src/main/java/com/owncloud/android/ui/preview/PreviewTextFragment.java
index 5abfa87ac53..c9cf9460f25 100644
--- a/owncloudApp/src/main/java/com/owncloud/android/ui/preview/PreviewTextFragment.java
+++ b/owncloudApp/src/main/java/com/owncloud/android/ui/preview/PreviewTextFragment.java
@@ -301,7 +301,7 @@ public void onPrepareOptionsMenu(Menu menu) {
                     mContainerActivity,
                     getActivity()
             );
-            mf.filter(menu, false, false, false, false);
+            mf.filter(menu, false, false, false, false,false);
         }
 
         // additional restriction for this fragment
diff --git a/owncloudApp/src/main/java/com/owncloud/android/ui/preview/PreviewVideoFragment.java b/owncloudApp/src/main/java/com/owncloud/android/ui/preview/PreviewVideoFragment.java
index 3eb79085164..0386d9d2f9a 100644
--- a/owncloudApp/src/main/java/com/owncloud/android/ui/preview/PreviewVideoFragment.java
+++ b/owncloudApp/src/main/java/com/owncloud/android/ui/preview/PreviewVideoFragment.java
@@ -321,7 +321,7 @@ public void onPrepareOptionsMenu(@NonNull Menu menu) {
                 mContainerActivity,
                 getActivity()
         );
-        mf.filter(menu, false, false, false, false);
+        mf.filter(menu, false, false, false, false, false);
 
         // additional restrictions for this fragment
 

From 98cf1d46fc93d8d0b6d3411267e21fba9ec2ecfc Mon Sep 17 00:00:00 2001
From: Fernando Sanz <fsanz@izertis.com>
Date: Wed, 4 May 2022 13:41:31 +0200
Subject: [PATCH 5/9] Button functionality added

---
 .../com/owncloud/android/ui/fragment/FileDetailFragment.java | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/owncloudApp/src/main/java/com/owncloud/android/ui/fragment/FileDetailFragment.java b/owncloudApp/src/main/java/com/owncloud/android/ui/fragment/FileDetailFragment.java
index 3b71a3d76bc..1a9325d974b 100644
--- a/owncloudApp/src/main/java/com/owncloud/android/ui/fragment/FileDetailFragment.java
+++ b/owncloudApp/src/main/java/com/owncloud/android/ui/fragment/FileDetailFragment.java
@@ -241,7 +241,7 @@ public void onPrepareOptionsMenu(Menu menu) {
                     mContainerActivity,
                     getActivity()
             );
-            mf.filter(menu, false, false, false, false,true);
+            mf.filter(menu, false, false, false, false, true);
         }
 
         // additional restriction for this fragment 
@@ -349,13 +349,12 @@ public void onClick(View v) {
             }
             case R.id.fdIcon: {
                 displayFile(getFile());
-
                 break;
             }
             case R.id.button_file_detail_download: {
                 OCFile file = getFile();
                 if (file.isDown()) {
-
+                    displayFile(getFile());
                 } else {
                     downloadFile();
                 }

From 06f84fa932ad65735d8b519b4c71debedad101fd Mon Sep 17 00:00:00 2001
From: Fernando Sanz <fsanz@izertis.com>
Date: Wed, 4 May 2022 13:46:07 +0200
Subject: [PATCH 6/9] Changelog added

---
 changelog/unreleased/3655 | 7 +++++++
 1 file changed, 7 insertions(+)
 create mode 100644 changelog/unreleased/3655

diff --git a/changelog/unreleased/3655 b/changelog/unreleased/3655
new file mode 100644
index 00000000000..264450cc1b5
--- /dev/null
+++ b/changelog/unreleased/3655
@@ -0,0 +1,7 @@
+Enhancement: Adding action buttons to the detail view of file
+
+Three new buttons have been added to the file details view
+and removed from the toolbar menu.
+
+https://github.com/owncloud/android/issues/3641
+https://github.com/owncloud/android/pull/3655

From e3c66638b04f0d111d18506f47b154743abcbd7c Mon Sep 17 00:00:00 2001
From: fesave <fesave@users.noreply.github.com>
Date: Wed, 4 May 2022 11:46:56 +0000
Subject: [PATCH 7/9] Calens changelog updated

---
 CHANGELOG.md | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 1e804f872a5..b7a0648d4ec 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -29,6 +29,7 @@ Summary
 * Enhancement - Support for SVG files added: [#1033](https://github.com/owncloud/android/issues/1033)
 * Enhancement - Improved copy/move dialog: [#1414](https://github.com/owncloud/android/issues/1414)
 * Enhancement - Thumbnail click action in file detail: [#3653](https://github.com/owncloud/android/pull/3653)
+* Enhancement - Adding action buttons to the detail view of file: [#3641](https://github.com/owncloud/android/issues/3641)
 
 Details
 -------
@@ -194,6 +195,13 @@ Details
 
    https://github.com/owncloud/android/pull/3653
 
+* Enhancement - Adding action buttons to the detail view of file: [#3641](https://github.com/owncloud/android/issues/3641)
+
+   Three new buttons have been added to the file details view and removed from the toolbar menu.
+
+   https://github.com/owncloud/android/issues/3641
+   https://github.com/owncloud/android/pull/3655
+
 Changelog for ownCloud Android Client [2.20.0] (2022-02-16)
 =======================================
 The following sections list the changes in ownCloud Android Client 2.20.0 relevant to

From 9f658942a8100b677d71246a08239e1aae2230cc Mon Sep 17 00:00:00 2001
From: Fernando Sanz <fsanz@izertis.com>
Date: Tue, 10 May 2022 10:49:08 +0200
Subject: [PATCH 8/9] ktlint fixes

---
 .../com/owncloud/android/ui/preview/PreviewImageFragment.kt     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/owncloudApp/src/main/java/com/owncloud/android/ui/preview/PreviewImageFragment.kt b/owncloudApp/src/main/java/com/owncloud/android/ui/preview/PreviewImageFragment.kt
index e26fd371603..9bbec273ebe 100644
--- a/owncloudApp/src/main/java/com/owncloud/android/ui/preview/PreviewImageFragment.kt
+++ b/owncloudApp/src/main/java/com/owncloud/android/ui/preview/PreviewImageFragment.kt
@@ -199,7 +199,7 @@ class PreviewImageFragment : FileFragment() {
                 mContainerActivity,
                 activity
             )
-            fileMenuFilter.filter(menu, false, false, false, false,false)
+            fileMenuFilter.filter(menu, false, false, false, false, false)
         }
 
         // additional restriction for this fragment

From b145378c822adfea2bb314ddcd9cafa8bbdb93dd Mon Sep 17 00:00:00 2001
From: davcres <david.crespo.rios@alumnos.uva.es>
Date: Mon, 16 May 2022 13:29:58 +0200
Subject: [PATCH 9/9] fix crash when open with without being downloaded

---
 .../com/owncloud/android/ui/helpers/FileOperationsHelper.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/owncloudApp/src/main/java/com/owncloud/android/ui/helpers/FileOperationsHelper.java b/owncloudApp/src/main/java/com/owncloud/android/ui/helpers/FileOperationsHelper.java
index 0525d215fca..e3156935ef4 100644
--- a/owncloudApp/src/main/java/com/owncloud/android/ui/helpers/FileOperationsHelper.java
+++ b/owncloudApp/src/main/java/com/owncloud/android/ui/helpers/FileOperationsHelper.java
@@ -88,7 +88,7 @@ private Intent getIntentForGuessedMimeType(String storagePath, String type, Uri
     }
 
     public void openFile(OCFile ocFile) {
-        if (ocFile != null) {
+        if (ocFile != null && ocFile.isDown()) {
             Intent intentForSavedMimeType = getIntentForSavedMimeType(ocFile.getExposedFileUri(mFileActivity), ocFile.getMimetype());
 
             Intent intentForGuessedMimeType = getIntentForGuessedMimeType(ocFile.getStoragePath(), ocFile.getMimetype(),
@@ -97,7 +97,7 @@ public void openFile(OCFile ocFile) {
             openFileWithIntent(intentForSavedMimeType, intentForGuessedMimeType);
 
         } else {
-            Timber.e("Trying to open a NULL OCFile");
+            Timber.e("Trying to open a NULL or not down OCFile");
         }
     }