DB: Use boolean parameter for unread/starred

Issue GH-1278

Signed-off-by: Sean Molenaar <sean@seanmolenaar.eu>

2 files changed, 51 insertions, 25 deletions

diff --git a/tests/Unit/Db/ItemMapperAfterTest.php b/tests/Unit/Db/ItemMapperAfterTest.php
index 5b0475522..68bad19dc 100644
--- a/tests/Unit/Db/ItemMapperAfterTest.php
+++ b/tests/Unit/Db/ItemMapperAfterTest.php
@@ -134,7 +134,7 @@ class ItemMapperAfterTest extends MapperTestUtility
                 ['feeds.user_id = :userId'],
                 ['feeds.id = :feedId'],
                 ['feeds.deleted_at = 0'],
-                ['items.unread = 1']
+                ['items.unread = :unread']
             )
             ->will($this->returnSelf());
 
@@ -147,6 +147,11 @@ class ItemMapperAfterTest extends MapperTestUtility
             ])
             ->will($this->returnSelf());
 
+        $this->builder->expects($this->exactly(1))
+            ->method('setParameter')
+            ->with('unread', true)
+            ->will($this->returnSelf());
+
         $this->builder->expects($this->once())
             ->method('orderBy')
             ->with('items.last_modified', 'DESC')
@@ -271,7 +276,7 @@ class ItemMapperAfterTest extends MapperTestUtility
                 ['feeds.user_id = :userId'],
                 ['feeds.deleted_at = 0'],
                 ['folders.id = :folderId'],
-                ['items.unread = 1']
+                ['items.unread = :unread']
             )
             ->will($this->returnSelf());
 
@@ -284,6 +289,11 @@ class ItemMapperAfterTest extends MapperTestUtility
             ])
             ->will($this->returnSelf());
 
+        $this->builder->expects($this->exactly(1))
+            ->method('setParameter')
+            ->with('unread', true)
+            ->will($this->returnSelf());
+
         $this->builder->expects($this->once())
             ->method('orderBy')
             ->with('items.last_modified', 'DESC')
@@ -336,7 +346,7 @@ class ItemMapperAfterTest extends MapperTestUtility
                 ['items.last_modified >= :updatedSince'],
                 ['feeds.deleted_at = 0'],
                 ['feeds.user_id = :userId'],
-                ['items.unread = 1']
+                ['items.unread = :unread']
             )
             ->will($this->returnSelf());
 
@@ -348,6 +358,11 @@ class ItemMapperAfterTest extends MapperTestUtility
             ])
             ->will($this->returnSelf());
 
+        $this->builder->expects($this->exactly(1))
+            ->method('setParameter')
+            ->with('unread', true)
+            ->will($this->returnSelf());
+
         $this->builder->expects($this->once())
             ->method('orderBy')
             ->with('items.last_modified', 'DESC')
@@ -400,7 +415,7 @@ class ItemMapperAfterTest extends MapperTestUtility
                 ['items.last_modified >= :updatedSince'],
                 ['feeds.deleted_at = 0'],
                 ['feeds.user_id = :userId'],
-                ['items.starred = 1']
+                ['items.starred = :starred']
             )
             ->will($this->returnSelf());
 
@@ -412,6 +427,11 @@ class ItemMapperAfterTest extends MapperTestUtility
             ])
             ->will($this->returnSelf());
 
+        $this->builder->expects($this->exactly(1))
+            ->method('setParameter')
+            ->with('starred', true)
+            ->will($this->returnSelf());
+
         $this->builder->expects($this->once())
             ->method('orderBy')
             ->with('items.last_modified', 'DESC')
diff --git a/tests/Unit/Db/ItemMapperPaginatedTest.php b/tests/Unit/Db/ItemMapperPaginatedTest.php
index 93521f541..448622280 100644
--- a/tests/Unit/Db/ItemMapperPaginatedTest.php
+++ b/tests/Unit/Db/ItemMapperPaginatedTest.php
@@ -215,13 +215,13 @@ class ItemMapperPaginatedTest extends MapperTestUtility
                 ['feeds.user_id = :userId'],
                 ['feeds.deleted_at = 0'],
                 ['items.id < :offset'],
-                ['items.unread = 1']
+                ['items.unread = :unread']
             )
             ->will($this->returnSelf());
 
-        $this->builder->expects($this->exactly(2))
+        $this->builder->expects($this->exactly(3))
             ->method('setParameter')
