Invalid query throws n.Term #1871

[vishal6557]

Fix #1871

Thank you for the time you are putting into AlaSQL!

[vishal6557]

@mathiasrw Have a look, any feedback is appreciated!

[mathiasrw]

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]

Fix released as part of v4.3.1

[vishal6557]

@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]

Neat! Ill look into this!

[mathiasrw]

It works. Neat!