Remove useless cell methods, sidebar_for_node_recent_topics cache key use node_id.

Author: huacnlee

app/cells/topics/high_likes_topics.html.erb

@@ -38,8 +38,8 @@ def sidebar_suggest_topics
   end
 
   # 节点下面的最新话题
-  cache :sidebar_for_node_recent_topics, :expires_in => 20.minutes do |cell, args|
-    args[:topic].id
+  cache :sidebar_for_node_recent_topics, :expires_in => 30.minutes do |cell, args|
+    ['node',args[:topic].node_id].join("-")
   end
   def sidebar_for_node_recent_topics(args = {})
     topic = args[:topic]
@@ -60,18 +60,6 @@ def index_locations
     render
   end
 
-  cache :high_likes_topics, :expires_in => 3.hours
-  def high_likes_topics
-    @topics = Topic.by_week.high_likes.limit(10)
-    render
-  end
-
-  cache :high_replies_topics, :expires_in => 3.hours
-  def high_replies_topics
-    @topics = Topic.by_week.high_replies.limit(10)
-    render
-  end
-
   def tips
     @tip = ""
     if !SiteConfig.tips.blank?

app/cells/topics/high_replies_topics.html.erb

@@ -50,26 +50,6 @@
     end
   end
 
-  describe "high topics" do
-    before(:each) do
-      t1 = Factory(:topic)
-      t2 = Factory(:topic)
-      t3 = Factory(:topic)
-    end
-
-    it "should render high_likes_topics" do
-      count = Topic.by_week.count
-      count = 10 if count > 10
-      render_cell(:topics, :high_likes_topics).should have_css('div.high_likes_topics li', :count => count)
-    end
-
-    it "should render high_replies_topics" do
-      count = Topic.by_week.count
-      count = 10 if count > 10
-      render_cell(:topics, :high_replies_topics).should have_css('div.high_replies_topics li', :count => count)
-    end
-  end
-
   describe "sidebar_for_node_recent_topics" do
     let(:node) { Factory(:node) }
     let(:topic0) { Factory(:topic, :node => node) }