Fix horizontalBar deprecation warnings (#6603)

Fix horizontalBar deprecation warnings
chartjs · Oct 26, 2019 · 21da5be · 21da5be
1 parent ea100d4
commit 21da5be
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 5 deletions.
diff --git a/src/controllers/controller.horizontalBar.js b/src/controllers/controller.horizontalBar.js
@@ -18,8 +18,6 @@ defaults._set('horizontalBar', {
 		yAxes: [{
 			type: 'category',
 			position: 'left',
-			categoryPercentage: 0.8,
-			barPercentage: 0.9,
 			offset: true,
 			gridLines: {
 				offsetGridLines: true
@@ -39,6 +37,15 @@ defaults._set('horizontalBar', {
 	}
 });
 
+defaults._set('global', {
+	datasets: {
+		horizontalBar: {
+			categoryPercentage: 0.8,
+			barPercentage: 0.9
+		}
+	}
+});
+
 module.exports = BarController.extend({
 	/**
 	 * @private
@@ -54,4 +61,3 @@ module.exports = BarController.extend({
 		return this.getMeta().yAxisID;
 	}
 });
-
diff --git a/test/specs/controller.bar.tests.js b/test/specs/controller.bar.tests.js
@@ -1593,8 +1593,9 @@ describe('Chart.controllers.bar', function() {
 			var meta = chart.getDatasetMeta(0);
 			var yScale = chart.scales[meta.yAxisID];
 
-			var categoryPercentage = yScale.options.categoryPercentage;
-			var barPercentage = yScale.options.barPercentage;
+			var config = meta.controller._config;
+			var categoryPercentage = config.categoryPercentage;
+			var barPercentage = config.barPercentage;
 			var stacked = yScale.options.stacked;
 
 			var totalBarHeight = 0;