made delete a POST

README.markdown

@@ -24,7 +24,6 @@ TODO
 ====
 
 * Javascript sorting of tables
-* Make delete a POST request
 * Better error handling
 * Move or Copy key to different server
 * Importing JSON

delete.php

@@ -1,7 +1,12 @@
 <?php
 
-require_once 'common.inc.php';
 
+if (!isset($_POST['post'])) {
+  die('Javascript needs to be enabled for you to delete keys.');
+}
+
+
+require_once 'common.inc.php';
 
 
 if (isset($_GET['key'])) {
@@ -41,18 +46,7 @@
   }
 
 
-
-  // Refresh the top so the key tree is updated.
-  require 'header.inc.php';
-
-  ?>
-  <script>
-  top.location.href = top.location.pathname+'?view&s=<?php echo $server['id']?>&key=<?php echo urlencode($_GET['key'])?>';
-  </script>
-  <?php
-
-  require 'footer.inc.php';
-  die;
+  die('?view&s='.$server['id'].'&key='.urlencode($_GET['key']));
 }
 
 ?>

js/frame.js

@@ -12,15 +12,18 @@ $(function() {
   }).change();
 
 
-  $('.delkey').click(function(e) {
-    if (!confirm('Are you sure you want to delete this key and all it\'s values?')) {
-      e.preventDefault();
-    }
-  });
+  $('.delkey, .delval').click(function(e) {
+    e.preventDefault();
 
-  $('.delval').click(function(e) {
-    if (!confirm('Are you sure you want to delete this value?')) {
-      e.preventDefault();
+    if (confirm($(this).hasClass('delkey') ? 'Are you sure you want to delete this key and all it\'s values?' : 'Are you sure you want to delete this value?')) {
+      $.ajax({
+        type: "POST",
+        url: this.href,
+        data: 'post=1',
+        success: function(url) {
+          top.location.href = top.location.pathname+url;
+        }
+      });
     }
   });
 });