search

AlaSQL / alasql

AlaSQL.js - JavaScript SQL database for browser and Node.js. Handles both traditional relational tables and nested JSON data (NoSQL). Export, store, and import data from localStorage, IndexedDB, or Excel.
http://alasql.org
MIT License
6.96k stars 651 forks source link

Invalid query throws n.Term #1871 #1903

Closed vishal6557 closed 3 months ago

vishal6557 commented 3 months ago

Fix #1871

Thank you for the time you are putting into AlaSQL!

vishal6557 commented 3 months ago

@mathiasrw Have a look, any feedback is appreciated!

mathiasrw commented 3 months ago

Very interesting how you solved this. Im wondering why the yy.Term was there in the first place.

Ill merge it.

Did you have any problem generating the file from the jison? I had some challenges with deep dependencies lately. Or did you just update the actual .js file?

mathiasrw commented 3 months ago

Fix released as part of v4.3.1

vishal6557 commented 3 months ago

@mathiasrw No, I don't have a problem generating files from the jison

I think yarn jison command is broken, I manually install jison globally and using command jison ./src/alasqlparser.jison -o ./src/alasqlparser.js

Can you try this command if it works, I'll create a pull request and update the package.json

mathiasrw commented 3 months ago

Neat! Ill look into this!

mathiasrw commented 3 months ago

It works. Neat!