diff --git a/frappe/core/doctype/file/file.py b/frappe/core/doctype/file/file.py
index b98c4e3286..c0c5445957 100755
--- a/frappe/core/doctype/file/file.py
+++ b/frappe/core/doctype/file/file.py
@@ -887,8 +887,10 @@ def has_permission(doc, ptype=None, user=None, debug=False):
 		return True
 	if (
 		user != "Guest"
-		and ptype
-		and frappe.share.get_shared("File", filters=[["share_name", "=", doc.name]], rights=[ptype], user=user)
+		and ptype in ["read", "write", "share", "submit"]
+		and frappe.share.get_shared(
+			"File", filters=[["share_name", "=", doc.name]], rights=[ptype], user=user
+		)
 	):
 		return True