Merge pull request #1487 from getredash/v1fixes

Fix #1432: delete visualization sends full visualization body instead…
getredash · Dec 23, 2016 · 9e5944d · 9e5944d
2 parents ff6bb2c + e4f1325
commit 9e5944d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/client/app/components/dynamic-table/index.js b/client/app/components/dynamic-table/index.js
@@ -43,9 +43,7 @@ function DynamicTable($sanitize) {
     }
   };
 
-  this.sanitize = (value) => {
-    return $sanitize(value);
-  }
+  this.sanitize = value => $sanitize(value);
 
   this.sortIcon = (column) => {
     if (column !== this.orderByField) {

diff --git a/client/app/pages/queries/source-view.js b/client/app/pages/queries/source-view.js
@@ -91,7 +91,7 @@ function QuerySourceCtrl(Events, toastr, $controller, $scope, $location, $http,
     AlertDialog.open(title, message, confirm).then(() => {
       Events.record('delete', 'visualization', vis.id);
 
-      Visualization.delete(vis, () => {
+      Visualization.delete({ id: vis.id }, () => {
         if ($scope.selectedTab === vis.id) {
           $scope.selectedTab = DEFAULT_TAB;
           $location.hash($scope.selectedTab);