search

nextcloud / tables

🍱 Nextcloud tables app
https://apps.nextcloud.com/apps/tables
GNU Affero General Public License v3.0
133 stars 21 forks source link

fix(DB): fetch pageId as int #1083

Closed blizzz closed 1 month ago

blizzz commented 1 month ago

fixes #1081

blizzz commented 1 month ago

/backport to stable0.7

blizzz commented 1 month ago

I always wonder if the query builder could not cast those automatically, but I guess it cause we don't have type information on the php side for the columns and sqlite considering everything as a string 🙈

The QBMapper usually would do that, but is not used here. A pitfall to be aware about. Having this automatic with the DB abstraction (based on migration?) would be fancy indeed.