diff options
author | Bernhard Posselt <dev@bernhard-posselt.com> | 2014-04-03 23:53:30 +0200 |
---|---|---|
committer | Bernhard Posselt <dev@bernhard-posselt.com> | 2014-04-03 23:53:42 +0200 |
commit | 4ab9494fc90d9131afd50048369b92c2e47a9b6a (patch) | |
tree | 1066a874b9fed2ab6f646349a0aafbf686321ee3 /js | |
parent | 2abf47a533e56d42262e5061b8ffa9234129b9c9 (diff) |
only instantiate one itemcontroller, prevent tons of requests when autopaging, fix #498
Diffstat (limited to 'js')
-rw-r--r-- | js/app/controllers/feedcontroller.coffee | 8 | ||||
-rw-r--r-- | js/app/controllers/itemcontroller.coffee | 10 | ||||
-rw-r--r-- | js/public/app.js | 20 |
3 files changed, 16 insertions, 22 deletions
diff --git a/js/app/controllers/feedcontroller.coffee b/js/app/controllers/feedcontroller.coffee index ee8b9375f..43186e689 100644 --- a/js/app/controllers/feedcontroller.coffee +++ b/js/app/controllers/feedcontroller.coffee @@ -47,6 +47,14 @@ ActiveFeed, FeedType, $window) -> @_$scope.starredBusinessLayer = @_starredBusinessLayer @_$scope.unreadCountFormatter = @_unreadCountFormatter + @_$scope.edit = (feed) -> + feed.editing = true + feed.originalValue = feed.title + + @_$scope.cancel = (feed) -> + feed.editing = false + feed.title = feed.originalValue + @_$scope.getTotalUnreadCount = => # also update title based on unreadcount count = @_subscriptionsBusinessLayer.getUnreadCount(0) diff --git a/js/app/controllers/itemcontroller.coffee b/js/app/controllers/itemcontroller.coffee index ee2fa7e82..f381baede 100644 --- a/js/app/controllers/itemcontroller.coffee +++ b/js/app/controllers/itemcontroller.coffee @@ -37,16 +37,6 @@ Language, AutoPageLoading, Compact) -> @_$scope.itemBusinessLayer = @_itemBusinessLayer @_$scope.feedBusinessLayer = @_feedBusinessLayer - @_$scope.edit = (feedId) => - feed = @_feedModel.getById(feedId) - feed.editing = true - feed.originalValue = feed.title - - @_$scope.cancel = (feedId) => - feed = @_feedModel.getById(feedId) - feed.editing = false - feed.title = feed.originalValue - @_$scope.isLoading = => return @_feedLoading.isLoading() diff --git a/js/public/app.js b/js/public/app.js index e4dd8880c..57306e34a 100644 --- a/js/public/app.js +++ b/js/public/app.js @@ -744,6 +744,14 @@ License along with this library. If not, see <http://www.gnu.org/licenses/>. this._$scope.subscriptionsBusinessLayer = this._subscriptionsBusinessLayer; this._$scope.starredBusinessLayer = this._starredBusinessLayer; this._$scope.unreadCountFormatter = this._unreadCountFormatter; + this._$scope.edit = function(feed) { + feed.editing = true; + return feed.originalValue = feed.title; + }; + this._$scope.cancel = function(feed) { + feed.editing = false; + return feed.title = feed.originalValue; + }; this._$scope.getTotalUnreadCount = function() { var appName, count, title, titleCount; count = _this._subscriptionsBusinessLayer.getUnreadCount(0); @@ -873,18 +881,6 @@ License along with this library. If not, see <http://www.gnu.org/licenses/>. this._autoPaging = true; this._$scope.itemBusinessLayer = this._itemBusinessLayer; this._$scope.feedBusinessLayer = this._feedBusinessLayer; - this._$scope.edit = function(feedId) { - var feed; - feed = _this._feedModel.getById(feedId); - feed.editing = true; - return feed.originalValue = feed.title; - }; - this._$scope.cancel = function(feedId) { - var feed; - feed = _this._feedModel.getById(feedId); - feed.editing = false; - return feed.title = feed.originalValue; - }; this._$scope.isLoading = function() { return _this._feedLoading.isLoading(); }; |