confirm before destroying questions

elan-ev · Feb 7, 2013 · e77cfaf · e77cfaf
1 parent f13464a
commit e77cfaf
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 5 deletions.
diff --git a/app/views/questions/_show_controls.php b/app/views/questions/_show_controls.php
@@ -12,7 +12,10 @@
     <?= \STUDIP\LinkButton::create(_("Ändern"),  $controller->url_for('questions/edit', $id)) ?>
 
   <!-- DELETE -->
-    <form action="<?= $controller->url_for('questions/destroy', $id) ?>" method="POST">
+    <form class="questions-destroy"
+          action="<?= $controller->url_for('questions/destroy', $id) ?>"
+          method="POST"
+          data-confirm="<?= ("Wollen Sie die Frage wirklich löschen?") ?>">
       <?= \STUDIP\Button::create(_("Löschen")) ?>
     </form>
 

diff --git a/assets/js/views/questions_show.coffee b/assets/js/views/questions_show.coffee
@@ -7,9 +7,10 @@ define [
 
   class QuestionView extends Backbone.View
     events:
-      #"click .fullscreen": "showFS"
-      "click .appeal.start button":   "startQuestion"
-      "click a.qr": "showQRCode"
+      # "click .fullscreen":             "showFS"
+      "click .appeal.start button":    "startQuestion"
+      "click a.qr":                    "showQRCode"
+      "submit form.questions-destroy": "confirmDestroy"
 
     ###
     showFS: ->
@@ -49,3 +50,7 @@ define [
       # do not show code, handle this on your own
       event.preventDefault()
       @$(".question").toggleClass "qr-visible"
+
+    confirmDestroy: (event) ->
+      unless confirm $(event.target).data "confirm"
+        event.preventDefault()
diff --git a/assets/js/views/questions_show.js b/assets/js/views/questions_show.js