-
Notifications
You must be signed in to change notification settings - Fork 25
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
SQLite Compatibility #92
Comments
OK, so for the |
@swissspidy I took a look at this, I'm not familiar with SQLite so I might be mistaking, but it seems that SQLite doesn't support
I did manage to get the tests to pass by doing three queries to replicate what this |
Interesting! Do you perhaps have a draft PR with that solution so that we could take a look? It does seem like a case the SQLite plugin could handle, so If you could open an issues I'm sure the maintainers can take a look. |
Turn out it's related to the escaping, like you suspected. Looking at the I'm able to fix the issue by modifying the
|
Let‘s try that 👍 |
When I manually run the tests locally with SQLite, the following scenarios are failing:
The transient command uses lots of custom MySQL queries. The SQLite integration plugin is supposed to make those SQLite-compatible, but perhaps something is not working there.
Or, since deletion and listing are failing, maybe the issue is that the insertion using
set_transient()
is failing in the first place.Let's figure it out together. This could involve changes to the command, the tests, core, or the SQLite plugin.
The text was updated successfully, but these errors were encountered: