From a97dd58e3b499b60ac8b37786d402d7f2e371a88 Mon Sep 17 00:00:00 2001
From: Daniel Opitz <danopz@users.noreply.github.com>
Date: Mon, 14 Aug 2017 10:34:53 +0200
Subject: 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
---
 appinfo/database.xml | 12 ++++++++++++
 appinfo/info.xml     |  8 +++++++-
 2 files changed, 19 insertions(+), 1 deletion(-)

(limited to 'appinfo')

diff --git a/appinfo/database.xml b/appinfo/database.xml
index 1044208cd..e9c65a947 100644
--- a/appinfo/database.xml
+++ b/appinfo/database.xml
@@ -363,6 +363,18 @@
                 <default>0</default>
                 <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>
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>
-- 
cgit v1.2.3