1 files changed, 7 insertions, 9 deletions
diff --git a/js/controller/ContentController.js b/js/controller/ContentController.js
index ed8d0be55..94098d6c8 100644
--- a/js/controller/ContentController.js
+++ b/js/controller/ContentController.js
@@ -18,7 +18,7 @@ app.controller('ContentController', function (Publisher, FeedResource, ItemResou
     // distribute data to models based on key
     Publisher.publishAll(data);
 
-    var getOrdering = function () {
+    var isOldestFirst = function () {
         var ordering = SettingsResource.get('oldestFirst');
 
         if (self.isFeed()) {
@@ -35,13 +35,11 @@ app.controller('ContentController', function (Publisher, FeedResource, ItemResou
 
     this.getFirstItem = function () {
         var orderedItems = this.getItems();
-        var item = orderedItems[orderedItems.length - 1];
-        var firstItem = orderedItems[0];
-        // If getOrdering == 1, then the sorting is set to
-        // newest first. So, item should be the first item
-        //
-        if (getOrdering()) {
-            item = firstItem;
+        var item = orderedItems[0];
+        var lastItem = orderedItems[orderedItems.length - 1];
+        // If isOldestFirst is set, item should be the last item
+        if (isOldestFirst()) {
+            item = lastItem;
         }
         if (item === undefined) {
             return undefined;
@@ -152,7 +150,7 @@ app.controller('ContentController', function (Publisher, FeedResource, ItemResou
         return $route.current.$$route.type === FEED_TYPE.FEED;
     };
 
-    this.oldestFirst = getOrdering();
+    this.oldestFirst = isOldestFirst();
 
     this.autoPage = function () {
         if (this.isNothingMoreToAutoPage) {