diff options
author | Daniel Opitz <danopz@users.noreply.github.com> | 2017-08-14 10:34:53 +0200 |
---|---|---|
committer | Bernhard Posselt <BernhardPosselt@users.noreply.github.com> | 2017-08-14 10:34:53 +0200 |
commit | a97dd58e3b499b60ac8b37786d402d7f2e371a88 (patch) | |
tree | 98bb8a6c750fb33fbef38d22407fa29fbf6c7b1e /appinfo | |
parent | 7d8a85c82c4c13a71b70ddb4ecb8c40ede4c9b70 (diff) |
Split binary to booleans (#203)
* replaced old status with 2 flags for unread and starred
* add fields to db, replace int(1,0) with booleans in sql queries, removed StatusFlags class + refactor code relying to it
* add repair step for migration
* again use integer(1,0) instead of bool in sql queries, because of sqlite doesn't support true/false
* add/fix unit tests for new boolean status
* set unread/starred flags as statements in sql
* fixed mysql unknown column items.unread, fixed marking of read items on repair step
* remove unnecessary bool casts
* add empty checks to Items::is* methods
* update migration to use native sql instead of the querybuilder
* don't cast the flags manually, let the api do the work
Diffstat (limited to 'appinfo')
-rw-r--r-- | appinfo/database.xml | 12 | ||||
-rw-r--r-- | appinfo/info.xml | 8 |
2 files changed, 19 insertions, 1 deletions
diff --git a/appinfo/database.xml b/appinfo/database.xml index 1044208cd..e9c65a947 100644 --- a/appinfo/database.xml +++ b/appinfo/database.xml @@ -364,6 +364,18 @@ <notnull>true</notnull> </field> <field> + <name>unread</name> + <type>boolean</type> + <default>false</default> + <notnull>true</notnull> + </field> + <field> + <name>starred</name> + <type>boolean</type> + <default>false</default> + <notnull>true</notnull> + </field> + <field> <name>last_modified</name> <type>integer</type> <default>0</default> diff --git a/appinfo/info.xml b/appinfo/info.xml index 6f97c052f..099f3d824 100644 --- a/appinfo/info.xml +++ b/appinfo/info.xml @@ -8,7 +8,7 @@ Before you update to a new version, [check the changelog](https://github.com/nextcloud/news/blob/master/CHANGELOG.md) to avoid surprises. **Important**: To enable feed updates you will need to enable either [Nextcloud system cron](https://docs.nextcloud.com/server/10/admin_manual/configuration_server/background_jobs_configuration.html#cron) or use [an updater](https://github.com/nextcloud/news-updater) which uses the built in update API and disable cron updates. More information can be found [in the README](https://github.com/nextcloud/news).]]></description> - <version>11.0.5</version> + <version>11.0.6</version> <licence>agpl</licence> <author>Bernhard Posselt</author> <author>Alessandro Cosentino</author> @@ -42,6 +42,12 @@ Before you update to a new version, [check the changelog](https://github.com/nex <job>OCA\News\Cron\Updater</job> </background-jobs> + <repair-steps> + <post-migration> + <step>OCA\News\Migration\MigrateStatusFlags</step> + </post-migration> + </repair-steps> + <settings> <admin>OCA\News\Settings\Admin</admin> <admin-section>OCA\News\Settings\Section</admin-section> |