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
7.02k stars 656 forks source link

Invalid query throws `n.Term is not a constructor` #1871

Closed jcubic closed 6 months ago

jcubic commented 8 months ago

I've noticed that the error message on the invalid token looks like this:

n.Term is not a constructor

When executing:

> xxxx

It looks like the invalid token is not handled and it just throws an unhandled exception.

You can see this on my Terminal App:

https://codepen.io/jcubic/pen/dVBaRm

mathiasrw commented 8 months ago

What are you expecting from the query XXXX?

jcubic commented 8 months ago

I expected a different error message, like "Invalid Query" or "Invalid Token", but the error is not handled and it throws an internal error.

vishal6557 commented 6 months ago

@mathiasrw Can you assign this to me? Thanks!

jcubic commented 6 months ago

@vishal6557 Open Source project don't use assignment, especially for people that are outside the project. Where did you learn that project need to assign the task to you?

vishal6557 commented 6 months ago

@jcubic Thank you for your feedback! You're right—open-source projects flourish through collaboration and volunteer efforts. My goal was simply to offer help with this issue, aiming to contribute positively without overstepping.

mathiasrw commented 6 months ago

Fix released as part of v4.3.1