chartjs / chartjs/Chart.js

Duplicate Property Names Error when using BundlerMinifier in Visual Studio

Open
#10,366 2 comments 2 reactions 0 assignees View on GitHub
type: bug
Dominant language
JavaScript
Stars
67.7k
Forks
11.9k
Avg merge
7h 39m
Merged PRs (30d)
5

Description

### Expected behavior

Using BuildBundlerMinifier (or BuildBundlerMinifierPlus) in Visual Studio and the raw Chart.js available here: https://cdn.jsdelivr.net/npm/chart.js@3.7.1/dist/chart.js you should be able to bundle and minify Chart.js.

### Current behavior

With the unmodified Chart.js file from the above CDN, you get the following errors, specifically due to the Chart.js file (I have removed it and they have disappeared, and also removed one-by-one every other js in the bundle and the errors did **not** disappear):

Error 0 Implicit property name must be identifier: stackWeight = shared.min.js 26694
Error 0 Strict-mode does not allow duplicate property names: = shared.min.js 34889
Error 0 Strict-mode does not allow duplicate property names: = shared.min.js 32303
Error 0 Strict-mode does not allow duplicate property names: = shared.min.js 28870
Error 0 Strict-mode does not allow duplicate property names: = shared.min.js 34809
Error 0 Strict-mode does not allow duplicate property names: = shared.min.js 34518
Error 0 Strict-mode does not allow duplicate property names: = shared.min.js 27071
Error 0 Strict-mode does not allow duplicate property names: = shared.min.js 27071

If you initialize stackWeight outside of line 2043 in Chart.js, and remove "use strict", you still get two errors:

Severity Code Description Project File Line Suppression State
Error 0 Strict-mode does not allow duplicate property names: = shared.min.js 32303
Error 0 Strict-mode does not allow duplicate property names: = shared.min.js 28870

### Reproducible sample

I can probably upload a barebones VS solution if need be.

### Optional extra steps/info to reproduce

_No response_

### Possible solution

I am not a js developer by any means but I think the issue is ESLint related?

### Context

Trying to update from Charts 2.9.3, which bundled just fine in Visual Studio using the same tool.

### chart.js version

3.7.1

### Browser name and version

NA

### Link to your project

_No response_

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.