]> git.ipfire.org Git - thirdparty/Chart.js.git/commitdiff
Prefix private controller methods with underscore (#7081)
authorBen McCann <322311+benmccann@users.noreply.github.com>
Mon, 10 Feb 2020 22:57:22 +0000 (14:57 -0800)
committerGitHub <noreply@github.com>
Mon, 10 Feb 2020 22:57:22 +0000 (17:57 -0500)
Prefix private controller methods with underscore

docs/getting-started/v3-migration.md
src/controllers/controller.bar.js
src/core/core.animator.js
src/core/core.controller.js
src/core/core.datasetController.js
src/core/core.plugins.js
test/fixtures/core.tooltip/opacity.js
test/specs/controller.bar.tests.js
test/specs/core.datasetController.tests.js
test/utils.js

index d00df76753da365127d417049c4a23159dd664a1..70bc75e2dce278476592a7946796e90afbea3502 100644 (file)
@@ -176,9 +176,33 @@ Animation system was completely rewritten in Chart.js v3. Each property can now
 
 #### Renamed private APIs
 
+* `BarController.calculateBarIndexPixels` was renamed to `BarController._calculateBarIndexPixels`
+* `BarController.calculateBarValuePixels` was renamed to `BarController._calculateBarValuePixels`
+* `BarController.getStackCount` was renamed to `BarController._getStackCount`
+* `BarController.getStackIndex` was renamed to `BarController._getStackIndex`
+* `BarController.getRuler` was renamed to `BarController._getRuler`
+* `Chart.destroyDatasetMeta` was renamed to `Chart._destroyDatasetMeta`
+* `Chart.drawDataset` was renamed to `Chart._drawDataset`
+* `Chart.drawDatasets` was renamed to `Chart._drawDatasets`
+* `Chart.eventHandler` was renamed to `Chart._eventHandler`
+* `Chart.handleEvent` was renamed to `Chart._handleEvent`
+* `Chart.initialize` was renamed to `Chart._initialize`
+* `Chart.resetElements` was renamed to `Chart._resetElements`
+* `Chart.unbindEvents` was renamed to `Chart._unbindEvents`
+* `Chart.updateDataset` was renamed to `Chart._updateDataset`
+* `Chart.updateDatasets` was renamed to `Chart._updateDatasets`
+* `Chart.updateLayout` was renamed to `Chart._updateLayout`
+* `DatasetController.destroy` was renamed to `DatasetController._destroy`
+* `DatasetController.insertElements` was renamed to `DatasetController._insertElements`
+* `DatasetController.onDataPop` was renamed to `DatasetController._onDataPop`
+* `DatasetController.onDataPush` was renamed to `DatasetController._onDataPush`
+* `DatasetController.onDataShift` was renamed to `DatasetController._onDataShift`
+* `DatasetController.onDataSplice` was renamed to `DatasetController._onDataSplice`
+* `DatasetController.onDataUnshift` was renamed to `DatasetController._onDataUnshift`
+* `DatasetController.removeElements` was renamed to `DatasetController._removeElements`
+* `DatasetController.resyncElements` was renamed to `DatasetController._resyncElements`
 * `helpers._alignPixel` was renamed to `helpers.canvas._alignPixel`
 * `helpers._decimalPlaces` was renamed to `helpers.math._decimalPlaces`
-* `chart.initialize` was renamed to `chart._initialize` (labeled as private but not named as such)
 
 ### Changed
 
index 9930aec5bb9effe0dcc22f550b962b70cd4d1520..a310902ec471df85fddd0e6e7b468332e49f38cd 100644 (file)
@@ -274,7 +274,7 @@ class BarController extends DatasetController {
                const vscale = me._cachedMeta.vScale;
                const base = vscale.getBasePixel();
                const horizontal = vscale.isHorizontal();
-               const ruler = me.getRuler();
+               const ruler = me._getRuler();
                const firstOpts = me._resolveDataElementOptions(start, mode);
                const sharedOptions = me._getSharedOptions(mode, rectangles[start], firstOpts);
                const includeOptions = me._includeOptions(mode, sharedOptions);
@@ -284,8 +284,8 @@ class BarController extends DatasetController {
                for (i = 0; i < rectangles.length; i++) {
                        const index = start + i;
                        const options = me._resolveDataElementOptions(index, mode);
-                       const vpixels = me.calculateBarValuePixels(index, options);
-                       const ipixels = me.calculateBarIndexPixels(index, ruler, options);
+                       const vpixels = me._calculateBarValuePixels(index, options);
+                       const ipixels = me._calculateBarIndexPixels(index, ruler, options);
 
                        const properties = {
                                horizontal,
@@ -357,7 +357,7 @@ class BarController extends DatasetController {
         * Returns the effective number of stacks based on groups and bar visibility.
         * @private
         */
-       getStackCount() {
+       _getStackCount() {
                return this._getStacks().length;
        }
 
@@ -368,7 +368,7 @@ class BarController extends DatasetController {
         * @returns {number} The stack index
         * @private
         */
-       getStackIndex(datasetIndex, name) {
+       _getStackIndex(datasetIndex, name) {
                var stacks = this._getStacks(datasetIndex);
                var index = (name !== undefined)
                        ? stacks.indexOf(name)
@@ -382,7 +382,7 @@ class BarController extends DatasetController {
        /**
         * @private
         */
-       getRuler() {
+       _getRuler() {
                const me = this;
                const meta = me._cachedMeta;
                const iScale = meta.iScale;
@@ -397,7 +397,7 @@ class BarController extends DatasetController {
                        pixels,
                        start: iScale._startPixel,
                        end: iScale._endPixel,
-                       stackCount: me.getStackCount(),
+                       stackCount: me._getStackCount(),
                        scale: iScale
                };
        }
@@ -406,7 +406,7 @@ class BarController extends DatasetController {
         * Note: pixel values are not clamped to the scale area.
         * @private
         */
-       calculateBarValuePixels(index, options) {
+       _calculateBarValuePixels(index, options) {
                const me = this;
                const meta = me._cachedMeta;
                const vScale = meta.vScale;
@@ -460,13 +460,13 @@ class BarController extends DatasetController {
        /**
         * @private
         */
-       calculateBarIndexPixels(index, ruler, options) {
+       _calculateBarIndexPixels(index, ruler, options) {
                var me = this;
                var range = options.barThickness === 'flex'
                        ? computeFlexCategoryTraits(index, ruler, options)
                        : computeFitCategoryTraits(index, ruler, options);
 
-               var stackIndex = me.getStackIndex(me.index, me._cachedMeta.stack);
+               var stackIndex = me._getStackIndex(me.index, me._cachedMeta.stack);
                var center = range.start + (range.chunk * stackIndex) + (range.chunk / 2);
                var size = Math.min(
                        valueOrDefault(options.maxBarThickness, Infinity),
index 3fe775ed9085bc1ad1b07985c4b6dd17de423488..234f0f909e7caa83d0efb4b818e5ac8aeea9bcc9 100644 (file)
@@ -119,6 +119,9 @@ class Animator {
                }
        }
 
+       /**
+        * @private
+        */
        _getAnims(chart) {
                const charts = this._charts;
                let anims = charts.get(chart);
index a0c8a05986e0fd549cbd32f099ce732564dc362f..3b2efafbe5bb2906043be3f302d1f3d15559060d 100644 (file)
@@ -208,6 +208,7 @@ class Chart {
                this._updating = false;
                this.scales = {};
                this.scale = undefined;
+               this.$plugins = undefined;
 
                // Add the chart instance to the global namespace
                Chart.instances[me.id] = me;
@@ -454,7 +455,7 @@ class Chart {
 
                if (numMeta > numData) {
                        for (let i = numData; i < numMeta; ++i) {
-                               me.destroyDatasetMeta(i);
+                               me._destroyDatasetMeta(i);
                        }
                        metasets.splice(numData, numMeta - numData);
                }
@@ -473,7 +474,7 @@ class Chart {
                        const type = dataset.type || me.config.type;
 
                        if (meta.type && meta.type !== type) {
-                               me.destroyDatasetMeta(i);
+                               me._destroyDatasetMeta(i);
                                meta = me.getDatasetMeta(i);
                        }
                        meta.type = type;
@@ -504,7 +505,7 @@ class Chart {
         * Reset the elements of all datasets
         * @private
         */
-       resetElements() {
+       _resetElements() {
                const me = this;
                helpers.each(me.data.datasets, function(dataset, datasetIndex) {
                        me.getDatasetMeta(datasetIndex).controller.reset();
@@ -515,7 +516,7 @@ class Chart {
        * Resets the chart back to its state before the initial animation
        */
        reset() {
-               this.resetElements();
+               this._resetElements();
                plugins.notify(this, 'reset');
        }
 
@@ -543,7 +544,7 @@ class Chart {
                        me.getDatasetMeta(i).controller.buildOrUpdateElements();
                }
 
-               me.updateLayout();
+               me._updateLayout();
 
                // Can only reset the new controllers after the scales have been updated
                if (me.options.animation) {
@@ -552,7 +553,7 @@ class Chart {
                        });
                }
 
-               me.updateDatasets(mode);
+               me._updateDatasets(mode);
 
                // Do this before render so that any plugins that need final scale updates can use it
                plugins.notify(me, 'afterUpdate');
@@ -561,7 +562,7 @@ class Chart {
 
                // Replay last event from before update
                if (me._lastEvent) {
-                       me.eventHandler(me._lastEvent);
+                       me._eventHandler(me._lastEvent);
                }
 
                me.render();
@@ -574,7 +575,7 @@ class Chart {
         * hook, in which case, plugins will not be called on `afterLayout`.
         * @private
         */
-       updateLayout() {
+       _updateLayout() {
                const me = this;
 
                if (plugins.notify(me, 'beforeLayout') === false) {
@@ -605,7 +606,7 @@ class Chart {
         * hook, in which case, plugins will not be called on `afterDatasetsUpdate`.
         * @private
         */
-       updateDatasets(mode) {
+       _updateDatasets(mode) {
                const me = this;
                const isFunction = typeof mode === 'function';
 
@@ -614,7 +615,7 @@ class Chart {
                }
 
                for (let i = 0, ilen = me.data.datasets.length; i < ilen; ++i) {
-                       me.updateDataset(i, isFunction ? mode({datasetIndex: i}) : mode);
+                       me._updateDataset(i, isFunction ? mode({datasetIndex: i}) : mode);
                }
 
                plugins.notify(me, 'afterDatasetsUpdate');
@@ -625,7 +626,7 @@ class Chart {
         * hook, in which case, plugins will not be called on `afterDatasetUpdate`.
         * @private
         */
-       updateDataset(index, mode) {
+       _updateDataset(index, mode) {
                const me = this;
                const meta = me.getDatasetMeta(index);
                const args = {meta, index, mode};
@@ -682,7 +683,7 @@ class Chart {
                        layers[i].draw(me.chartArea);
                }
 
-               me.drawDatasets();
+               me._drawDatasets();
 
                // Rest of layers
                for (; i < layers.length; ++i) {
@@ -723,7 +724,7 @@ class Chart {
         * hook, in which case, plugins will not be called on `afterDatasetsDraw`.
         * @private
         */
-       drawDatasets() {
+       _drawDatasets() {
                const me = this;
                let metasets, i;
 
@@ -733,7 +734,7 @@ class Chart {
 
                metasets = me._getSortedVisibleDatasetMetas();
                for (i = metasets.length - 1; i >= 0; --i) {
-                       me.drawDataset(metasets[i]);
+                       me._drawDataset(metasets[i]);
                }
 
                plugins.notify(me, 'afterDatasetsDraw');
@@ -744,7 +745,7 @@ class Chart {
         * hook, in which case, plugins will not be called on `afterDatasetDraw`.
         * @private
         */
-       drawDataset(meta) {
+       _drawDataset(meta) {
                const me = this;
                const ctx = me.ctx;
                const clip = meta._clip;
@@ -879,12 +880,12 @@ class Chart {
        /**
         * @private
         */
-       destroyDatasetMeta(datasetIndex) {
+       _destroyDatasetMeta(datasetIndex) {
                const me = this;
                const meta = me._metasets && me._metasets[datasetIndex];
 
                if (meta) {
-                       meta.controller.destroy();
+                       meta.controller._destroy();
                        delete me._metasets[datasetIndex];
                }
        }
@@ -898,7 +899,7 @@ class Chart {
 
                // dataset controllers need to cleanup associated data
                for (i = 0, ilen = me.data.datasets.length; i < ilen; ++i) {
-                       me.destroyDatasetMeta(i);
+                       me._destroyDatasetMeta(i);
                }
 
                if (canvas) {
@@ -925,7 +926,7 @@ class Chart {
                const me = this;
                const listeners = me._listeners;
                let listener = function() {
-                       me.eventHandler.apply(me, arguments);
+                       me._eventHandler.apply(me, arguments);
                };
 
                helpers.each(me.options.events, function(type) {
@@ -1000,14 +1001,14 @@ class Chart {
        /**
         * @private
         */
-       eventHandler(e) {
+       _eventHandler(e) {
                const me = this;
 
                if (plugins.notify(me, 'beforeEvent', [e]) === false) {
                        return;
                }
 
-               me.handleEvent(e);
+               me._handleEvent(e);
 
                plugins.notify(me, 'afterEvent', [e]);
 
@@ -1018,11 +1019,11 @@ class Chart {
 
        /**
         * Handle an event
-        * @private
         * @param {IEvent} e the event to handle
         * @return {boolean} true if the chart needs to re-render
+        * @private
         */
-       handleEvent(e) {
+       _handleEvent(e) {
                const me = this;
                const options = me.options || {};
                const hoverOptions = options.hover;
index dc6e3683cffe134590a2ae1ec18b0aff23c36412..b516109f1699f179a042450ca9ee9a4b183d60c8 100644 (file)
@@ -10,7 +10,7 @@ const arrayEvents = ['push', 'pop', 'shift', 'splice', 'unshift'];
 /**
  * Hooks the array methods that add or remove values ('push', pop', 'shift', 'splice',
  * 'unshift') and notify the listener AFTER the array has been altered. Listeners are
- * called on the 'onData*' callbacks (e.g. onDataPush, etc.) with same arguments.
+ * called on the '_onData*' callbacks (e.g. _onDataPush, etc.) with same arguments.
  */
 function listenArrayEvents(array, listener) {
        if (array._chartjs) {
@@ -27,7 +27,7 @@ function listenArrayEvents(array, listener) {
        });
 
        arrayEvents.forEach(function(key) {
-               var method = 'onData' + key.charAt(0).toUpperCase() + key.slice(1);
+               var method = '_onData' + key.charAt(0).toUpperCase() + key.slice(1);
                var base = array[key];
 
                Object.defineProperty(array, key, {
@@ -322,7 +322,7 @@ class DatasetController {
        /**
         * @private
         */
-       destroy() {
+       _destroy() {
                if (this._data) {
                        unlistenArrayEvents(this._data, this);
                }
@@ -427,7 +427,7 @@ class DatasetController {
 
                // Re-sync meta data in case the user replaced the data array or if we missed
                // any updates and so make sure that we handle number of datapoints changing.
-               me.resyncElements(dataChanged || labelsChanged || scaleChanged || stackChanged);
+               me._resyncElements(dataChanged || labelsChanged || scaleChanged || stackChanged);
 
                // if stack changed, update stack values for the whole dataset
                if (stackChanged) {
@@ -1039,16 +1039,16 @@ class DatasetController {
        /**
         * @private
         */
-       resyncElements(changed) {
+       _resyncElements(changed) {
                const me = this;
                const meta = me._cachedMeta;
                const numMeta = meta.data.length;
                const numData = me._data.length;
 
                if (numData > numMeta) {
-                       me.insertElements(numMeta, numData - numMeta);
+                       me._insertElements(numMeta, numData - numMeta);
                        if (changed && numMeta) {
-                               // insertElements parses the new elements. The old ones might need parsing too.
+                               // _insertElements parses the new elements. The old ones might need parsing too.
                                me._parse(0, numMeta);
                        }
                } else if (numData < numMeta) {
@@ -1063,7 +1063,7 @@ class DatasetController {
        /**
         * @private
         */
-       insertElements(start, count) {
+       _insertElements(start, count) {
                const me = this;
                const elements = new Array(count);
                const meta = me._cachedMeta;
@@ -1088,7 +1088,7 @@ class DatasetController {
        /**
         * @private
         */
-       removeElements(start, count) {
+       _removeElements(start, count) {
                const me = this;
                if (me._parsing) {
                        me._cachedMeta._parsed.splice(start, count);
@@ -1100,38 +1100,38 @@ class DatasetController {
        /**
         * @private
         */
-       onDataPush() {
+       _onDataPush() {
                const count = arguments.length;
-               this.insertElements(this.getDataset().data.length - count, count);
+               this._insertElements(this.getDataset().data.length - count, count);
        }
 
        /**
         * @private
         */
-       onDataPop() {
-               this.removeElements(this._cachedMeta.data.length - 1, 1);
+       _onDataPop() {
+               this._removeElements(this._cachedMeta.data.length - 1, 1);
        }
 
        /**
         * @private
         */
-       onDataShift() {
-               this.removeElements(0, 1);
+       _onDataShift() {
+               this._removeElements(0, 1);
        }
 
        /**
         * @private
         */
-       onDataSplice(start, count) {
-               this.removeElements(start, count);
-               this.insertElements(start, arguments.length - 2);
+       _onDataSplice(start, count) {
+               this._removeElements(start, count);
+               this._insertElements(start, arguments.length - 2);
        }
 
        /**
         * @private
         */
-       onDataUnshift() {
-               this.insertElements(0, arguments.length);
+       _onDataUnshift() {
+               this._insertElements(0, arguments.length);
        }
 }
 
index d698e402142c274e7461132d0757a645b0fe2f25..0407396760b0f8cefa6a3e8ec77f5bfd0109fb31 100644 (file)
@@ -107,7 +107,7 @@ class PluginService {
         * @returns {boolean} false if any of the plugins return false, else returns true.
         */
        notify(chart, hook, args) {
-               var descriptors = this.descriptors(chart);
+               var descriptors = this._descriptors(chart);
                var ilen = descriptors.length;
                var i, descriptor, plugin, params, method;
 
@@ -129,10 +129,11 @@ class PluginService {
 
        /**
         * Returns descriptors of enabled plugins for the given chart.
+        * @param {Chart} chart
         * @returns {object[]} [{ plugin, options }]
         * @private
         */
-       descriptors(chart) {
+       _descriptors(chart) {
                var cache = chart.$plugins || (chart.$plugins = {});
                if (cache.id === this._cacheId) {
                        return cache.descriptors;
@@ -175,6 +176,7 @@ class PluginService {
         * Invalidates cache for the given chart: descriptors hold a reference on plugin option,
         * but in some cases, this reference can be changed by the user when updating options.
         * https://github.com/chartjs/Chart.js/issues/5111#issuecomment-355934167
+        * @param {Chart} chart
         * @private
         */
        _invalidate(chart) {
index 12652cc90df370adb6a92c299db6dd70387bfd3f..244205ed2468805f66148f5d6305ca2dae4a102e 100644 (file)
@@ -85,7 +85,7 @@ module.exports = {
                                                        clientX: rect.left + point.x,
                                                        clientY: rect.top + point.y
                                                };
-                                               chart.handleEvent(event);
+                                               chart._handleEvent(event);
                                                chart.tooltip.handleEvent(event);
                                                chart.tooltip.opacity = j / 10;
                                                chart.tooltip.draw(chart.ctx);
index 4ef8026d339ce8e19bc540084ddae656bc674828..7ea64528fbcf9d50d528805bd6f8b6b19d2c554f 100644 (file)
@@ -67,7 +67,7 @@ describe('Chart.controllers.bar', function() {
                        });
 
                        var meta = chart.getDatasetMeta(1);
-                       expect(meta.controller.getStackCount()).toBe(2);
+                       expect(meta.controller._getStackCount()).toBe(2);
                });
        });
 
@@ -90,7 +90,7 @@ describe('Chart.controllers.bar', function() {
                        });
 
                        var meta = chart.getDatasetMeta(1);
-                       expect(meta.controller.getStackCount()).toBe(4);
+                       expect(meta.controller._getStackCount()).toBe(4);
                });
        });
 
@@ -123,7 +123,7 @@ describe('Chart.controllers.bar', function() {
                        });
 
                        var meta = chart.getDatasetMeta(1);
-                       expect(meta.controller.getStackCount()).toBe(1);
+                       expect(meta.controller._getStackCount()).toBe(1);
                });
        });
 
@@ -156,7 +156,7 @@ describe('Chart.controllers.bar', function() {
                        });
 
                        var meta = chart.getDatasetMeta(1);
-                       expect(meta.controller.getStackCount()).toBe(4);
+                       expect(meta.controller._getStackCount()).toBe(4);
                });
        });
 
@@ -179,7 +179,7 @@ describe('Chart.controllers.bar', function() {
                        });
 
                        var meta = chart.getDatasetMeta(3);
-                       expect(meta.controller.getStackCount()).toBe(3);
+                       expect(meta.controller._getStackCount()).toBe(3);
                });
        });
 
@@ -212,7 +212,7 @@ describe('Chart.controllers.bar', function() {
                        });
 
                        var meta = chart.getDatasetMeta(3);
-                       expect(meta.controller.getStackCount()).toBe(2);
+                       expect(meta.controller._getStackCount()).toBe(2);
                });
        });
 
@@ -245,7 +245,7 @@ describe('Chart.controllers.bar', function() {
                        });
 
                        var meta = chart.getDatasetMeta(3);
-                       expect(meta.controller.getStackCount()).toBe(4);
+                       expect(meta.controller._getStackCount()).toBe(4);
                });
        });
 
@@ -268,7 +268,7 @@ describe('Chart.controllers.bar', function() {
                        });
 
                        var meta = chart.getDatasetMeta(3);
-                       expect(meta.controller.getStackCount()).toBe(2);
+                       expect(meta.controller._getStackCount()).toBe(2);
                });
        });
 
@@ -301,7 +301,7 @@ describe('Chart.controllers.bar', function() {
                        });
 
                        var meta = chart.getDatasetMeta(3);
-                       expect(meta.controller.getStackCount()).toBe(2);
+                       expect(meta.controller._getStackCount()).toBe(2);
                });
        });
 
@@ -334,7 +334,7 @@ describe('Chart.controllers.bar', function() {
                        });
 
                        var meta = chart.getDatasetMeta(3);
-                       expect(meta.controller.getStackCount()).toBe(4);
+                       expect(meta.controller._getStackCount()).toBe(4);
                });
        });
 
@@ -357,8 +357,8 @@ describe('Chart.controllers.bar', function() {
                        });
 
                        var meta = chart.getDatasetMeta(1);
-                       expect(meta.controller.getStackIndex(0)).toBe(0);
-                       expect(meta.controller.getStackIndex(3)).toBe(1);
+                       expect(meta.controller._getStackIndex(0)).toBe(0);
+                       expect(meta.controller._getStackIndex(3)).toBe(1);
                });
        });
 
@@ -381,10 +381,10 @@ describe('Chart.controllers.bar', function() {
                        });
 
                        var meta = chart.getDatasetMeta(1);
-                       expect(meta.controller.getStackIndex(0)).toBe(0);
-                       expect(meta.controller.getStackIndex(1)).toBe(1);
-                       expect(meta.controller.getStackIndex(2)).toBe(2);
-                       expect(meta.controller.getStackIndex(3)).toBe(3);
+                       expect(meta.controller._getStackIndex(0)).toBe(0);
+                       expect(meta.controller._getStackIndex(1)).toBe(1);
+                       expect(meta.controller._getStackIndex(2)).toBe(2);
+                       expect(meta.controller._getStackIndex(3)).toBe(3);
                });
        });
 
@@ -417,10 +417,10 @@ describe('Chart.controllers.bar', function() {
                        });
 
                        var meta = chart.getDatasetMeta(1);
-                       expect(meta.controller.getStackIndex(0)).toBe(0);
-                       expect(meta.controller.getStackIndex(1)).toBe(0);
-                       expect(meta.controller.getStackIndex(2)).toBe(0);
-                       expect(meta.controller.getStackIndex(3)).toBe(0);
+                       expect(meta.controller._getStackIndex(0)).toBe(0);
+                       expect(meta.controller._getStackIndex(1)).toBe(0);
+                       expect(meta.controller._getStackIndex(2)).toBe(0);
+                       expect(meta.controller._getStackIndex(3)).toBe(0);
                });
        });
 
@@ -453,10 +453,10 @@ describe('Chart.controllers.bar', function() {
                        });
 
                        var meta = chart.getDatasetMeta(1);
-                       expect(meta.controller.getStackIndex(0)).toBe(0);
-                       expect(meta.controller.getStackIndex(1)).toBe(1);
-                       expect(meta.controller.getStackIndex(2)).toBe(2);
-                       expect(meta.controller.getStackIndex(3)).toBe(3);
+                       expect(meta.controller._getStackIndex(0)).toBe(0);
+                       expect(meta.controller._getStackIndex(1)).toBe(1);
+                       expect(meta.controller._getStackIndex(2)).toBe(2);
+                       expect(meta.controller._getStackIndex(3)).toBe(3);
                });
        });
 
@@ -479,10 +479,10 @@ describe('Chart.controllers.bar', function() {
                        });
 
                        var meta = chart.getDatasetMeta(1);
-                       expect(meta.controller.getStackIndex(0)).toBe(0);
-                       expect(meta.controller.getStackIndex(1)).toBe(0);
-                       expect(meta.controller.getStackIndex(2)).toBe(1);
-                       expect(meta.controller.getStackIndex(3)).toBe(2);
+                       expect(meta.controller._getStackIndex(0)).toBe(0);
+                       expect(meta.controller._getStackIndex(1)).toBe(0);
+                       expect(meta.controller._getStackIndex(2)).toBe(1);
+                       expect(meta.controller._getStackIndex(3)).toBe(2);
                });
        });
 
@@ -515,10 +515,10 @@ describe('Chart.controllers.bar', function() {
                        });
 
                        var meta = chart.getDatasetMeta(1);
-                       expect(meta.controller.getStackIndex(0)).toBe(0);
-                       expect(meta.controller.getStackIndex(1)).toBe(0);
-                       expect(meta.controller.getStackIndex(2)).toBe(1);
-                       expect(meta.controller.getStackIndex(3)).toBe(1);
+                       expect(meta.controller._getStackIndex(0)).toBe(0);
+                       expect(meta.controller._getStackIndex(1)).toBe(0);
+                       expect(meta.controller._getStackIndex(2)).toBe(1);
+                       expect(meta.controller._getStackIndex(3)).toBe(1);
                });
        });
 
@@ -551,10 +551,10 @@ describe('Chart.controllers.bar', function() {
                        });
 
                        var meta = chart.getDatasetMeta(1);
-                       expect(meta.controller.getStackIndex(0)).toBe(0);
-                       expect(meta.controller.getStackIndex(1)).toBe(1);
-                       expect(meta.controller.getStackIndex(2)).toBe(2);
-                       expect(meta.controller.getStackIndex(3)).toBe(3);
+                       expect(meta.controller._getStackIndex(0)).toBe(0);
+                       expect(meta.controller._getStackIndex(1)).toBe(1);
+                       expect(meta.controller._getStackIndex(2)).toBe(2);
+                       expect(meta.controller._getStackIndex(3)).toBe(3);
                });
        });
 
@@ -577,10 +577,10 @@ describe('Chart.controllers.bar', function() {
                        });
 
                        var meta = chart.getDatasetMeta(1);
-                       expect(meta.controller.getStackIndex(0)).toBe(0);
-                       expect(meta.controller.getStackIndex(1)).toBe(0);
-                       expect(meta.controller.getStackIndex(2)).toBe(1);
-                       expect(meta.controller.getStackIndex(3)).toBe(1);
+                       expect(meta.controller._getStackIndex(0)).toBe(0);
+                       expect(meta.controller._getStackIndex(1)).toBe(0);
+                       expect(meta.controller._getStackIndex(2)).toBe(1);
+                       expect(meta.controller._getStackIndex(3)).toBe(1);
                });
        });
 
@@ -613,10 +613,10 @@ describe('Chart.controllers.bar', function() {
                        });
 
                        var meta = chart.getDatasetMeta(1);
-                       expect(meta.controller.getStackIndex(0)).toBe(0);
-                       expect(meta.controller.getStackIndex(1)).toBe(0);
-                       expect(meta.controller.getStackIndex(2)).toBe(1);
-                       expect(meta.controller.getStackIndex(3)).toBe(1);
+                       expect(meta.controller._getStackIndex(0)).toBe(0);
+                       expect(meta.controller._getStackIndex(1)).toBe(0);
+                       expect(meta.controller._getStackIndex(2)).toBe(1);
+                       expect(meta.controller._getStackIndex(3)).toBe(1);
                });
        });
 
@@ -649,10 +649,10 @@ describe('Chart.controllers.bar', function() {
                        });
 
                        var meta = chart.getDatasetMeta(1);
-                       expect(meta.controller.getStackIndex(0)).toBe(0);
-                       expect(meta.controller.getStackIndex(1)).toBe(1);
-                       expect(meta.controller.getStackIndex(2)).toBe(2);
-                       expect(meta.controller.getStackIndex(3)).toBe(3);
+                       expect(meta.controller._getStackIndex(0)).toBe(0);
+                       expect(meta.controller._getStackIndex(1)).toBe(1);
+                       expect(meta.controller._getStackIndex(2)).toBe(2);
+                       expect(meta.controller._getStackIndex(3)).toBe(3);
                });
        });
 
index b69afb6c5d4c37cbe4f9d4a57e79b3b54d55ac9a..70e1d00dbc4df01ad9e50e55237183e519e25cbe 100644 (file)
@@ -12,11 +12,11 @@ describe('Chart.DatasetController', function() {
 
                var controller = chart.getDatasetMeta(0).controller;
                var methods = [
-                       'onDataPush',
-                       'onDataPop',
-                       'onDataShift',
-                       'onDataSplice',
-                       'onDataUnshift'
+                       '_onDataPush',
+                       '_onDataPop',
+                       '_onDataShift',
+                       '_onDataSplice',
+                       '_onDataUnshift'
                ];
 
                methods.forEach(function(method) {
index 0a376f7ea8a1e929785ca0c31276e82e4586c938..db0515769af09be08c40a00b8dbe0d13598305c5 100644 (file)
@@ -108,11 +108,11 @@ function waitForResize(chart, callback) {
 }
 
 function afterEvent(chart, type, callback) {
-       var override = chart.eventHandler;
-       chart.eventHandler = function(event) {
+       var override = chart._eventHandler;
+       chart._eventHandler = function(event) {
                override.call(this, event);
                if (event.type === type) {
-                       chart.eventHandler = override;
+                       chart._eventHandler = override;
                        // eslint-disable-next-line callback-return
                        callback();
                }