Fix missing tooltip value in radar charts (#6209)

chartjs · Apr 18, 2019 · 89af7b1 · 89af7b1
1 parent faad023
commit 89af7b1
Show file tree

Hide file tree

Showing 2 changed files with 33 additions and 0 deletions.
diff --git a/src/controllers/controller.radar.js b/src/controllers/controller.radar.js
@@ -20,6 +20,19 @@ defaults._set('radar', {
 });
 
 module.exports = DatasetController.extend({
+	/**
+	 * @private
+	 */
+	_getValueScaleId: function() {
+		return this.chart.scale.id;
+	},
+
+	/**
+	 * @private
+	 */
+	_getIndexScaleId: function() {
+		return this.chart.scale.id;
+	},
 
 	datasetElementType: elements.Line,
 

diff --git a/test/specs/controller.radar.tests.js b/test/specs/controller.radar.tests.js
@@ -443,4 +443,24 @@ describe('Chart.controllers.radar', function() {
 		expect(meta0.data[0]._model.radius).toBe(10);
 		expect(meta1.data[0]._model.radius).toBe(20);
 	});
+
+	it('should return same id for index and value scale', function() {
+		var chart = window.acquireChart({
+			type: 'radar',
+			data: {
+				datasets: [{
+					data: [10, 15, 0, 4],
+					pointBorderWidth: 0
+				}],
+				labels: ['label1', 'label2', 'label3', 'label4']
+			},
+			options: {
+				scale: {id: 'test'}
+			}
+		});
+
+		var controller = chart.getDatasetMeta(0).controller;
+		expect(controller._getIndexScaleId()).toBe('test');
+		expect(controller._getValueScaleId()).toBe('test');
+	});
 });