jqxdata.export.js

[dflemingParticipant]

While upgrading to v4.0.0 from v2.8.3, I’ve run into a problem with jqxdata.export.js. It gets an error as it is being loaded by require.js: “Uncaught TypeError: $ is not a function”

Only jqxdata.export.js has an error. Other jqWidgets modules load and work without any problems.

This is the way that we load it in App.js:

require(
[
// Load our app module and pass it to our definition function
‘App’,

// Some plugins have to be loaded in order due to there non AMD compliance
// Because these scripts are not “modules” they do not pass any values to the definition function below
‘order!../../common/js/libs/jquery/jquery-1.8.3.min’,

… other files defined here…
‘order!../../common/js/libs/jquery/plugins/jqwidgets/jqwidgets’,

], function(App) …

Then, the ../../common/js/libs/jquery/plugins/jqwidgets/jqwidgets.js file just has the jqwidgets files that are loaded for our app:

define([
‘order!/common/js/libs/jquery/plugins/jqwidgets/jqwidgets/jqxcore.js’,
‘order!/common/js/libs/jquery/plugins/jqwidgets/jqwidgets/jqxdata.js’,

… more jqwidgets files here …

‘order!/common/js/libs/jquery/plugins/jqwidgets/jqwidgets/jqxdata.export.js’,

… more jqwidgets files here …

], function() {
// Tell Require.js that this module returns a reference to jQuery
return $;
});

[DimitarParticipant]

Hello dfleming,

Maybe the issue occurs because of the composite file jqwidgets.js and the order you have put the jQWidgets components in it. Please note that jqxdata.export.js is dependant on jqxdata.js. Please also note that there have been several breaking changes since version 2.8.3, all of which you can find documented in Release History. You can also try using our own composite file – jqx-all.js.

Best Regards,
Dimitar

jQWidgets team
http://www.jqwidgets.com/

[dflemingParticipant]

As an aside, I’m very pleased with how well you implemented all the changes from v2.8.3 to v4.0.0. Other than working around this error, I’ve only had to make one small change to the our cellsrenderer function in jqxGrid. I wish other code that we update would be as trouble free.

I’ve been through the release history looking for where the problem was introduced. The only reference to jqxdata.export.js is in v3.9.1. This problem was actually introduced into the code in v3.8.1.

This error occurs because the saveAs function in jqxdata.export.js is being defined as a global function. In v4, line 4510 of jqxdata.export.js, jQuery is used like this: $(““)[0]. Because of the way we are using require.js, the $ function isn’t defined at that point, causing the error.

That was a change in v3.8.1. If you look at line 4511 in v4.0.0 jqxdata.export.js, you’ll see the way it was done in v3.8.0. If I comment line 4510 and uncomment line 4511, the code loads without errors. I can also just change line 4510 to jQuery(““)[0] to eliminate the error.

This is probably caused by the particular way in which we are using require.js to load the files.

[Author]

Posts