-
Notifications
You must be signed in to change notification settings - Fork 23
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
support sqlite 3.41.0 by avoiding double quotes for values #235
Comments
getting same issue. thanks for the work around. |
Unless I am misunderstanding it, it is not quite a fix, you just going back to the old sqlite version that the extension uses by default. |
@GreLeBr correct, its a workaround to restore functionality at the time of the extenion's release but does not fix the underlying issue. however the actual fix should be pretty simple. just replace double quotes with single quotes |
Hi, Thanks for this awesome extension ! Unfortunately, I am having the same error on Linux (Manjaro), with latest sqlite3 version installed : v. 3.42.0 I am afraid the (grateful) workaround does not work on Linux, unfortunately.. Fixing the extension path to Could you please fix this ? Thankxx by advance |
Hey @Pyrolistical , Can you please provide more detail on what I actually need to change? Is it this setting of the extension? What value am I putting there? |
@smargoli2 That should be correct |
Thanks, the full path that I used is "C:\Users\Sora Teichman.vscode\extensions\alexcvzz.vscode-sqlite-0.14.1\bin\sqlite-v3.26.0-win32-x86.exe", the database is not opening though. |
@smargoli2 you are missing a slash before .vscode |
It is working, I was just looking in the wrong place :) Thank you! |
I am running into this issue on Linux box as well. I changed the path to point to binary of the extension and still getting the error |
I also solved my problem following this. thanks @smargoli2 and @Pyrolistical . you guys saved me from stress. |
A different fix for this: I went to C:\Users<me>.vscode\extensions\alexcvzz.vscode-sqlite-0.14.1\dist\extension.js |
@jmforsythe make a PR 😃 |
#241 Already exists, my fix just does what it does after the plugin is installed. |
The following process worked for me on Windows 10 In VSCODE, I changed the slashes from back slashes to forward slashes in the sqlite extension configuration and it worked. I tested the following line in the VSCODE terminal to verify it runs. |
This definitely works! Thanks. Maybe make a fork of this estension ? |
In SQlite Extention setting custom . SQLite3 command or executable path. => sqlite.sqlite3 it's will work oke |
This does not work at all, this project is dead. Who is taking over this extension? |
In standard sql single quotes are for values and double quotes are for identities like column names. Although sqlite does support double quote values this seems to have changed in 3.41.0.
I have sqlite 3.41.0 installed and when this extension uses it, I get the following error when I try to open a database:
I was able to track down the query:
vscode-sqlite/src/sqlite/schema.ts
Lines 46 to 50 in c09e348
If one tries to run that query on an empty sqlite database in the cli, we get a similar error
The workaround is to pin the sqlite client to the one included with the extension. Set the vscode setting
sqlite.sqlite3
to the correct client in.vscode\extensions\alexcvzz.vscode-sqlite-0.14.1\bin
The text was updated successfully, but these errors were encountered: