Commit 71cf9dfa by Rashid Khan

Revert "added check for 404 response from the index_save call, assumes that…

Revert "added check for 404 response from the index_save call, assumes that auto_create_index is disabled and attempts to create the index before saving a second time"

This reverts commit 556925f8.
parent 0ab6e73f
...@@ -800,10 +800,6 @@ angular.module('kibana.services', []) ...@@ -800,10 +800,6 @@ angular.module('kibana.services', [])
); );
}; };
this.elasticsearch_create_index = function () {
return ejs.client.post('/'+config.kibana_index);
};
this.elasticsearch_delete = function(id) { this.elasticsearch_delete = function(id) {
return ejs.Document(config.kibana_index,'dashboard',id).doDelete( return ejs.Document(config.kibana_index,'dashboard',id).doDelete(
// Success // Success
......
...@@ -17,7 +17,7 @@ ...@@ -17,7 +17,7 @@
* hide_control :: Upon save, hide this panel * hide_control :: Upon save, hide this panel
* elasticsearch_size :: show this many dashboards under the ES section in the load drop down * elasticsearch_size :: show this many dashboards under the ES section in the load drop down
* temp :: Allow saving of temp dashboards * temp :: Allow saving of temp dashboards
* ttl :: Enable setting ttl. * ttl :: Enable setting ttl.
* temp_ttl :: How long should temp dashboards persist * temp_ttl :: How long should temp dashboards persist
*/ */
...@@ -61,10 +61,6 @@ angular.module('kibana.dashcontrol', []) ...@@ -61,10 +61,6 @@ angular.module('kibana.dashcontrol', [])
services: {} services: {}
}; };
function notice(type, title, message) {
alertSrv.set(title, message, type, 5000);
}
$scope.init = function() { $scope.init = function() {
$scope.gist_pattern = /(^\d{5,}$)|(^[a-z0-9]{10,}$)|(gist.github.com(\/*.*)\/[a-z0-9]{5,}\/*$)/; $scope.gist_pattern = /(^\d{5,}$)|(^[a-z0-9]{10,}$)|(gist.github.com(\/*.*)\/[a-z0-9]{5,}\/*$)/;
$scope.gist = {}; $scope.gist = {};
...@@ -73,17 +69,17 @@ angular.module('kibana.dashcontrol', []) ...@@ -73,17 +69,17 @@ angular.module('kibana.dashcontrol', [])
$scope.set_default = function() { $scope.set_default = function() {
if(dashboard.set_default()) { if(dashboard.set_default()) {
notice('success', 'Local Default Set', dashboard.current.title+' has been set as your local default'); alertSrv.set('Local Default Set',dashboard.current.title+' has been set as your local default','success',5000);
} else { } else {
notice('error', 'Incompatible Browser', 'Sorry, your browser is too old for this feature'); alertSrv.set('Incompatible Browser','Sorry, your browser is too old for this feature','error',5000);
} }
}; };
$scope.purge_default = function() { $scope.purge_default = function() {
if(dashboard.purge_default()) { if(dashboard.purge_default()) {
notice('success', 'Local Default Clear', 'Your local default dashboard has been cleared'); alertSrv.set('Local Default Clear','Your local default dashboard has been cleared','success',5000);
} else { } else {
notice('error', 'Incompatible Browser', 'Sorry, your browser is too old for this feature'); alertSrv.set('Incompatible Browser','Sorry, your browser is too old for this feature','error',5000);
} }
}; };
...@@ -95,30 +91,13 @@ angular.module('kibana.dashcontrol', []) ...@@ -95,30 +91,13 @@ angular.module('kibana.dashcontrol', [])
).then( ).then(
function(result) { function(result) {
if(!_.isUndefined(result._id)) { if(!_.isUndefined(result._id)) {
notice( alertSrv.set('Dashboard Saved','This dashboard has been saved to Elasticsearch as "' +
'success', result._id + '"','success',5000);
'Dashboard Saved',
'This dashboard has been saved to Elasticsearch as "'+result._id + '"'
);
if(type === 'temp') { if(type === 'temp') {
$scope.share = dashboard.share_link(dashboard.current.title, 'temp', result._id); $scope.share = dashboard.share_link(dashboard.current.title,'temp',result._id);
} }
} else { } else {
if (result.status === 404) { alertSrv.set('Save failed','Dashboard could not be saved to Elasticsearch','error',5000);
// auto create must be disabled and the index doesn't exist, create it!
return dashboard.elasticsearch_create_index().then(function () {
return $scope.elasticsearch_save(type, ttl);
}, function () {
notice(
'error',
'Save failed',
'Dashboard could not be saved because the "'+config.kibana_index+'" '+
'index does not exist and could not be created.'
);
});
} else {
notice('error', 'Save failed', 'Dashboard could not be saved to Elasticsearch');
}
} }
}); });
}; };
...@@ -128,15 +107,15 @@ angular.module('kibana.dashcontrol', []) ...@@ -128,15 +107,15 @@ angular.module('kibana.dashcontrol', [])
function(result) { function(result) {
if(!_.isUndefined(result)) { if(!_.isUndefined(result)) {
if(result.found) { if(result.found) {
notice('success', 'Dashboard Deleted', id+' has been deleted'); alertSrv.set('Dashboard Deleted',id+' has been deleted','success',5000);
// Find the deleted dashboard in the cached list and remove it // Find the deleted dashboard in the cached list and remove it
var toDelete = _.where($scope.elasticsearch.dashboards,{_id:id})[0]; var toDelete = _.where($scope.elasticsearch.dashboards,{_id:id})[0];
$scope.elasticsearch.dashboards = _.without($scope.elasticsearch.dashboards,toDelete); $scope.elasticsearch.dashboards = _.without($scope.elasticsearch.dashboards,toDelete);
} else { } else {
notice('warning', 'Dashboard Not Found', 'Could not find '+id+' in Elasticsearch'); alertSrv.set('Dashboard Not Found','Could not find '+id+' in Elasticsearch','warning',5000);
} }
} else { } else {
notice('error', 'Dashboard Not Deleted', 'An error occurred deleting the dashboard'); alertSrv.set('Dashboard Not Deleted','An error occurred deleting the dashboard','error',5000);
} }
} }
); );
...@@ -158,14 +137,10 @@ angular.module('kibana.dashcontrol', []) ...@@ -158,14 +137,10 @@ angular.module('kibana.dashcontrol', [])
function(link) { function(link) {
if(!_.isUndefined(link)) { if(!_.isUndefined(link)) {
$scope.gist.last = link; $scope.gist.last = link;
notice( alertSrv.set('Gist saved','You will be able to access your exported dashboard file at '+
'success', '<a href="'+link+'">'+link+'</a> in a moment','success');
'Gist saved',
'You will be able to access your exported dashboard file at '+
'<a href="'+link+'">'+link+'</a> in a moment'
);
} else { } else {
notice('error', 'Save failed', 'Gist could not be saved'); alertSrv.set('Save failed','Gist could not be saved','error',5000);
} }
}); });
}; };
...@@ -176,7 +151,7 @@ angular.module('kibana.dashcontrol', []) ...@@ -176,7 +151,7 @@ angular.module('kibana.dashcontrol', [])
if(files && files.length > 0) { if(files && files.length > 0) {
$scope.gist.files = files; $scope.gist.files = files;
} else { } else {
notice('error', 'Gist Failed', 'Could not retrieve dashboard list from gist',5000); alertSrv.set('Gist Failed','Could not retrieve dashboard list from gist','error',5000);
} }
}); });
}; };
...@@ -208,7 +183,7 @@ angular.module('kibana.dashcontrol', []) ...@@ -208,7 +183,7 @@ angular.module('kibana.dashcontrol', [])
// Something // Something
document.getElementById('dashupload').addEventListener('change', file_selected, false); document.getElementById('dashupload').addEventListener('change', file_selected, false);
} else { } else {
alertSrv.set('Oops', 'Sorry, the HTML5 File APIs are not fully supported in this browser.', 'error', 5000); alertSrv.set('Oops','Sorry, the HTML5 File APIs are not fully supported in this browser.','error');
} }
} }
}; };
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment