讨论区局部优化

app/controllers/messages_controller.rb

@@ -37,16 +37,6 @@ class MessagesController < ApplicationController
 
   # Show a topic and its replies
   def show
-=begin
-    if @course
-      topic_id = params[:r]?params[:r]:params[:id]
-      parent_id = params[:id]
-      url = course_boards_path(@course,:topic_id => topic_id,:parent_id=>parent_id);
-      redirect_to url
-      return;
-    end
-=end
-
     @isReply = true
     page = params[:page]
     # Find the page of the requested reply
@@ -54,16 +44,7 @@ class MessagesController < ApplicationController
       offset = @topic.children.count(:conditions => ["#{Message.table_name}.id < ?", params[:r].to_i])
       page = 1 + offset / REPLIES_PER_PAGE
     end
-
     @reply_count = @topic.children.count
-    # @reply_pages = Paginator.new @reply_count, REPLIES_PER_PAGE, page
-    # @replies =  @topic.children.
-    #   includes(:author, :attachments, {:board => :project}).
-    #   reorder("#{Message.table_name}.created_on DESC").
-    #   limit(@reply_pages.per_page).
-    #   offset(@reply_pages.offset).
-    #   all
-
     @reply = Message.new(:subject => "RE: #{@message.subject}")
     if @course
       messages_replies =  @topic.children.

app/views/boards/_course_show_detail.html.erb

@@ -2,20 +2,7 @@
     <%= import_ke(enable_at: false, prettify: false) %>
     <%= javascript_include_tag "create_kindeditor" %>
 <% end %>
-
+<% if topics %>
-<style type="text/css">
-    /*回复框*/
-    /*.homepagePostReplyInputContainer .ke-toolbar {display: none; width: 400px; border: none; background: none; padding: 0px 0px;}*/
-    /*.homepagePostReplyInputContainer .ke-toolbar-icon {line-height: 26px; font-size: 14px; padding-left: 26px;}*/
-    /*.homepagePostReplyInputContainer .ke-toolbar-icon-url {background-image: url(/images/public_icon.png)}*/
-    /*.homepagePostReplyInputContainer .ke-outline {padding: 0px 0px; line-height: 26px; font-size: 14px;}*/
-    /*.homepagePostReplyInputContainer .ke-icon-emoticons {background-position: 0px -671px; width: 50px; height: 26px;}*/
-    /*.homepagePostReplyInputContainer .ke-icon-emoticons:hover {background-position: -79px -671px; width: 50px; height: 26px;}*/
-    /*.homepagePostReplyInputContainer .ke-outline {border: none;}*/
-    /*.homepagePostReplyInputContainer .ke-inline-block {display: none;}*/
-    /*.homepagePostReplyInputContainer .ke-container {float: left;}*/
-</style>
-<% if topics%>
     <% topics.each do |topic| %>
         <script>
             function expand_reply(container, btnid) {
@@ -44,19 +31,12 @@
             });
         </script>
         <% if topic %>
-            <%= render :partial => 'users/course_message', :locals => {:activity => topic, :user_activity_id => topic.id,:is_course=>1,:is_board=>1} %>
+            <%= render :partial => 'users/course_message', :locals => {:activity => topic, :user_activity_id => topic.id, :is_course => 1, :is_board=>1} %>
         <% end %>
     <% end %>
 
     <% if topics.count == 10 %>
-        <!--<div id="show_more_course_topic" class="loadMore mt10 f_grey">展开更多<%#= link_to "", boards_topic_path(@board, :course_id => @board.course.id ,:page => page), :id => "more_topic_link", :remote => "true", :class => "none" %></div>-->
         <%= link_to "点击展开更多",boards_topic_path(@board, :course_id => @board.course.id ,:page => page),:id => "show_more_course_topic",:remote => "true",:class => "loadMore mt10 f_grey"%>
     <% end %>
 <% end%>
 
-<!--
-<script type="text/javascript">
-    $("#show_more_course_topic").mouseover(function () {
-        $("#more_topic_link").click();
-    });
-</script>-->

app/views/boards/show.html.erb

@@ -192,6 +192,4 @@ function nh_init_board(params){
         });
     }
 }
-
-
 </script>

app/views/layouts/_project_info.html.erb

@@ -25,15 +25,15 @@
     <%= @course.name %>
   </a>
   <span id="course_outline_bar">
-      <%if User.current && @course.tea_id == User.current.id && (@course.outline == 0 ||  BlogComment.where(:id=>@course.outline).count == 0) %>
+      <% if User.current && @course.tea_id == User.current.id && (@course.outline == 0 ||  BlogComment.where(:id=>@course.outline).count == 0) %>
           <a href="javascript:void(0);" title="设置课程大纲" onclick="course_outline('<%= @course.id%>')" class="mr5 syllabusSetting fl"> </a>
