file-store: rename module from upload_store to file_store.

rhodecode/api/views/server_api.py

@@ -39,8 +39,8 b' from rhodecode.lib.utils2 import safe_in'
 from rhodecode.model.db import UserIpMap
 from rhodecode.model.scm import ScmModel
 from rhodecode.model.settings import VcsSettingsModel
-from rhodecode.apps.upload_store import utils
-from rhodecode.apps.upload_store.exceptions import FileNotAllowedException, \
+from rhodecode.apps.file_store import utils
+from rhodecode.apps.file_store.exceptions import FileNotAllowedException, \
     FileOverSizeException
 
 log = logging.getLogger(__name__)
@@ -469,7 +469,7 b' def upload_file(request, apiuser, filena'
                           'user_id': apiuser.user_id,
                           'ip': apiuser.ip_addr}}
     try:
-        store_fid = storage.save_file(file_obj, filename, metadata=metadata)
+        store_fid, metadata = storage.save_file(file_obj, filename, metadata=metadata)
     except FileNotAllowedException:
         raise JSONRPCError('File `{}` is not allowed.'.format(filename))
 

rhodecode/apps/file_store/__init__.py

@@ -18,7 +18,7 b''
 # RhodeCode Enterprise Edition, including its added features, Support services,
 # and proprietary license terms, please see https://rhodecode.com/licenses/
 import os
-from rhodecode.apps.upload_store import config_keys
+from rhodecode.apps.file_store import config_keys
 from rhodecode.config.middleware import _bool_setting, _string_setting
 
 

rhodecode/apps/file_store/local_store.py

@@ -23,9 +23,9 b' import time'
 import shutil
 
 from rhodecode.lib.ext_json import json
-from rhodecode.apps.upload_store import utils
-from rhodecode.apps.upload_store.extensions import resolve_extensions
-from rhodecode.apps.upload_store.exceptions import FileNotAllowedException
+from rhodecode.apps.file_store import utils
+from rhodecode.apps.file_store.extensions import resolve_extensions
+from rhodecode.apps.file_store.exceptions import FileNotAllowedException
 
 METADATA_VER = 'v1'
 
@@ -110,7 +110,8 b' class LocalFileStorage(object):'
         Checks if an extension is permitted. Both e.g. ".jpg" and
         "jpg" can be passed in. Extension lookup is case-insensitive.
 
-        :param extensions: iterable of extensions (or self.extensions)
+        :param ext: extension to check
+        :param extensions: iterable of extensions to validate against (or self.extensions)
         """
 
         extensions = extensions or self.extensions
@@ -160,12 +161,15 b' class LocalFileStorage(object):'
 
         if metadata:
             size = os.stat(path).st_size
-            metadata.update({'size': size, "time": time.time(),
+            metadata.update(
+                {"size": size,
+                 "time": time.time(),
                              "meta_ver": METADATA_VER})
+
             with open(os.path.join(dest_directory, filename_meta), "wb") as dest_meta:
                 dest_meta.write(json.dumps(metadata))
 
         if directory:
             filename = os.path.join(directory, filename)
 
-        return filename
+        return filename, metadata

rhodecode/apps/file_store/tests/test_upload_file.py

@@ -22,7 +22,7 b' import pytest'
 
 from rhodecode.lib.ext_json import json
 from rhodecode.tests import TestController
-from rhodecode.apps.upload_store import utils, config_keys
+from rhodecode.apps.file_store import utils, config_keys
 
 
 def route_path(name, params=None, **kwargs):
@@ -58,7 +58,7 b' class TestFileStoreViews(TestController)'
                 f.write(content)
 
             with open(filesystem_file, 'rb') as f:
-                fid = store.save_file(f, fid, metadata={'filename': fid})
+                fid, metadata = store.save_file(f, fid, metadata={'filename': fid})
 
         else:
             status = 404

rhodecode/apps/file_store/utils.py

@@ -24,8 +24,8 b' import uuid'
 
 
 def get_file_storage(settings):
-    from rhodecode.apps.upload_store.local_store import LocalFileStorage
-    from rhodecode.apps.upload_store import config_keys
+    from rhodecode.apps.file_store.local_store import LocalFileStorage
+    from rhodecode.apps.file_store import config_keys
     store_path = settings.get(config_keys.store_path)
     return LocalFileStorage(base_path=store_path)
 

rhodecode/apps/file_store/views.py

@@ -24,8 +24,8 b' from pyramid.response import FileRespons'
 from pyramid.httpexceptions import HTTPFound, HTTPNotFound
 
 from rhodecode.apps._base import BaseAppView
-from rhodecode.apps.upload_store import utils
-from rhodecode.apps.upload_store.exceptions import (
+from rhodecode.apps.file_store import utils
+from rhodecode.apps.file_store.exceptions import (
     FileNotAllowedException,FileOverSizeException)
 
 from rhodecode.lib import helpers as h
@@ -69,8 +69,7 b' class FileStoreView(BaseAppView):'
                               'user_id': self._rhodecode_user.user_id,
                               'ip': self._rhodecode_user.ip_addr}}
         try:
-            store_fid = self.storage.save_file(file_obj.file, filename,
-                                               metadata=metadata)
+            store_fid, metadata = self.storage.save_file(file_obj.file, filename, metadata=metadata)
         except FileNotAllowedException:
             return {'store_fid': None,
                     'access_path': None,

rhodecode/config/middleware.py

@@ -281,7 +281,7 b' def includeme(config):'
         config.include('rhodecode.apps.ops')
         config.include('rhodecode.apps.admin')
         config.include('rhodecode.apps.channelstream')
-        config.include('rhodecode.apps.upload_store')
+        config.include('rhodecode.apps.file_store')
         config.include('rhodecode.apps.login')
         config.include('rhodecode.apps.home')
         config.include('rhodecode.apps.journal')