diff options
author | Bernhard Posselt <nukeawhale@gmail.com> | 2013-05-02 13:45:34 +0200 |
---|---|---|
committer | Bernhard Posselt <nukeawhale@gmail.com> | 2013-05-02 13:45:34 +0200 |
commit | c5dde50a7166f6e24776b680ea8da1c0e78a7543 (patch) | |
tree | 7ade52e5e6e929a87dfb9afca547d6cacc480dce | |
parent | 2dfab0dbec8a1868c2287aee7f7a93a56d9a301c (diff) |
implemented first feeds get all method
-rw-r--r-- | appinfo/routes.php | 74 | ||||
-rw-r--r-- | dependencyinjection/dicontainer.php | 12 | ||||
-rw-r--r-- | external/feed.php | 55 | ||||
-rw-r--r-- | external/feedapi.php | 77 | ||||
-rw-r--r-- | tests/unit/external/FeedAPITest.php | 137 |
5 files changed, 232 insertions, 123 deletions
diff --git a/appinfo/routes.php b/appinfo/routes.php index fd7b6ad20..7d1c087e4 100644 --- a/appinfo/routes.php +++ b/appinfo/routes.php @@ -206,74 +206,12 @@ $this->create('news_usersettings_language', '/usersettings/language')->get()->ac /** * Feed API */ - -\OCP\API::register( - 'get', '/news/feeds', - function($urlParams) { - $container = new DIContainer(); - return $container['FeedApi']->getAll($urlParams); - }, - 'news', \OC_API::USER_AUTH -); -\OCP\API::register( - 'get', '/news/feeds/{feedid}', - function($urlParams) { - $container = new DIContainer(); - return $container['FeedApi']->getById($urlParams); - }, - 'news', \OC_API::USER_AUTH -); -\OCP\API::register( - 'post', '/news/feeds/create', - function($urlParams) { - $container = new DIContainer(); - return $container['FeedApi']->create($urlParams); - }, - 'news', \OC_API::USER_AUTH -); -\OCP\API::register( - 'post', '/news/feeds/{feedid}/delete', - function($urlParams) { - $container = new DIContainer(); - return $container['FeedApi']->delete($urlParams); - }, - 'news', \OC_API::USER_AUTH -); - -/** - * Folder API - */ - -\OCP\API::register( - 'get', '/news/folders', - function($urlParams) { - $container = new DIContainer(); - return $container['FolderApi']->getAll($urlParams); - }, - 'news', \OC_API::USER_AUTH -); -\OCP\API::register( - 'post', '/news/folders/create', - function($urlParams) { - $container = new DIContainer(); - return $container['FolderApi']->create($urlParams); - }, - 'news', \OC_API::USER_AUTH -); - -\OCP\API::register( - 'get', '/news/folders/{folderid}/delete', - function($urlParams) { - $container = new DIContainer(); - return $container['FolderApi']->delete($urlParams); - }, - 'news', \OC_API::USER_AUTH -); -\OCP\API::register( - 'post', '/news/folders/{folderid}/modify', +\OCP\API::register('get', '/apps/news/feeds', function($urlParams) { $container = new DIContainer(); - return $container['FolderApi']->modify($urlParams); - }, - 'news', \OC_API::USER_AUTH + $response = $container['FeedAPI']->getAll($urlParams); + return \OC_OCS_Result($response); + }, + 'news', + \OC_API::USER_AUTH ); diff --git a/dependencyinjection/dicontainer.php b/dependencyinjection/dicontainer.php index db647e852..42b5b4238 100644 --- a/dependencyinjection/dicontainer.php +++ b/dependencyinjection/dicontainer.php @@ -43,6 +43,8 @@ use \OCA\News\Db\FeedMapper; use \OCA\News\Db\ItemMapper; use \OCA\News\Db\StatusFlag; +use \OCA\News\External\FeedAPI; + use \OCA\News\Utility\Fetcher; use \OCA\News\Utility\FeedFetcher; use \OCA\News\Utility\TwitterFetcher; @@ -150,6 +152,16 @@ class DIContainer extends BaseContainer { /** + * External API + */ + $this['FeedAPI'] = $this->share(function($c){ + return new FeedAPI($c['API'], + $c['FolderBusinessLayer'], + $c['FeedBusinessLayer'], + $c['ItemBusinessLayer']); + }); + + /** * Utility */ $this['Fetcher'] = $this->share(function($c){ diff --git a/external/feed.php b/external/feed.php deleted file mode 100644 index a56cd2253..000000000 --- a/external/feed.php +++ /dev/null @@ -1,55 +0,0 @@ -<?php - -namespace OCA\News; - -use \OCA\News\Controller\FeedController; - -class FeedApi { - - public function __construct($bl){ - $this->bl = $bl; - } - - public function getAll() { - $feeds = $this->bl->getAll(); - $serializedFeeds = array(); - foreach ($feeds as $feed) { - $serializedFeeds[] = $feed->jsonSerialize(); - } - return new \OC_OCS_Result($serializedFeeds); - } - - public function getById($params) { - $feed = $this->bl->getById($feedid); - $serializedFeed = array($feed->jsonSerialize()); - return new \OC_OCS_Result($serializedFeed); - } - - public function delete($params) { - //TODO: check parameters here - - $success = $this->bl->delete($params["feedid"]); - - if ($success) { - return new \OC_OCS_Result(); - } - else { - return new \OC_OCS_Result(null, 101); - } - } - - public function create() { - $url = $_POST['url']; - $folderId = $_POST['folderid']; - //TODO: check parameters here - - $success = $this->bl->create($url, $folderId); - - if ($success) { - return new \OC_OCS_Result(); - } - else { - return new \OC_OCS_Result(null, 101); - } - } -} diff --git a/external/feedapi.php b/external/feedapi.php new file mode 100644 index 000000000..219c8a893 --- /dev/null +++ b/external/feedapi.php @@ -0,0 +1,77 @@ +<?php + +/** +* ownCloud - News +* +* @author Alessandro Cosentino +* @author Bernhard Posselt +* @copyright 2012 Alessandro Cosentino cosenal@gmail.com +* @copyright 2012 Bernhard Posselt nukeawhale@gmail.com +* +* This library is free software; you can redistribute it and/or +* modify it under the terms of the GNU AFFERO GENERAL PUBLIC LICENSE +* License as published by the Free Software Foundation; either +* version 3 of the License, or any later version. +* +* This library is distributed in the hope that it will be useful, +* but WITHOUT ANY WARRANTY; without even the implied warranty of +* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +* GNU AFFERO GENERAL PUBLIC LICENSE for more details. +* +* You should have received a copy of the GNU Affero General Public +* License along with this library. If not, see <http://www.gnu.org/licenses/>. +* +*/ + +namespace OCA\News\External; + +use \OCA\AppFramework\Core\API; + +use \OCA\News\BusinessLayer\FeedBusinessLayer; +use \OCA\News\BusinessLayer\FolderBusinessLayer; +use \OCA\News\BusinessLayer\ItemBusinessLayer; +use \OCA\News\BusinessLayer\BusinessLayerException; + + +class FeedAPI { + + private $itemBusinessLayer; + private $feedBusinessLayer; + private $folderBusinessLayer; + private $api; + + public function __construct(API $api, + FolderBusinessLayer $folderBusinessLayer, + FeedBusinessLayer $feedBusinessLayer, + ItemBusinessLayer $itemBusinessLayer){ + $this->api = $api; + $this->folderBusinessLayer = $folderBusinessLayer; + $this->feedBusinessLayer = $feedBusinessLayer; + $this->itemBusinessLayer = $itemBusinessLayer; + } + + + public function getAll(array $urlParams=array()) { + + $userId = $this->api->getUserId(); + + $result = array( + 'feeds' => array(), + 'starredCount' => $this->itemBusinessLayer->starredCount($userId) + ); + + foreach ($this->feedBusinessLayer->findAll($userId) as $feed) { + array_push($result['feeds'], $feed->toAPI()); + } + + // check case when there are no items + try { + $result['newestItemId'] = + $this->itemBusinessLayer->getNewestItemId($userId); + } catch(BusinessLayerException $ex) {} + + return $result; + } + + +} diff --git a/tests/unit/external/FeedAPITest.php b/tests/unit/external/FeedAPITest.php new file mode 100644 index 000000000..8ceafd016 --- /dev/null +++ b/tests/unit/external/FeedAPITest.php @@ -0,0 +1,137 @@ +<?php + +/** +* ownCloud - News +* +* @author Alessandro Cosentino +* @author Bernhard Posselt +* @copyright 2012 Alessandro Cosentino cosenal@gmail.com +* @copyright 2012 Bernhard Posselt nukeawhale@gmail.com +* +* This library is free software; you can redistribute it and/or +* modify it under the terms of the GNU AFFERO GENERAL PUBLIC LICENSE +* License as published by the Free Software Foundation; either +* version 3 of the License, or any later version. +* +* This library is distributed in the hope that it will be useful, +* but WITHOUT ANY WARRANTY; without even the implied warranty of +* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +* GNU AFFERO GENERAL PUBLIC LICENSE for more details. +* +* You should have received a copy of the GNU Affero General Public +* License along with this library. If not, see <http://www.gnu.org/licenses/>. +* +*/ + +namespace OCA\News\External; + +use \OCA\News\BusinessLayer\BusinessLayerException; + +use \OCA\News\Db\Folder; +use \OCA\News\Db\Feed; +use \OCA\News\Db\Item; + +require_once(__DIR__ . "/../../classloader.php"); + + +class FeedAPITest extends \PHPUnit_Framework_TestCase { + + private $folderBusinessLayer; + private $feedBusinessLayer; + private $itemBusinessLayer; + private $feedAPI; + private $api; + private $user; + + protected function setUp() { + $this->api = $this->folderBusinessLayer = $this->getMockBuilder( + '\OCA\AppFramework\Core\API') + ->disableOriginalConstructor() + ->getMock(); + $this->folderBusinessLayer = $this->getMockBuilder( + '\OCA\News\BusinessLayer\FolderBusinessLayer') + ->disableOriginalConstructor() + ->getMock(); + $this->feedBusinessLayer = $this->getMockBuilder( + '\OCA\News\BusinessLayer\FeedBusinessLayer') + ->disableOriginalConstructor() + ->getMock(); + $this->itemBusinessLayer = $this->getMockBuilder( + '\OCA\News\BusinessLayer\ItemBusinessLayer') + ->disableOriginalConstructor() + ->getMock(); + $this->feedAPI = new FeedAPI( + $this->api, + $this->folderBusinessLayer, + $this->feedBusinessLayer, + $this->itemBusinessLayer + ); + $this->user = 'tom'; + } + + + public function testGetAll() { + $feeds = array( + new Feed() + ); + $starredCount = 3; + $newestItemId = 2; + + $this->api->expects($this->once()) + ->method('getUserId') + ->will($this->returnValue($this->user)); + $this->itemBusinessLayer->expects($this->once()) + ->method('starredCount') + ->with($this->equalTo($this->user)) + ->will($this->returnValue($starredCount)); + $this->itemBusinessLayer->expects($this->once()) + ->method('getNewestItemId') + ->with($this->equalTo($this->user)) + ->will($this->returnValue($newestItemId)); + $this->feedBusinessLayer->expects($this->once()) + ->method('findAll') + ->with($this->equalTo($this->user)) + ->will($this->returnValue($feeds)); + + $response = $this->feedAPI->getAll(); + + $this->assertEquals(array( + 'feeds' => array($feeds[0]->toAPI()), + 'starredCount' => $starredCount, + 'newestItemId' => $newestItemId + ), $response); + } + + + public function testGetAllNoNewestItemId() { + $feeds = array( + new Feed() + ); + $starredCount = 3; + + $this->api->expects($this->once()) + ->method('getUserId') + ->will($this->returnValue($this->user)); + $this->itemBusinessLayer->expects($this->once()) + ->method('starredCount') + ->with($this->equalTo($this->user)) + ->will($this->returnValue($starredCount)); + $this->itemBusinessLayer->expects($this->once()) + ->method('getNewestItemId') + ->with($this->equalTo($this->user)) + ->will($this->throwException(new BusinessLayerException(''))); + $this->feedBusinessLayer->expects($this->once()) + ->method('findAll') + ->with($this->equalTo($this->user)) + ->will($this->returnValue($feeds)); + + $response = $this->feedAPI->getAll(); + + $this->assertEquals(array( + 'feeds' => array($feeds[0]->toAPI()), + 'starredCount' => $starredCount, + ), $response); + } + + +}
\ No newline at end of file |