-            ->withConsecutive(['userId', 'jack'], ['offset', 10])
+            ->withConsecutive(['userId', 'jack'], ['offset', 10], ['unread', true])
             ->will($this->returnSelf());
 
         $this->builder->expects($this->exactly(1))
@@ -286,13 +286,13 @@ class ItemMapperPaginatedTest extends MapperTestUtility
                 ['feeds.user_id = :userId'],
                 ['feeds.deleted_at = 0'],
                 ['items.id < :offset'],
-                ['items.unread = 1']
+                ['items.unread = :unread']
             )
             ->will($this->returnSelf());
 
-        $this->builder->expects($this->exactly(2))
+        $this->builder->expects($this->exactly(3))
             ->method('setParameter')
-            ->withConsecutive(['userId', 'jack'], ['offset', 10])
+            ->withConsecutive(['userId', 'jack'], ['offset', 10], ['unread', true])
             ->will($this->returnSelf());
 
         $this->builder->expects($this->never())
@@ -355,13 +355,13 @@ class ItemMapperPaginatedTest extends MapperTestUtility
                 ['feeds.user_id = :userId'],
                 ['feeds.deleted_at = 0'],
                 ['items.id < :offset'],
-                ['items.starred = 1']
+                ['items.starred = :starred']
             )
             ->will($this->returnSelf());
 
-        $this->builder->expects($this->exactly(2))
+        $this->builder->expects($this->exactly(3))
             ->method('setParameter')
-            ->withConsecutive(['userId', 'jack'], ['offset', 10])
+            ->withConsecutive(['userId', 'jack'], ['offset', 10], ['starred', true])
             ->will($this->returnSelf());
 
 
@@ -432,13 +432,19 @@ class ItemMapperPaginatedTest extends MapperTestUtility
                 ['items.search_index LIKE :term0'],
                 ['items.search_index LIKE :term1'],
                 ['items.id < :offset'],
-                ['items.starred = 1']
+                ['items.starred = :starred']
             )
             ->will($this->returnSelf());
 
-        $this->builder->expects($this->exactly(4))
+        $this->builder->expects($this->exactly(5))
             ->method('setParameter')
-            ->withConsecutive(['userId', 'jack'], ['term0', '%key%'], ['term1', '%word%'], ['offset', 10])
+            ->withConsecutive(
+                ['userId', 'jack'],
+                ['term0', '%key%'],
+                ['term1', '%word%'],
+                ['offset', 10],
+                ['starred', true]
+            )
             ->will($this->returnSelf());
 
 
@@ -721,13 +727,13 @@ class ItemMapperPaginatedTest extends MapperTestUtility
                 ['feeds.user_id = :userId'],
                 ['items.feed_id = :feedId'],
                 ['items.id < :offset'],
-                ['items.unread = 1']
+                ['items.unread = :unread']
             )
             ->will($this->returnSelf());
 
-        $this->builder->expects($this->exactly(3))
+        $this->builder->expects($this->exactly(4))
             ->method('setParameter')
-            ->withConsecutive(['userId', 'jack'], ['feedId', 2], ['offset', 10])
+            ->withConsecutive(['userId', 'jack'], ['feedId', 2], ['offset', 10], ['unread', true])
             ->will($this->returnSelf());
 
 
@@ -1059,13 +1065,13 @@ class ItemMapperPaginatedTest extends MapperTestUtility
                 ['feeds.deleted_at = 0'],
                 ['x IS NULL'],
                 ['items.id < :offset'],
-                ['items.unread = 1']
+                ['items.unread = :unread']
             )
             ->will($this->returnSelf());
 
-        $this->builder->expects($this->exactly(2))
+        $this->builder->expects($this->exactly(3))
             ->method('setParameter')
-            ->withConsecutive(['userId', 'jack'], ['offset', 10])
+            ->withConsecutive(['userId', 'jack'], ['offset', 10], ['unread', true])
             ->will($this->returnSelf());
 
 
@@ -1145,13 +1151,13 @@ class ItemMapperPaginatedTest extends MapperTestUtility
                 ['feeds.deleted_at = 0'],
                 ['x IS NULL'],
                 ['items.id > :offset'],
-                ['items.unread = 1']
+                ['items.unread = :unread']
             )
             ->will($this->returnSelf());
 
-        $this->builder->expects($this->exactly(2))
+        $this->builder->expects($this->exactly(3))
             ->method('setParameter')
-            ->withConsecutive(['userId', 'jack'], ['offset', 10])
+            ->withConsecutive(['userId', 'jack'], ['offset', 10], ['unread', true])
             ->will($this->returnSelf());