Source files have plenty of implicitly declared variables in global scope.

Thread ID:	Created:	Updated:	Platform:	Replies:
142405	Jan 31,2019 09:13 PM UTC	Feb 5,2019 01:07 PM UTC	jQuery	1

Subscribe to this post

Tags: ejChart

Norbertas Gaulia

Asked On January 31, 2019 09:23 PM UTC

I'm running into a lot of issues like this:

For example file ej.chart.js

Problem is that loop declares variable `m` but `m` is not declared at the function `seriesRender` scope.

In turn it declares `m` in Global scope and after running line 26728

`this.renderSeries(...)` function has exact same bug - for loop without locally declared `m` and it changes `m` value,

then obviously line 26729 fails with `this.model.series[m]` m being out Array range:

So i've checked whole library with code quality tool for specific rule on Intellij 'Implicitly declared global JavaScript error' or EsLint 'no-undef'

it has 824 those bugs:

Please fix this as soon as possible.

Thank you.

Reply | Reply with quote

Dharanidharan Dharmasivam [Syncfusion]

Replied On February 5, 2019 01:07 PM UTC

Hi Norbertas,

Greetings from Syncfusion.

We were able to reproduce the issue Plenty of implicitly declared variables and we confirm this as a bug and logged a defect report. You can keep track of the bug from the feedback portal below.

Link: https://www.syncfusion.com/feedback/4577/

The fix will be available in our upcoming Volume 1 main release which is scheduled to be rolled out at middle of March 2019.

If you have any more specification/precise replication procedure or a scenario to be tested you can add it as a comment in the portal.

Thanks,

Dharani.

Reply | Reply with quote

CONTACT US

Source files have plenty of implicitly declared variables in global scope.

Thread ID:

Created:

Updated:

Platform:

Replies:

Please sign in to access our forum

CONTACT US