Merge pull request #1726 from kongminhao/master

Fix #1725

flask_admin/_compat.py

@@ -38,7 +38,7 @@ if not PY2:
 
     # Various tools
     from functools import reduce
-    from urllib.parse import urljoin, urlparse
+    from urllib.parse import urljoin, urlparse, quote
 else:
     text_type = unicode
     string_types = (str, unicode)
@@ -62,6 +62,7 @@ else:
     # Helpers
     reduce = __builtins__['reduce'] if isinstance(__builtins__, dict) else __builtins__.reduce
     from urlparse import urljoin, urlparse
+    from urllib import quote
 
 
 def with_metaclass(meta, *bases):

flask_admin/contrib/fileadmin/__init__.py

@@ -12,7 +12,7 @@ from werkzeug import secure_filename
 from wtforms import fields, validators
 
 from flask_admin import form, helpers
-from flask_admin._compat import urljoin, as_unicode
+from flask_admin._compat import urljoin, as_unicode, quote
 from flask_admin.base import BaseView, expose
 from flask_admin.actions import action, ActionsMixin
 from flask_admin.babel import gettext, lazy_gettext
@@ -944,7 +944,7 @@ class BaseFileAdmin(BaseView, ActionsMixin):
         base_url = self.get_base_url()
         if base_url:
             base_url = urljoin(self.get_url('.index_view'), base_url)
-            return redirect(urljoin(base_url, path))
+            return redirect(urljoin(quote(base_url), quote(path)))
 
         return self.storage.send_file(directory)