From 53914dee915fd8ea789b3873da93ea36630086af Mon Sep 17 00:00:00 2001
From: Gustavo Durand <gdurand@iq.harvard.edu>
Date: Fri, 11 Feb 2022 14:22:25 -0500
Subject: [PATCH] Update ManagePermissionsPage.java

---
 .../java/edu/harvard/iq/dataverse/ManagePermissionsPage.java    | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/main/java/edu/harvard/iq/dataverse/ManagePermissionsPage.java b/src/main/java/edu/harvard/iq/dataverse/ManagePermissionsPage.java
index 590cfb9a671..889656ff8b5 100644
--- a/src/main/java/edu/harvard/iq/dataverse/ManagePermissionsPage.java
+++ b/src/main/java/edu/harvard/iq/dataverse/ManagePermissionsPage.java
@@ -420,6 +420,8 @@ public List<DataverseRole> getAvailableRoles() {
                 }
 
             } else if (dvObject instanceof DataFile) {
+                // TODO: consider changing the logic here to be more like for Dataset
+                // in order to capture new roles that get added having only File scoped permissions
                 roles.add(roleService.findBuiltinRoleByAlias(DataverseRole.FILE_DOWNLOADER));
             }