fix: Keep download action for files and hide only for relevant files

Signed-off-by: Julius Härtl <jus@bitgrid.net>

2 files changed, 20 insertions, 0 deletions

diff --git a/apps/files/js/fileactions.js b/apps/files/js/fileactions.js
index ec09ec1bb8c..fc79d63687f 100644
--- a/apps/files/js/fileactions.js
+++ b/apps/files/js/fileactions.js
@@ -653,6 +653,13 @@
 						context.fileList.showFileBusyState(filename, true);
 						OCA.Files.Files.handleDownload(url, disableLoadingState);
 					}
+				},
+				shouldRender: function(context) {
+					var fileData = context.fileInfoModel
+					if (_.isFunction(fileData.canDownload) && !fileData.canDownload()) {
+						return false;
+					}
+					return true;
 				}
 			});
 
@@ -707,6 +714,13 @@
 							}
 							context.fileList.dirInfo.dirLastCopiedTo = targetPath;
 						}, false, "httpd/unix-directory", true, actions, dialogDir);
+				},
+				shouldRender: function(context) {
+					var fileData = context.fileInfoModel
+					if (_.isFunction(fileData.canDownload) && !fileData.canDownload()) {
+						return false;
+					}
+					return true;
 				}
 			});
 
diff --git a/apps/files/js/fileactionsmenu.js b/apps/files/js/fileactionsmenu.js
index 7eecf209984..f6166c22e27 100644
--- a/apps/files/js/fileactionsmenu.js
+++ b/apps/files/js/fileactionsmenu.js
@@ -82,6 +82,12 @@
 			var defaultAction = fileActions.getCurrentDefaultFileAction();
 
 			var items = _.filter(actions, function(actionSpec) {
+				if (actionSpec.shouldRender) {
+					if (!actionSpec.shouldRender(self._context)) {
+						return false;
+					}
+				}
+
 				return !defaultAction || actionSpec.name !== defaultAction.name;
 			});
 			items = _.map(items, function(item) {