From b2cdacc32ed9b5096df5e602b8ae7586b9e1fa28 Mon Sep 17 00:00:00 2001
From: Ben McCann <322311+benmccann@users.noreply.github.com>
Date: Tue, 22 Jan 2019 07:39:17 -0800
Subject: [PATCH] Remove _update and _config methods. Move default options to
 defaults scope

---
 src/controllers/controller.line.js    | 10 ++++-----
 src/controllers/controller.scatter.js |  8 +++----
 src/core/core.controller.js           |  2 +-
 src/core/core.datasetController.js    | 32 +--------------------------
 4 files changed, 9 insertions(+), 43 deletions(-)

diff --git a/src/controllers/controller.line.js b/src/controllers/controller.line.js
index 7151a1aa9a5..cd0f1fd4d88 100644
--- a/src/controllers/controller.line.js
+++ b/src/controllers/controller.line.js
@@ -26,12 +26,10 @@ defaults._set('line', {
 	}
 });
 
-defaults._set('global', {
-	datasets: {
-		line: {
-			showLine: true,
-			spanGaps: false
-		}
+defaults._set('datasets', {
+	line: {
+		showLine: true,
+		spanGaps: false
 	}
 });
 
diff --git a/src/controllers/controller.scatter.js b/src/controllers/controller.scatter.js
index 177e563203f..7cdc088490a 100644
--- a/src/controllers/controller.scatter.js
+++ b/src/controllers/controller.scatter.js
@@ -33,11 +33,9 @@ defaults._set('scatter', {
 	}
 });
 
-defaults._set('global', {
-	datasets: {
-		scatter: {
-			showLine: false
-		}
+defaults._set('datasets', {
+	scatter: {
+		showLine: false
 	}
 });
 
diff --git a/src/core/core.controller.js b/src/core/core.controller.js
index 0e110fb14c1..b0175e8a67f 100644
--- a/src/core/core.controller.js
+++ b/src/core/core.controller.js
@@ -503,7 +503,7 @@ helpers.extend(Chart.prototype, /** @lends Chart */ {
 			return;
 		}
 
-		meta.controller._update();
+		meta.controller.update();
 
 		plugins.notify(me, 'afterDatasetUpdate', [args]);
 	},
diff --git a/src/core/core.datasetController.js b/src/core/core.datasetController.js
index e823ae17f3a..3f9b8d5464e 100644
--- a/src/core/core.datasetController.js
+++ b/src/core/core.datasetController.js
@@ -235,36 +235,6 @@ helpers.extend(DatasetController.prototype, {
 		me.resyncElements();
 	},
 
-	/**
-	 * Private functionality to be called before every update
-	 * @private
-	 */
-	_update: function() {
-		var me = this;
-		me._config = me._configure();
-		me.update.apply(me, arguments);
-	},
-
-	/**
-	 * Returns the merged user-supplied and default dataset-level options
-	 * @private
-	 */
-	_configure: function() {
-		var me = this;
-		var dataset = me.getDataset();
-		var datasetOptions = {};
-		var datasetKeys = Object.keys(dataset);
-		var nonOptionKeys = ['data', 'type', '_meta'];
-		var i, ilen, key;
-		for (i = 0, ilen = datasetKeys.length; i < ilen; i++) {
-			key = datasetKeys[i];
-			if (nonOptionKeys.indexOf(key) < 0) {
-				datasetOptions[key] = helpers.clone(dataset[key]);
-			}
-		}
-		return helpers.merge({}, [datasetOptions, me._defaults()]);
-	},
-
 	/**
 	 * Returns the default dataset-level options
 	 * @private
@@ -273,7 +243,7 @@ helpers.extend(DatasetController.prototype, {
 		var me = this;
 		var dataset = me.getDataset();
 		var type = helpers.valueOrDefault(dataset.type, me.chart.config.type);
-		return defaults.global.datasets[type];
+		return defaults.datasets[type];
 	},
 
 	update: helpers.noop,