maxmustermann/kibana
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

adding hasTimeField to as a vislib param to determine whether to add brushing

Browse source 
Fixes #5560
This commit is contained in:
Shelby Sturgis 2015-12-04 12:23:56 -08:00 committed by Joe Fleming
parent 5593f27442
commit f0289a077d
2 changed files with 6 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/ui/public/vislib/lib/dispatch.js
View file

@ -161,11 +161,10 @@ define(function (require) {
*/
Dispatch.prototype.allowBrushing = function () {
var xAxis = this.handler.xAxis;
// Don't allow brushing for time based charts from non-time-based indices
if (xAxis.ordered && xAxis.ordered.endzones === false) return false;
var hasTimeField = this.handler.vis._attr.hasTimeField;
return Boolean(xAxis.ordered && xAxis.xScale && _.isFunction(xAxis.xScale.invert));
return Boolean(xAxis.ordered && xAxis.xScale && _.isFunction(xAxis.xScale.invert) && hasTimeField);
};
/**

5
src/ui/public/vislib_vis_type/VislibRenderbot.js
View file

@ -31,7 +31,10 @@ module.exports = function VislibRenderbotFactory(Private) {
return _.assign(
{},
self.vis.type.params.defaults,
{ type: self.vis.type.name },
{
type: self.vis.type.name,
hasTimeField: self.vis.indexPattern.hasTimeField()
},
self.vis.params
);
};