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')

http://farol.wald.intevation.org