Skip to content

Commit

Permalink
Update spelling of cancellable (#8236)
Browse files Browse the repository at this point in the history
  • Loading branch information
benmccann committed Dec 28, 2020
1 parent 78e7825 commit e6dfc47
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 14 deletions.
24 changes: 12 additions & 12 deletions src/core/core.controller.js
Original file line number Diff line number Diff line change
Expand Up @@ -462,7 +462,7 @@ class Chart {
// https://github.com/chartjs/Chart.js/issues/5111#issuecomment-355934167
me._plugins.invalidate();

if (me.notifyPlugins('beforeUpdate', {mode, cancelable: true}) === false) {
if (me.notifyPlugins('beforeUpdate', {mode, cancellable: true}) === false) {
return;
}

Expand Down Expand Up @@ -504,7 +504,7 @@ class Chart {
_updateLayout() {
const me = this;

if (me.notifyPlugins('beforeLayout', {cancelable: true}) === false) {
if (me.notifyPlugins('beforeLayout', {cancellable: true}) === false) {
return;
}

Expand Down Expand Up @@ -544,7 +544,7 @@ class Chart {
const me = this;
const isFunction = typeof mode === 'function';

if (me.notifyPlugins('beforeDatasetsUpdate', {mode, cancelable: true}) === false) {
if (me.notifyPlugins('beforeDatasetsUpdate', {mode, cancellable: true}) === false) {
return;
}

Expand All @@ -563,21 +563,21 @@ class Chart {
_updateDataset(index, mode) {
const me = this;
const meta = me.getDatasetMeta(index);
const args = {meta, index, mode, cancelable: true};
const args = {meta, index, mode, cancellable: true};

if (me.notifyPlugins('beforeDatasetUpdate', args) === false) {
return;
}

meta.controller._update(mode);

args.cancelable = false;
args.cancellable = false;
me.notifyPlugins('afterDatasetUpdate', args);
}

render() {
const me = this;
if (me.notifyPlugins('beforeRender', {cancelable: true}) === false) {
if (me.notifyPlugins('beforeRender', {cancellable: true}) === false) {
return;
}

Expand Down Expand Up @@ -605,7 +605,7 @@ class Chart {
return;
}

if (me.notifyPlugins('beforeDraw', {cancelable: true}) === false) {
if (me.notifyPlugins('beforeDraw', {cancellable: true}) === false) {
return;
}

Expand Down Expand Up @@ -662,7 +662,7 @@ class Chart {
_drawDatasets() {
const me = this;

if (me.notifyPlugins('beforeDatasetsDraw', {cancelable: true}) === false) {
if (me.notifyPlugins('beforeDatasetsDraw', {cancellable: true}) === false) {
return;
}

Expand All @@ -687,7 +687,7 @@ class Chart {
const args = {
meta,
index: meta.index,
cancelable: true
cancellable: true
};

if (me.notifyPlugins('beforeDatasetDraw', args) === false) {
Expand All @@ -705,7 +705,7 @@ class Chart {

unclipArea(ctx);

args.cancelable = false;
args.cancellable = false;
me.notifyPlugins('afterDatasetDraw', args);
}

Expand Down Expand Up @@ -1020,15 +1020,15 @@ class Chart {
*/
_eventHandler(e, replay) {
const me = this;
const args = {event: e, replay, cancelable: true};
const args = {event: e, replay, cancellable: true};

if (me.notifyPlugins('beforeEvent', args) === false) {
return;
}

const changed = me._handleEvent(e, replay);

args.cancelable = false;
args.cancellable = false;
me.notifyPlugins('afterEvent', args);

if (changed) {
Expand Down
2 changes: 1 addition & 1 deletion src/core/core.plugins.js
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ export default class PluginService {
const plugin = descriptor.plugin;
const method = plugin[hook];
const params = [chart, args, descriptor.options];
if (callCallback(method, params, plugin) === false && args.cancelable) {
if (callCallback(method, params, plugin) === false && args.cancellable) {
return false;
}
}
Expand Down
2 changes: 1 addition & 1 deletion test/specs/core.plugin.tests.js
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,7 @@ describe('Chart.plugins', function() {
spyOn(plugin, 'hook').and.callThrough();
});

var ret = chart.notifyPlugins('hook', {cancelable: true});
var ret = chart.notifyPlugins('hook', {cancellable: true});
expect(ret).toBeFalsy();
expect(plugins[0].hook).toHaveBeenCalled();
expect(plugins[1].hook).toHaveBeenCalled();
Expand Down

0 comments on commit e6dfc47

Please sign in to comment.