FIX: Pagination while sorting on column headers

app/assets/javascripts/discourse/controllers/user_topics_list_controller.js

@@ -11,10 +11,6 @@ Discourse.UserTopicsListController = Discourse.ObjectController.extend({
   actions: {
     loadMore: function() {
       this.get('model').loadMore();
-    },
-
-    changeSort: function() {
-      console.log('sort changed!');
     }
   }
 

app/assets/javascripts/discourse/models/topic_list.js

@@ -71,7 +71,7 @@ Discourse.TopicList = Discourse.Model.extend({
 
       topics.clear();
       topics.pushObjects(newTopics);
-      self.set('loaded', true);
+      self.setProperties({ loaded: true, more_topics_url: result.topic_list.more_topics_url });
     });
 
   }.observes('sortOrder.order', 'sortOrder.descending'),
@@ -82,26 +82,24 @@ Discourse.TopicList = Discourse.Model.extend({
     var moreUrl = this.get('more_topics_url');
     if (moreUrl) {
 
-      var topicList = this;
+      var self = this;
       this.set('loadingMore', true);
 
       return Discourse.ajax({url: moreUrl}).then(function (result) {
         var topicsAdded = 0;
         if (result) {
           // the new topics loaded from the server
-          var newTopics = Discourse.TopicList.topicsFrom(result);
+          var newTopics = Discourse.TopicList.topicsFrom(result),
-          var topics = topicList.get("topics");
+              topics = self.get("topics");
 
-          topicList.forEachNew(newTopics, function(t) {
+          self.forEachNew(newTopics, function(t) {
             t.set('highlight', topicsAdded++ === 0);
             topics.pushObject(t);
           });
 
-          topicList.set('more_topics_url', result.topic_list.more_topics_url);
+          self.setProperties({ loadingMore: false, more_topics_url: result.topic_list.more_topics_url });
-          Discourse.Session.currentProp('topicList', topicList);
+          Discourse.Session.currentProp('topicList', self);
-          topicList.set('loadingMore', false);
+          return self.get('more_topics_url');
-
-          return result.topic_list.more_topics_url;
         }
       });
     } else {

app/controllers/list_controller.rb

@@ -58,9 +58,9 @@ class ListController < ApplicationController
     @description = @category.description
 
     if params[:parent_category].present?
-      list.more_topics_url = url_for(category_list_parent_path(params[:parent_category], params[:category], page: next_page, format: "json"))
+      list.more_topics_url = url_for(category_list_parent_path(params[:parent_category], params[:category], next_page_params))
     else
-      list.more_topics_url = url_for(category_list_path(params[:category], page: next_page, format: "json"))
+      list.more_topics_url = url_for(category_list_path(params[:category], next_page_params))
     end
 
     respond(list)
@@ -108,8 +108,12 @@ class ListController < ApplicationController
     end
   end
 
-  def next_page
+  def next_page_params(opts=nil)
-    params[:page].to_i + 1
+    opts = opts || {}
+    route_params = { format: 'json', page: params[:page].to_i + 1 }
+    route_params[:sort_order] = opts[:sort_order] if opts[:sort_order].present?
+    route_params[:sort_descending] = opts[:sort_descending] if opts[:sort_descending].present?
+    route_params
   end
 
   private
@@ -155,12 +159,11 @@ class ListController < ApplicationController
   end
 
   def generate_list_for(action, target_user, opts)
-    list = TopicQuery.new(current_user, opts)
+    TopicQuery.new(current_user, opts).send("list_#{action}", target_user)
-    list = list.send("list_#{action}", target_user)
   end
 
   def construct_url_with(action, opts, url_prefix=nil)
     method = url_prefix.blank? ? "#{action}_path" : "#{url_prefix}_#{action}_path"
-    public_send(method, opts.merge(format: 'json', page: next_page))
+    public_send(method, opts.merge(next_page_params(opts)))
   end
 end