Skip to content

Commit b2cdacc

Browse files
committed
Remove _update and _config methods. Move default options to defaults scope
1 parent 0b86cf9 commit b2cdacc

File tree

4 files changed

+9
-43
lines changed

4 files changed

+9
-43
lines changed

src/controllers/controller.line.js

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -26,12 +26,10 @@ defaults._set('line', {
2626
}
2727
});
2828

29-
defaults._set('global', {
30-
datasets: {
31-
line: {
32-
showLine: true,
33-
spanGaps: false
34-
}
29+
defaults._set('datasets', {
30+
line: {
31+
showLine: true,
32+
spanGaps: false
3533
}
3634
});
3735

src/controllers/controller.scatter.js

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -33,11 +33,9 @@ defaults._set('scatter', {
3333
}
3434
});
3535

36-
defaults._set('global', {
37-
datasets: {
38-
scatter: {
39-
showLine: false
40-
}
36+
defaults._set('datasets', {
37+
scatter: {
38+
showLine: false
4139
}
4240
});
4341

src/core/core.controller.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -503,7 +503,7 @@ helpers.extend(Chart.prototype, /** @lends Chart */ {
503503
return;
504504
}
505505

506-
meta.controller._update();
506+
meta.controller.update();
507507

508508
plugins.notify(me, 'afterDatasetUpdate', [args]);
509509
},

src/core/core.datasetController.js

Lines changed: 1 addition & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -235,36 +235,6 @@ helpers.extend(DatasetController.prototype, {
235235
me.resyncElements();
236236
},
237237

238-
/**
239-
* Private functionality to be called before every update
240-
* @private
241-
*/
242-
_update: function() {
243-
var me = this;
244-
me._config = me._configure();
245-
me.update.apply(me, arguments);
246-
},
247-
248-
/**
249-
* Returns the merged user-supplied and default dataset-level options
250-
* @private
251-
*/
252-
_configure: function() {
253-
var me = this;
254-
var dataset = me.getDataset();
255-
var datasetOptions = {};
256-
var datasetKeys = Object.keys(dataset);
257-
var nonOptionKeys = ['data', 'type', '_meta'];
258-
var i, ilen, key;
259-
for (i = 0, ilen = datasetKeys.length; i < ilen; i++) {
260-
key = datasetKeys[i];
261-
if (nonOptionKeys.indexOf(key) < 0) {
262-
datasetOptions[key] = helpers.clone(dataset[key]);
263-
}
264-
}
265-
return helpers.merge({}, [datasetOptions, me._defaults()]);
266-
},
267-
268238
/**
269239
* Returns the default dataset-level options
270240
* @private
@@ -273,7 +243,7 @@ helpers.extend(DatasetController.prototype, {
273243
var me = this;
274244
var dataset = me.getDataset();
275245
var type = helpers.valueOrDefault(dataset.type, me.chart.config.type);
276-
return defaults.global.datasets[type];
246+
return defaults.datasets[type];
277247
},
278248

279249
update: helpers.noop,

0 commit comments

Comments
 (0)