Fix a case when the wrong topic is loaded because the slug starts with a number

app/controllers/list_controller.rb

@@ -111,8 +111,8 @@ class ListController < ApplicationController
   private
 
   def set_category
-    category_slug = params.fetch(:category)
-    @category = Category.where("slug = ? or id = ?", category_slug, category_slug.to_i).includes(:featured_users).first
+    slug = params.fetch(:category)
+    @category = Category.where("slug = ?", slug).includes(:featured_users).first || Category.where("id = ?", slug.to_i).includes(:featured_users).first
   end
 
   def request_is_for_uncategorized?

spec/controllers/list_controller_spec.rb

@@ -63,6 +63,21 @@ describe ListController do
         it { should respond_with(:success) }
       end
 
+      context 'another category exists with a number at the beginning of its name' do
+        # One category has another category's id at the beginning of its name
+        let!(:other_category) { Fabricate(:category, name: "#{category.id} name") }
+
+        before do
+          xhr :get, :category, category: other_category.slug
+        end
+
+        it { should respond_with(:success) }
+
+        it 'uses the correct category' do
+          assigns(:category).should == other_category
+        end
+      end
+
       describe 'feed' do
         it 'renders RSS' do
           get :category_feed, category: category.slug, format: :rss