Commit 22ff9439 authored by Serge S. Koval's avatar Serge S. Koval

Use multiple arguments instead of string concatenation

parent ea5d3837
...@@ -546,14 +546,13 @@ class FileAdmin(BaseView, ActionsMixin): ...@@ -546,14 +546,13 @@ class FileAdmin(BaseView, ActionsMixin):
""" """
Edit view method Edit view method
""" """
path = request.args.get('path') path = request.args.getlist('path')
next_url = None next_url = None
if not path: if not path:
return redirect(url_for('.index')) return redirect(url_for('.index'))
path = path.split(':')
if len(path) > 1: if len(path) > 1:
next_url = url_for('.edit', path=':'.join(path[1:])) next_url = url_for('.edit', path=path[1:])
path = path[0] path = path[0]
base_path, full_path, path = self._normalize_path(path) base_path, full_path, path = self._normalize_path(path)
...@@ -562,6 +561,7 @@ class FileAdmin(BaseView, ActionsMixin): ...@@ -562,6 +561,7 @@ class FileAdmin(BaseView, ActionsMixin):
form = EditForm() form = EditForm()
error = False error = False
if request.method == 'POST': if request.method == 'POST':
form.process(request.form, content='') form.process(request.form, content='')
if form.validate(): if form.validate():
...@@ -619,4 +619,4 @@ class FileAdmin(BaseView, ActionsMixin): ...@@ -619,4 +619,4 @@ class FileAdmin(BaseView, ActionsMixin):
@action('edit', lazy_gettext('Edit')) @action('edit', lazy_gettext('Edit'))
def action_edit(self, items): def action_edit(self, items):
return redirect(url_for('.edit', path=':'.join(items))) return redirect(url_for('.edit', path=items))
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment