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. (Last updated on: June 24, 2019).
Unfortunately, activation email could not send to your email. Please try again.
Syncfusion Feedback
Essential JS2

Syncfusion jQuery based widgets are no longer in active development. Switch to our pure JavaScript based next generation Essential JS 2 library.

Try Essential JS 2

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
loading
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.




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. 
 
 
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. 


CONFIRMATION

This post will be permanently deleted. Are you sure you want to continue?

Sorry, An error occured while processing your request. Please try again later.

Please sign in to access our forum

This page will automatically be redirected to the sign-in page in 10 seconds.

Warning Icon You are using an outdated version of Internet Explorer that may not display all features of this and other websites. Upgrade to Internet Explorer 8 or newer for a better experience.Close Icon

Live Chat Icon For mobile
Live Chat Icon