Records: Add conditional file dumping

invenio_rdm_records/records/api.py

@@ -43,7 +43,6 @@
 from .dumpers import (
     EDTFDumperExt,
     EDTFListDumperExt,
-    FilesDumperExt,
     GrantTokensDumperExt,
     StatisticsDumperExt,
 )
@@ -56,6 +55,7 @@
     RecordStatisticsField,
     TombstoneField,
 )
+from .systemfields.access.protection import Visibility
 from .systemfields.draft_status import DraftStatus
 
 
@@ -97,15 +97,6 @@ class RDMParent(ParentRecordBase):
 #
 # Common properties between records and drafts.
 #
-COMMON_DUMPER_EXTENSIONS = [
-    EDTFDumperExt("metadata.publication_date"),
-    EDTFListDumperExt("metadata.dates", "date"),
-    RelationDumperExt("relations"),
-    CustomFieldsDumperExt(fields_var="RDM_CUSTOM_FIELDS"),
-    StatisticsDumperExt("stats"),
-]
-
-
 class CommonFieldsMixin:
     """Common system fields between records and drafts."""
 
@@ -116,7 +107,15 @@ class CommonFieldsMixin:
     # update the JSONSchema and mappings to a new version.
     schema = ConstantField("$schema", "local://records/record-v6.0.0.json")
 
-    dumper = SearchDumper(extensions=COMMON_DUMPER_EXTENSIONS)
+    dumper = SearchDumper(
+        extensions=[
+            EDTFDumperExt("metadata.publication_date"),
+            EDTFListDumperExt("metadata.dates", "date"),
+            RelationDumperExt("relations"),
+            CustomFieldsDumperExt(fields_var="RDM_CUSTOM_FIELDS"),
+            StatisticsDumperExt("stats"),
+        ]
+    )
 
     relations = MultiRelationsField(
         creator_affiliations=PIDNestedListRelation(
@@ -332,7 +331,6 @@ class RDMDraftMediaFiles(RDMDraft):
 RDMMediaFileDraft.record_cls = RDMDraftMediaFiles
 
 
-#
 # Record API
 #
 class RDMFileRecord(FileRecord):
@@ -358,11 +356,14 @@ class RDMRecord(CommonFieldsMixin, Record):
         "rdmrecords-records-record-v6.0.0", search_alias="rdmrecords-records"
     )
 
-    dumper = SearchDumper(extensions=COMMON_DUMPER_EXTENSIONS + [FilesDumperExt()])
-
     files = FilesField(
         store=False,
         dump=True,
+        # Don't dump files if record is public and files restricted.
+        dump_entries=lambda record: not (
+            record.access.protection.record == Visibility.PUBLIC.value
+            and record.access.protection.files == Visibility.RESTRICTED.value
+        ),
         file_cls=RDMFileRecord,
         # Don't create
         create=False,

invenio_rdm_records/records/dumpers/__init__.py

@@ -9,15 +9,13 @@
 
 from .access import GrantTokensDumperExt
 from .edtf import EDTFDumperExt, EDTFListDumperExt
-from .files import FilesDumperExt
 from .locations import LocationsDumper
 from .pids import PIDsDumperExt
 from .statistics import StatisticsDumperExt
 
 __all__ = (
     "EDTFDumperExt",
     "EDTFListDumperExt",
-    "FilesDumperExt",
     "PIDsDumperExt",
     "GrantTokensDumperExt",
     "LocationsDumper",