global: removal of Invenio-Records-UI

* Removes `record_file_factory` extenstion property. (addresses inveniosoftware#134)

* Moves `file_download_ui` utility function to Invenio-Records-Files.

Signed-off-by: Jiri Kuncar <jiri.kuncar@cern.ch>

.editorconfig

@@ -38,7 +38,7 @@ indent_size = 4
 # isort plugin configuration
 known_standard_library = mimetypes
 known_first_party = invenio_files_rest
-known_third_party = invenio_access, invenio_admin, invenio_accounts, invenio_db, invenio_records, invenio_rest, invenio_records_files, invenio_records_ui
+known_third_party = invenio_access, invenio_admin, invenio_accounts, invenio_db, invenio_rest
 multi_line_output = 2
 default_section = THIRDPARTY
 

invenio_files_rest/ext.py

@@ -44,21 +44,6 @@ def __init__(self, app):
         """Initialize state."""
         self.app = app
 
-    @cached_property
-    def record_file_factory(self):
-        """Load default storage factory."""
-        r = load_or_import_from_config(
-            'FILES_REST_RECORD_FILE_FACTORY', app=self.app)
-        if r is not None:
-            return r
-        else:
-            try:
-                get_distribution('invenio-records-files')
-                from invenio_records_files.utils import record_file_factory
-                return record_file_factory
-            except DistributionNotFound:
-                return lambda pid, record, filename: None
-
     @cached_property
     def storage_factory(self):
         """Load default storage factory."""

invenio_files_rest/views.py

@@ -302,52 +302,6 @@ def decorate(*args, **kwargs):
 )
 
 
-#
-# Records-UI integration
-#
-def file_download_ui(pid, record, **kwargs):
-    """File download view for a given record.
-
-    Plug this method into your ``RECORDS_UI_ENDPOINTS`` configuration:
-
-    .. code-block:: python
-
-        RECORDS_UI_ENDPOINTS = dict(
-            recid=dict(
-                # ...
-                route='/records/<pid_value/files/<filename>',
-                view_imp='invenio_files_rest.views.file_download_ui',
-                record_class='invenio_records_files.api:Record',
-            )
-        )
-
-    :param pid: The :class:`invenio_pidstore.models.PersistentIdentifier`
-        instance.
-    :param record: The record metadata.
-    """
-    # Extract file from record.
-    fileobj = current_files_rest.record_file_factory(
-        pid, record, request.view_args.get('filename'))
-
-    if not fileobj:
-        abort(404)
-
-    obj = fileobj.obj
-
-    # Check permissions
-    ObjectResource.check_object_permission(obj)
-
-    # Send file.
-    return ObjectResource.send_object(
-        obj.bucket, obj,
-        expected_chksum=fileobj.get('checksum'),
-        logger_data=dict(
-            bucket_id=obj.bucket_id,
-            pid_type=pid.pid_type,
-            pid_value=pid.pid_value,
-        ))
-
-
 #
 # REST resources
 #

setup.py

@@ -40,8 +40,6 @@
     'invenio-accounts>=1.0.0a15',
     'invenio-admin>=1.0.0a3',
     'invenio-celery>=1.0.0a4',
-    'invenio-records-files>=1.0.0a4',
-    'invenio-records-ui>=1.0.0a5',
     'isort>=4.2.2',
     'mock>=1.3.0',
     'pydocstyle>=1.0.0',