Skip to content

Fix SQLite regression#4389

Merged
Alkarex merged 1 commit intoFreshRSS:edgefrom
Alkarex:fix-sqlite
May 26, 2022
Merged

Fix SQLite regression#4389
Alkarex merged 1 commit intoFreshRSS:edgefrom
Alkarex:fix-sqlite

Conversation

@Alkarex
Copy link
Copy Markdown
Member

@Alkarex Alkarex commented May 26, 2022

Revert SQLite part of #4387 , fix SQLite part of #4347
Apparently INT and INTEGER are not perfectly synonym in SQLite!

An INTEGER PRIMARY KEY column is an alias for the rowid, but an INT PRIMARY KEY column is not.

https://www.sqlite.org/stricttables.html#strict_tables

Revert SQLite part of FreshRSS#4387
FreshRSS#4347
Apparently INT and INTEGER are not perfectly synonym in SQLite!
> An INTEGER PRIMARY KEY column is an alias for the rowid, but an INT PRIMARY KEY column is not.
https://www.sqlite.org/stricttables.html#strict_tables
@Alkarex Alkarex added this to the 1.20.0 milestone May 26, 2022
@Alkarex Alkarex merged commit 516f0c0 into FreshRSS:edge May 26, 2022
@Alkarex Alkarex deleted the fix-sqlite branch May 26, 2022 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant