Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

3.8 Update Bug? #849

Closed
bbrearle opened this issue Mar 23, 2017 · 2 comments
Closed

3.8 Update Bug? #849

bbrearle opened this issue Mar 23, 2017 · 2 comments

Comments

@bbrearle
Copy link

bbrearle commented Mar 23, 2017

Quering an external csv resource appears to have stopped working (atleast in this use case) on the update from 3.6 to 3.8.

I have a back up copy of the 3.6 version. It is working here: fiddle.

And 3.8 not working here: fiddle.

Console Error:

Uncaught TypeError: Cannot read property 'autoExt' of null at Object.C.autoExtFilename (https://cdnjs.cloudflare.com/ajax/libs/alasql/0.3.8/alasql.min.js:11:9316) at Object.A.from.CSV (https://cdnjs.cloudflare.com/ajax/libs/alasql/0.3.8/alasql.min.js:15:22714) at Object.eval [as datafn] (eval at <anonymous> (https://cdnjs.cloudflare.com/ajax/libs/alasql/0.3.8/alasql.min.js:1:1), <anonymous>:3:27) at https://cdnjs.cloudflare.com/ajax/libs/alasql/0.3.8/alasql.min.js:2:6976 at Array.forEach (native) at o (https://cdnjs.cloudflare.com/ajax/libs/alasql/0.3.8/alasql.min.js:2:6936) at s (https://cdnjs.cloudflare.com/ajax/libs/alasql/0.3.8/alasql.min.js:12:8299) at Function.A.dexec (https://cdnjs.cloudflare.com/ajax/libs/alasql/0.3.8/alasql.min.js:11:16432) at Function.A.exec (https://cdnjs.cloudflare.com/ajax/libs/alasql/0.3.8/alasql.min.js:11:15867) at A (https://cdnjs.cloudflare.com/ajax/libs/alasql/0.3.8/alasql.min.js:2:20273)

I think it has to do with the auto-extension, in this case .csv, being appended to the end of the csv location directory. In this case a call with query string (api key).

Any way to disable auto extension? Thanks.

@mathiasrw
Copy link
Member

mathiasrw commented Mar 23, 2017

Woops! Thank you for taking the time to let us know.

Please use version 0.3.9 - just released...

For 0.3.8 you would need to add a (epmty?) config object as second parameter in CSV(filename, {})

@bbrearle
Copy link
Author

bbrearle commented Mar 24, 2017 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants