Merge pull request #2186 from nnnick/fix/2115

Better handling when data.datasets does not exist
This commit is contained in:
Evert Timberg 2016-03-29 20:13:42 -04:00
commit 37da19df78
2 changed files with 9 additions and 7 deletions

View File

@ -347,11 +347,13 @@ module.exports = function(Chart) {
var elementsArray = [];
var found = (function() {
for (var i = 0; i < this.data.datasets.length; i++) {
if (helpers.isDatasetVisible(this.data.datasets[i])) {
for (var j = 0; j < this.data.datasets[i].metaData.length; j++) {
if (this.data.datasets[i].metaData[j].inRange(eventPosition.x, eventPosition.y)) {
return this.data.datasets[i].metaData[j];
if (this.data.datasets) {
for (var i = 0; i < this.data.datasets.length; i++) {
if (helpers.isDatasetVisible(this.data.datasets[i])) {
for (var j = 0; j < this.data.datasets[i].metaData.length; j++) {
if (this.data.datasets[i].metaData[j].inRange(eventPosition.x, eventPosition.y)) {
return this.data.datasets[i].metaData[j];
}
}
}
}

View File

@ -35,7 +35,7 @@ module.exports = function(Chart) {
// lineJoin :
// lineWidth :
generateLabels: function(data) {
return data.datasets.map(function(dataset, i) {
return helpers.isArray(data.datasets) ? data.datasets.map(function(dataset, i) {
return {
text: dataset.label,
fillStyle: dataset.backgroundColor,
@ -50,7 +50,7 @@ module.exports = function(Chart) {
// Below is extra data used for toggling the datasets
datasetIndex: i
};
}, this);
}, this) : [];
}
}
};