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

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.




1 Reply

DD Dharanidharan Dharmasivam Syncfusion Team 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. 
 
 
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. 


Loader.
Up arrow icon