We use cookies to give you the best experience on our website. If you continue to browse, then you agree to our privacy policy and cookie policy. Image for the cookie policy date

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