Fixed trailing 0 on new revisions, Fixed tablesorter refresh.

jnoleau · Jun 10, 2014 · 04727d5 · 04727d5
1 parent 85fe4bc
commit 04727d5
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 9 deletions.
diff --git a/DBV.php b/DBV.php
@@ -221,7 +221,13 @@ public function saveRevisionFileAction()
     public function addRevisionFolderAction()
     {
     	$revisions = $this->_getAllRevisions();
-    	$revision = $_GET['revision'];
+    	$revision = intval($_GET['revision']);
+
+    	$revisionsDir = $this->_getRevisions();
+    	if(in_array($revision, $revisionsDir)){
+    		$this->_json(array('ok' => false, 'message' => __("Revision #{revision} already exists!", array('revision' => "<strong>$revision</strong>"))));
+    		return;
+    	}
 
     	$dir = DBV_REVISIONS_PATH . DS . $revision;
     	if (!@file_exists($dir)) {

diff --git a/templates/revisions.php b/templates/revisions.php
@@ -88,17 +88,24 @@
 			url: 'index.php?a=addRevisionFolder',
 			data: {'revision': revision},
 			success: function(response){
-				render_messages('success', '#revisions', response.message);
+				if(response.ok){
+					render_messages('success', '#revisions', response.message);
 
-				$('#body_revisions').prepend(response.html);
+					$('#body_revisions').prepend(response.html);
 
-				/// Remove the save buttons
-				$(response.revision).each(function(value){
-					value = value.split('/')[0];
-					$('tr[data-revision='+value+'] button[data-role="editor-save"]').remove();
-				});
+					/// Remove the save buttons
+					$(response.revision).each(function(value){
+						value = value.split('/')[0];
+						$('tr[data-revision='+value+'] button[data-role="editor-save"]').remove();
+					});
 
-				reloadCodeMirror();
+					reloadCodeMirror();
+
+					$("#table-sorter").trigger("update"); 
+				}
+				else{
+					render_messages('error', '#revisions', response.message);
+				}
 			}
 		});
 	});