Fix #791

1 files changed, 178 insertions, 156 deletions

diff --git a/js/controller/ContentController.js b/js/controller/ContentController.js
index dfee837d5..c63fc65e8 100644
--- a/js/controller/ContentController.js
+++ b/js/controller/ContentController.js
@@ -8,197 +8,219 @@
  * @copyright Bernhard Posselt 2014
  */
 app.controller('ContentController',
-function (Publisher, FeedResource, ItemResource, SettingsResource, data,
-    $route, $routeParams, $location, FEED_TYPE, ITEM_AUTO_PAGE_SIZE, Loading) {
-    'use strict';
-
-    ItemResource.clear();
-
-    // distribute data to models based on key
-    Publisher.publishAll(data);
+    function (Publisher, FeedResource, ItemResource, SettingsResource, data,
+              $route, $routeParams, $location, FEED_TYPE, ITEM_AUTO_PAGE_SIZE,
+              Loading, $filter) {
+        'use strict';
 
+        var self = this;
+        ItemResource.clear();
 
-    this.isAutoPagingEnabled = true;
+        // distribute data to models based on key
+        Publisher.publishAll(data);
 
-    // the interface should show a hint if there are not enough items sent so
-    // it's assumed that theres nothing to autpage
-    if (ItemResource.size() >= ITEM_AUTO_PAGE_SIZE) {
-        this.isNothingMoreToAutoPage = false;
-    } else {
-        this.isNothingMoreToAutoPage = true;
-    }
+        this.getFirstItem = function () {
+            var orderFilter = $filter('orderBy');
+            var orderedItems = orderFilter(this.getItems(), this.orderBy());
+            var firstItem = orderedItems[0];
+            if (firstItem === undefined) {
+                return undefined;
+            } else {
+                return firstItem.id;
+            }
+        };
 
-    this.getItems = function () {
-        return ItemResource.getAll();
-    };
 
-    this.toggleStar = function (itemId) {
-        ItemResource.toggleStar(itemId);
-    };
+        this.isAutoPagingEnabled = true;
+        // the interface should show a hint if there are not enough items sent
+        // it's assumed that theres nothing to autpage
 
-    this.toggleItem = function (item) {
-        // TODO: unittest
-        if (this.isCompactView()) {
-            item.show = !item.show;
+        if (ItemResource.size() >= ITEM_AUTO_PAGE_SIZE) {
+            this.isNothingMoreToAutoPage = false;
+        } else {
+            this.isNothingMoreToAutoPage = true;
         }
-    };
 
-    this.isShowAll = function () {
-        return SettingsResource.get('showAll');
-    };
+        this.getItems = function () {
+            return ItemResource.getAll();
+        };
 
-    this.markRead = function (itemId) {
-        var item = ItemResource.get(itemId);
+        this.isItemActive = function (id) {
+            return this.activeItem === id;
+        };
 
-        if (!item.keepUnread && item.unread === true) {
-            ItemResource.markItemRead(itemId);
-            FeedResource.markItemOfFeedRead(item.feedId);
-        }
-    };
+        this.setItemActive = function (id) {
+            this.activeItem = id;
+        };
 
-    this.getFeed = function (feedId) {
-        return FeedResource.getById(feedId);
-    };
+        this.toggleStar = function (itemId) {
+            ItemResource.toggleStar(itemId);
+        };
 
-    this.toggleKeepUnread = function (itemId) {
-        var item = ItemResource.get(itemId);
-        if (!item.unread) {
-            FeedResource.markItemOfFeedUnread(item.feedId);
-            ItemResource.markItemRead(itemId, false);
-        }
+        this.toggleItem = function (item) {
+            // TODO: unittest
+            if (this.isCompactView()) {
+                item.show = !item.show;
+            }
+        };
 
-        item.keepUnread = !item.keepUnread;
-    };
+        this.isShowAll = function () {
+            return SettingsResource.get('showAll');
+        };
 
-    var self = this;
-    var getOrdering = function () {
-        var ordering = SettingsResource.get('oldestFirst');
+        this.markRead = function (itemId) {
+            var item = ItemResource.get(itemId);
 
-        if (self.isFeed()) {
-            var feed = FeedResource.getById($routeParams.id);
-            if (feed && feed.ordering === 1) {
-                ordering = true;
-            } else if (feed && feed.ordering === 2) {
-                ordering = false;
+            if (!item.keepUnread && item.unread === true) {
+                ItemResource.markItemRead(itemId);
+                FeedResource.markItemOfFeedRead(item.feedId);
             }
-        }
-
-        return ordering;
-    };
+        };
 
-    this.orderBy = function () {
-        if (getOrdering()) {
-            return 'id';
-        } else {
-            return '-id';
-        }
-    };
+        this.getFeed = function (feedId) {
+            return FeedResource.getById(feedId);
+        };
 
-    this.isCompactView = function () {
-        return SettingsResource.get('compact');
-    };
+        this.toggleKeepUnread = function (itemId) {
+            var item = ItemResource.get(itemId);
+            if (!item.unread) {
+                FeedResource.markItemOfFeedUnread(item.feedId);
+                ItemResource.markItemRead(itemId, false);
+            }
 
-    this.isCompactExpand = function () {
-        return SettingsResource.get('compactExpand');
-    };
+            item.keepUnread = !item.keepUnread;
+        };
 
-    this.autoPagingEnabled = function () {
-        return this.isAutoPagingEnabled;
-    };
+        var getOrdering = function () {
+            var ordering = SettingsResource.get('oldestFirst');
 
-    this.markReadEnabled = function () {
-        return !SettingsResource.get('preventReadOnScroll');
-    };
+            if (self.isFeed()) {
+                var feed = FeedResource.getById($routeParams.id);
+                if (feed && feed.ordering === 1) {
+                    ordering = true;
+                } else if (feed && feed.ordering === 2) {
+                    ordering = false;
+                }
+            }
 
-    this.scrollRead = function (itemIds) {
-        var ids = [];
-        var feedIds = [];
+            return ordering;
+        };
 
-        itemIds.forEach(function (itemId) {
-            var item = ItemResource.get(itemId);
-            if (!item.keepUnread) {
-                ids.push(itemId);
-                feedIds.push(item.feedId);
+        this.orderBy = function () {
+            if (getOrdering()) {
+                return 'id';
+            } else {
+                return '-id';
+            }
+        };
+
+        this.isCompactView = function () {
+            return SettingsResource.get('compact');
+        };
+
+        this.isCompactExpand = function () {
+            return SettingsResource.get('compactExpand');
+        };
+
+        this.autoPagingEnabled = function () {
+            return this.isAutoPagingEnabled;
+        };
+
+        this.markReadEnabled = function () {
+            return !SettingsResource.get('preventReadOnScroll');
+        };
+
+        this.scrollRead = function (itemIds) {
+            var ids = [];
+            var feedIds = [];
+
+            itemIds.forEach(function (itemId) {
+                var item = ItemResource.get(itemId);
+                if (!item.keepUnread) {
+                    ids.push(itemId);
+                    feedIds.push(item.feedId);
+                }
+            });
+
+            if (ids.length > 0) {
+                FeedResource.markItemsOfFeedsRead(feedIds);
+                ItemResource.markItemsRead(ids);
             }
-        });
+        };
 
-        if (ids.length > 0) {
-            FeedResource.markItemsOfFeedsRead(feedIds);
-            ItemResource.markItemsRead(ids);
-        }
-    };
+        this.isFeed = function () {
+            return $route.current.$$route.type === FEED_TYPE.FEED;
+        };
 
-    this.isFeed = function () {
-        return $route.current.$$route.type === FEED_TYPE.FEED;
-    };
+        this.autoPage = function () {
+            if (this.isNothingMoreToAutoPage) {
+                return;
+            }
 
-    this.autoPage = function () {
-        if (this.isNothingMoreToAutoPage) {
-            return;
-        }
+            // in case a subsequent autopage request comes in wait until
+            // the current one finished and execute a request immediately
+            // afterwards
+            if (!this.isAutoPagingEnabled) {
+                this.autoPageAgain = true;
+                return;
+            }
 
-        // in case a subsequent autopage request comes in wait until
-        // the current one finished and execute a request immediately afterwards
-        if (!this.isAutoPagingEnabled) {
-            this.autoPageAgain = true;
-            return;
-        }
+            this.isAutoPagingEnabled = false;
+            this.autoPageAgain = false;
 
-        this.isAutoPagingEnabled = false;
-        this.autoPageAgain = false;
+            var type = $route.current.$$route.type;
+            var id = $routeParams.id;
+            var oldestFirst = getOrdering();
+            var showAll = SettingsResource.get('showAll');
+            var self = this;
+            var search = $location.search().search;
 
-        var type = $route.current.$$route.type;
-        var id = $routeParams.id;
-        var oldestFirst = getOrdering();
-        var showAll = SettingsResource.get('showAll');
-        var self = this;
-        var search = $location.search().search;
+            Loading.setLoading('autopaging', true);
 
-        Loading.setLoading('autopaging', true);
+            ItemResource.autoPage(type, id, oldestFirst, showAll, search)
+                .success(function (data) {
+                    Publisher.publishAll(data);
 
-        ItemResource.autoPage(type, id, oldestFirst, showAll, search)
-        .success(function (data) {
-            Publisher.publishAll(data);
+                    if (data.items.length >= ITEM_AUTO_PAGE_SIZE) {
+                        self.isAutoPagingEnabled = true;
+                    } else {
+                        self.isNothingMoreToAutoPage = true;
+                    }
 
-            if (data.items.length >= ITEM_AUTO_PAGE_SIZE) {
+                    if (self.isAutoPagingEnabled && self.autoPageAgain) {
+                        self.autoPage();
+                    }
+                }).error(function () {
                 self.isAutoPagingEnabled = true;
+            }).finally(function () {
+                Loading.setLoading('autopaging', false);
+            });
+        };
+
+        this.getRelativeDate = function (timestamp) {
+            if (timestamp !== undefined && timestamp !== '') {
+                var languageCode = SettingsResource.get('language');
+                var date =
+                    moment.unix(timestamp).locale(languageCode).fromNow() + '';
+                return date;
             } else {
-                self.isNothingMoreToAutoPage = true;
+                return '';
             }
+        };
 
-            if (self.isAutoPagingEnabled && self.autoPageAgain) {
-                self.autoPage();
-            }
-        }).error(function () {
-            self.isAutoPagingEnabled = true;
-        }).finally(function () {
-            Loading.setLoading('autopaging', false);
-        });
-    };
-
-    this.getRelativeDate = function (timestamp) {
-        if (timestamp !== undefined && timestamp !== '') {
-            var languageCode = SettingsResource.get('language');
-            var date =
-                moment.unix(timestamp).locale(languageCode).fromNow() + '';
-            return date;
-        } else {
-            return '';
-        }
-    };
-
-    this.refresh = function () {
-        $route.reload();
-    };
-
-    this.getMediaType = function (type) {
-        if (type && type.indexOf('audio') === 0) {
-            return 'audio';
-        } else if (type && type.indexOf('video') === 0) {
-            return 'video';
-        } else {
-            return undefined;
-        }
-    };
+        this.refresh = function () {
+            $route.reload();
+        };
 
-});
\ No newline at end of file
+        this.getMediaType = function (type) {
+            if (type && type.indexOf('audio') === 0) {
+                return 'audio';
+            } else if (type && type.indexOf('video') === 0) {
+                return 'video';
+            } else {
+                return undefined;
+            }
+        };
+        
+        this.activeItem = this.getFirstItem();
+    });
\ No newline at end of file