diff options
author | Bernhard Posselt <nukeawhale@gmail.com> | 2013-03-23 16:18:43 +0100 |
---|---|---|
committer | Bernhard Posselt <nukeawhale@gmail.com> | 2013-03-23 16:18:43 +0100 |
commit | ecc4a533ccf0c56527b4aaaa94a22a0dddf29fa2 (patch) | |
tree | 72955c4a30595c4a8f20fbb1672c441a553df4f1 | |
parent | d65fa8090469286396342ff2299b58b79d83f8f3 (diff) |
use all articles if active feed is null
-rw-r--r-- | controller/feedcontroller.php | 4 | ||||
-rw-r--r-- | js/app/controllers/controllers.coffee | 6 | ||||
-rw-r--r-- | js/app/controllers/itemcontroller.coffee | 5 | ||||
-rw-r--r-- | templates/main.php | 2 | ||||
-rw-r--r-- | tests/controller/FeedControllerTest.php | 19 |
5 files changed, 31 insertions, 5 deletions
diff --git a/controller/feedcontroller.php b/controller/feedcontroller.php index 27ab07408..1a29731b4 100644 --- a/controller/feedcontroller.php +++ b/controller/feedcontroller.php @@ -81,8 +81,12 @@ class FeedController extends Controller { try { if($feedType === FeedType::FOLDER){ $this->folderBl->find($feedId, $userId); + } elseif ($feedType === FeedType::FEED){ $this->feedBl->find($feedId, $userId); + + } elseif($feedType === null || $feedId === null){ + throw new BLException(''); } } catch (BLException $ex){ $feedId = 0; diff --git a/js/app/controllers/controllers.coffee b/js/app/controllers/controllers.coffee index 55566f7d0..b3a246fda 100644 --- a/js/app/controllers/controllers.coffee +++ b/js/app/controllers/controllers.coffee @@ -36,8 +36,8 @@ angular.module('News').controller 'FeedController', ] angular.module('News').controller 'ItemController', -['$scope', '_ItemController', 'ItemModel', -($scope, _ItemController, ItemModel)-> +['$scope', '_ItemController', 'ItemModel', 'FeedLoading', +($scope, _ItemController, ItemModel, FeedLoading)-> - return new _ItemController($scope, ItemModel) + return new _ItemController($scope, ItemModel, FeedLoading) ]
\ No newline at end of file diff --git a/js/app/controllers/itemcontroller.coffee b/js/app/controllers/itemcontroller.coffee index 3350c674e..163d652ca 100644 --- a/js/app/controllers/itemcontroller.coffee +++ b/js/app/controllers/itemcontroller.coffee @@ -25,7 +25,10 @@ angular.module('News').factory '_ItemController', -> class ItemController - constructor: (@$scope, @itemModel) -> + constructor: (@$scope, @itemModel, @feedLoading) -> + + @$scope.isLoading = => + return @feedLoading.isLoading() return ItemController
\ No newline at end of file diff --git a/templates/main.php b/templates/main.php index 422259c64..9690e3cbe 100644 --- a/templates/main.php +++ b/templates/main.php @@ -47,7 +47,7 @@ </div> - <div id="right-content" ng-class="{loading: loading.loading>0}" + <div id="right-content" ng-class="{loading: isLoading()}" ng-controller="ItemController" when-scrolled="scroll()" feed-navigation> <?php print_unescaped($this->inc("part.items")); ?> </div> diff --git a/tests/controller/FeedControllerTest.php b/tests/controller/FeedControllerTest.php index 6adae204c..4ccb7bed6 100644 --- a/tests/controller/FeedControllerTest.php +++ b/tests/controller/FeedControllerTest.php @@ -215,6 +215,25 @@ class FeedControllerTest extends ControllerTestUtility { } + public function testActiveActiveIsNull(){ + $id = 3; + $type = null; + $result = array( + 'activeFeed' => array( + 'id' => 0, + 'type' => FeedType::SUBSCRIPTIONS + ) + ); + + $this->activeInitMocks($id, $type); + + $response = $this->controller->active(); + + $this->assertEquals($result, $response->getParams()); + $this->assertTrue($response instanceof JSONResponse); + } + + public function testCreate(){ $result = array( 'feeds' => array(new Feed()) |