1 files changed, 10 insertions, 19 deletions

diff --git a/lib/Service/ItemService.php b/lib/Service/ItemService.php
index f657b3eff..fee00ffc2 100644
--- a/lib/Service/ItemService.php
+++ b/lib/Service/ItemService.php
@@ -13,11 +13,11 @@
 
 namespace OCA\News\Service;
 
+use OCA\News\Db\Item;
 use OCP\IConfig;
 use OCP\AppFramework\Db\DoesNotExistException;
 
 use OCA\News\Db\ItemMapper;
-use OCA\News\Db\StatusFlag;
 use OCA\News\Db\FeedType;
 use OCA\News\Config\Config;
 use OCA\News\Utility\Time;
@@ -25,19 +25,16 @@ use OCA\News\Utility\Time;
 
 class ItemService extends Service {
 
-    private $statusFlag;
     private $config;
     private $timeFactory;
     private $itemMapper;
     private $systemConfig;
 
     public function __construct(ItemMapper $itemMapper,
-                                StatusFlag $statusFlag,
                                 Time $timeFactory,
                                 Config $config,
                                 IConfig $systemConfig){
         parent::__construct($itemMapper);
-        $this->statusFlag = $statusFlag;
         $this->config = $config;
         $this->timeFactory = $timeFactory;
         $this->itemMapper = $itemMapper;
@@ -56,20 +53,18 @@ class ItemService extends Service {
      * @return array of items
      */
     public function findAllNew($id, $type, $updatedSince, $showAll, $userId){
-        $status = $this->statusFlag->typeToStatus($type, $showAll);
-
         switch($type){
             case FeedType::FEED:
                 return $this->itemMapper->findAllNewFeed(
-                    $id, $updatedSince, $status, $userId
+                    $id, $updatedSince, $showAll, $userId
                 );
             case FeedType::FOLDER:
                 return $this->itemMapper->findAllNewFolder(
-                    $id, $updatedSince, $status, $userId
+                    $id, $updatedSince, $showAll, $userId
                 );
             default:
                 return $this->itemMapper->findAllNew(
-                    $updatedSince, $status, $userId
+                    $updatedSince, $type, $showAll, $userId
                 );
         }
     }
@@ -90,22 +85,20 @@ class ItemService extends Service {
      */
     public function findAll($id, $type, $limit, $offset, $showAll, $oldestFirst,
                             $userId, $search=[]){
-        $status = $this->statusFlag->typeToStatus($type, $showAll);
-
         switch($type){
             case FeedType::FEED:
                 return $this->itemMapper->findAllFeed(
-                    $id, $limit, $offset, $status, $oldestFirst, $userId,
+                    $id, $limit, $offset, $showAll, $oldestFirst, $userId,
                     $search
                 );
             case FeedType::FOLDER:
                 return $this->itemMapper->findAllFolder(
-                    $id, $limit, $offset, $status, $oldestFirst, $userId,
+                    $id, $limit, $offset, $showAll, $oldestFirst, $userId,
                     $search
                 );
             default:
                 return $this->itemMapper->findAll(
-                    $limit, $offset, $status, $oldestFirst, $userId, $search
+                    $limit, $offset, $type, $showAll, $oldestFirst, $userId, $search
                 );
         }
     }
@@ -122,15 +115,13 @@ class ItemService extends Service {
      */
     public function star($feedId, $guidHash, $isStarred, $userId){
         try {
+            /** @var Item $item */
             $item = $this->itemMapper->findByGuidHash(
                 $guidHash, $feedId, $userId
             );
 
-            if($isStarred){
-                $item->setStarred();
-            } else {
-                $item->setUnstarred();
-            }
+            $item->setStarred($isStarred);
+
             $this->itemMapper->update($item);
         } catch(DoesNotExistException $ex) {
             throw new ServiceNotFoundException($ex->getMessage());