FEATURE: add redirect_users_to_top_page site setting (default to true)

app/models/user.rb

@@ -560,6 +560,8 @@ class User < ActiveRecord::Base
   end
 
   def redirected_to_top_reason
+    # redirect is enabled
+    return unless SiteSetting.redirect_users_to_top_page
     # top must be in the top_menu
     return unless SiteSetting.top_menu =~ /top/i
     # there should be enough topics

config/locales/server.en.yml

@@ -665,6 +665,7 @@ en:
 
     topics_per_period_in_top_summary: "How many topics loaded on the top topics summary"
     topics_per_period_in_top_page: "How many topics loaded on the top topics page"
+    redirect_users_to_top_page: "Automatically redirect new & long-time-no-see users to top page"
     redirect_new_users_to_top_page_duration: "Number of days during which new users are automatically redirect to the top page"
 
     enable_badges: "Enable the badge system (experimental)"

config/site_settings.yml

@@ -148,6 +148,7 @@ users:
     client: true
     default: 60
   delete_all_posts_max: 15
+  redirect_users_to_top_page: true
 
 posting:
   min_post_length:

spec/models/user_spec.rb

@@ -1093,6 +1093,14 @@ describe User do
   describe "redirected_to_top_reason" do
     let!(:user) { Fabricate(:user) }
 
+    it "should have no reason when redirect_users_to_top_page is disabled" do
+      SiteSetting.expects(:redirect_users_to_top_page).returns(false)
+      user.redirected_to_top_reason.should == nil
+    end
+
+    context "redirect_users_to_top_page is enabled" do
+      before { SiteSetting.stubs(:redirect_users_to_top_page).returns(true) }
+
       it "should have no reason when top is not in the top_menu" do
         SiteSetting.expects(:top_menu).returns("latest")
         user.redirected_to_top_reason.should == nil
@@ -1140,3 +1148,5 @@ describe User do
     end
 
   end
+
+end