go back to order by id, fix #138, use a newest item id to prevent marking items as read that the user didnt see yet fix #141, also update the starred count periodically

1 files changed, 3 insertions, 23 deletions

diff --git a/controller/itemcontroller.php b/controller/itemcontroller.php
index 459435a7e..a4d9abf3f 100644
--- a/controller/itemcontroller.php
+++ b/controller/itemcontroller.php
@@ -61,7 +61,6 @@ class ItemController extends Controller {
 		$type = (int) $this->params('type');
 		$id = (int) $this->params('id');
 		$offset = (int) $this->params('offset', 0);
-		$newestItemId = (int) $this->params('newestItemId');
 
 		$this->api->setUserValue('lastViewedFeedId', $id);
 		$this->api->setUserValue('lastViewedFeedType', $type);
@@ -76,14 +75,12 @@ class ItemController extends Controller {
 			if($offset === 0) {
 				$params['newestItemId'] = 
 					$this->itemBusinessLayer->getNewestItemId($userId);
-				$newestItemId = $params['newestItemId'];
 				$params['feeds'] = $this->feedBusinessLayer->findAll($userId);
+				$params['starred'] = $this->itemBusinessLayer->starredCount($userId);
 			}
 						
-			$params['items'] = $this->itemBusinessLayer->findAll(
-				                                       $id, $type, $limit, 
-				                                       $offset, $newestItemId, 
-				                                       $showAll, $userId);
+			$params['items'] = $this->itemBusinessLayer->findAll($id, $type, $limit, 
+				                                       $offset, $showAll, $userId);
 		// this gets thrown if there are no items
 		// in that case just return an empty array
 		} catch(BusinessLayerException $ex) {}
@@ -92,23 +89,6 @@ class ItemController extends Controller {
 	}
 
 
-	/**
-	 * @IsAdminExemption
-	 * @IsSubAdminExemption
-	 * @Ajax
-	 */
-	public function starred(){
-		$userId = $this->api->getUserId();
-		$starredCount = $this->itemBusinessLayer->starredCount($userId);
-
-		$params = array(
-			'starred' => (int) $starredCount
-		);
-
-		return $this->renderJSON($params);
-	}
-
-
 	private function setStarred($isStarred){
 		$userId = $this->api->getUserId();
 		$feedId = (int) $this->params('feedId');