-       <% elsif User.current && @course.tea_id == User.current.id && @course.outline != 0 && BlogComment.where(:id=>@course.outline).count != 0%>
+       <% elsif User.current && @course.tea_id == User.current.id && @course.outline != 0 && BlogComment.where(:id=>@course.outline).count != 0 %>
           <a href="<%=syllabus_course_path(@course) %>" title="课程大纲" class="mr5 syllabusIcon fl"> </a>
-        <% elsif User.current && @course.tea_id != User.current.id && !@course.is_public? && User.current.member_of_course?(@course)  && @course.outline != 0%>
+        <% elsif User.current && @course.tea_id != User.current.id && !@course.is_public? && User.current.member_of_course?(@course)  && @course.outline != 0 %>
           <a href="<%=syllabus_course_path(@course) %>" title="课程大纲" class="mr5 syllabusIcon fl"> </a>
-        <% elsif User.current && @course.tea_id != User.current.id && @course.is_public? && @course.outline != 0%>
+        <% elsif User.current && @course.tea_id != User.current.id && @course.is_public? && @course.outline != 0 %>
           <a href="<%=syllabus_course_path(@course) %>" title="课程大纲" class="mr5 syllabusIcon fl"> </a>
-        <%else%>
+        <%else %>
         <%end %>
   </span>
 </div>

app/views/users/_course_message.html.erb

@@ -2,14 +2,14 @@
   <div class="homepagePostBrief">
     <div class="homepagePostPortrait">
       <%= link_to image_tag(url_to_avatar(activity.author), :width => "50", :height => "50"), user_path(activity.author_id,:host=>Setting.host_user), :alt => "用户头像" %>
-      <%= render :partial => 'users/show_detail_info', :locals => {:user => activity.author} %>
+      <%#= render :partial => 'users/show_detail_info', :locals => {:user => activity.author} %>
     </div>
     <div class="homepagePostDes">
       <div class="homepagePostTo break_word mt-4">
         <% if activity.try(:author).try(:realname) == ' ' %>
-            <%= link_to activity.try(:author), user_path(activity.author_id,:host=>Setting.host_user), :class => "newsBlue mr15" %>
+            <%= link_to activity.try(:author), user_path(activity.author_id, :host=>Setting.host_user), :class => "newsBlue mr15" %>
         <% else %>
-            <%= link_to activity.try(:author).try(:realname), user_path(activity.author_id,:host=>Setting.host_user), :class => "newsBlue mr15" %>
+            <%= link_to activity.try(:author).try(:realname), user_path(activity.author_id, :host=>Setting.host_user), :class => "newsBlue mr15" %>
         <% end %>
         TO
         <%= link_to activity.course.name.to_s+" | 课程问答区", course_boards_path(activity.course,:host=> Setting.host_course), :class => "newsBlue ml15 mr5"%>
@@ -18,15 +18,15 @@
         <% if activity.parent_id.nil? %> <!--+"(帖子标题)"-->
             <%= link_to activity.subject.to_s.html_safe, board_message_path(activity.board_id, activity), :class=> "postGrey" %>
         <% else %>
-            <%= link_to activity.parent.subject.to_s.html_safe, board_message_path(activity.board_id, activity), :class=> "postGrey"%>
+            <%= link_to activity.parent.subject.to_s.html_safe, board_message_path(activity.board_id, activity), :class=> "postGrey" %>
         <% end %>
       </div>
-      <% if activity.sticky == 1%>
+      <% if activity.sticky == 1 %>
           <span class="sticky_btn_cir ml10">置顶</span>
       <% end%>
-      <% if activity.locked%>
+      <% if activity.locked %>
           <span class="locked_btn_cir ml10 fl" title="已锁定">&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;</span>
-      <% end%>
+      <% end %>
       <div class="cl"></div>
       <div class="homepagePostDate fl">
         发帖时间：<%= format_time(activity.created_on) %>
@@ -36,9 +36,9 @@
       </div>
       <div class="cl"></div>
       <% if activity.parent_id.nil? %>
-          <% content = activity.content%>
+          <% content = activity.content %>
       <% else %>
-          <% content = activity.parent.content%>
+          <% content = activity.parent.content %>
       <% end %>
       <%=render :partial =>"users/intro_content", :locals=>{:user_activity_id =>user_activity_id, :content=>content} %>
       <div class="cl"></div>