Mercurial > farol > farol
diff farol/document.py @ 73:88adf067cae1
Fix delete urls
author | Benoît Allard <benoit.allard@greenbone.net> |
---|---|
date | Thu, 09 Oct 2014 11:14:50 +0200 |
parents | f9aa7b1e697e |
children | 80cd8f65e72b |
line wrap: on
line diff
--- a/farol/document.py Thu Oct 09 10:31:06 2014 +0200 +++ b/farol/document.py Thu Oct 09 11:14:50 2014 +0200 @@ -142,10 +142,9 @@ tracking._currentDate = date return redirect(url_for('.view')) -@document.route('/revision/del', methods=['POST']) +@document.route('/revision/<int:index>/del', methods=['POST']) @document_required -def del_revision(): - index = int(request.form['index']) +def del_revision(index): history = get_current()._tracking._history if not (0 <= index < len(history)): flash('Revision not found', 'danger') @@ -213,17 +212,15 @@ get_current().addNote(note) return redirect(url_for('.view')) -@document.route('/note/del', methods=['POST']) +@document.route('/note/<int:ordinal>/del', methods=['POST']) @document_required -def del_note(): - ordinal = int(request.form['ordinal']) +def del_note(ordinal): cvrf = get_current() - notes = cvrf._notes note = cvrf.getNote(ordinal) if note is None: flash('Note not found', 'danger') abort(404) - notes.remove(note) + cvrf._notes.remove(note) return redirect(url_for('.view')) @document.route('/reference/<int:index>/edit', methods=['GET', 'POST']) @@ -250,10 +247,9 @@ get_current().addReference(ref) return redirect(url_for('.view')) -@document.route('/reference/del', methods=['POST']) +@document.route('/reference/<int:index>/del', methods=['POST']) @document_required -def del_reference(): - index = int(request.form['index']) +def del_reference(index): refs = get_current()._references if not (0 <= index < len(refs)): flash('Reference not found', 'danger')