Merge branch 'sas' into 'master'

Fixed some issues with filepaths and see the future with some preload

See merge request !59

core/models.py

@@ -12,6 +12,8 @@ from django.contrib.staticfiles.storage import staticfiles_storage
 from django.utils.html import escape
 from django.utils.functional import cached_property
 
+import os
+
 from phonenumber_field.modelfields import PhoneNumberField
 
 from datetime import datetime, timedelta, date
@@ -664,10 +666,10 @@ class SithFile(models.Model):
             if self.is_folder:
                 for c in self.children.all():
                     c.move_to(self)
-                shutil.rmtree(settings.MEDIA_ROOT + old_file_name)
+                shutil.rmtree(os.path.join(settings.MEDIA_ROOT, old_file_name))
             else:
                 self.file.save(name=self.name, content=self.file)
-                os.remove(settings.MEDIA_ROOT + old_file_name)
+                os.remove(os.path.join(settings.MEDIA_ROOT, old_file_name))
 
     def __getattribute__(self, attr):
         if attr == "is_file":

core/views/files.py

@@ -38,10 +38,11 @@ def send_file(request, file_id, file_class=SithFile, file_attr="file"):
             ):
         raise PermissionDenied
     name = f.__getattribute__(file_attr).name
-    with open((settings.MEDIA_ROOT + name).encode('utf-8'), 'rb') as filename:
+    filepath = os.path.join(settings.MEDIA_ROOT, name)
+    with open(filepath.encode('utf-8'), 'rb') as filename:
         wrapper = FileWrapper(filename)
         response = HttpResponse(wrapper, content_type=f.mime_type)
-        response['Content-Length'] = os.path.getsize((settings.MEDIA_ROOT + name).encode('utf-8'))
+        response['Content-Length'] = os.path.getsize(filepath.encode('utf-8'))
         response['Content-Disposition'] = ('inline; filename="%s"' % f.name).encode('utf-8')
         return response