From 0e2b7ba362f995ec0ceedef1d68ea434cda5219c Mon Sep 17 00:00:00 2001
From: yanxd <sxty32@hotmail.com>
Date: Thu, 3 Jul 2014 17:22:48 +0800
Subject: [PATCH] tag search fix

---
 app/controllers/tags_controller.rb            | 41 ++++++++++++-------
 app/helpers/courses_helper.rb                 |  5 +++
 app/views/tags/_show_attachments.html.erb     |  2 +-
 app/views/tags/_show_bids.html.erb            |  2 +-
 app/views/tags/_show_contests.html.erb        |  2 +-
 app/views/tags/_show_courses.html.erb         | 14 +++++++
 app/views/tags/_show_forums.html.erb          |  2 +-
 app/views/tags/_show_issues.html.erb          |  2 +-
 .../tags/_show_open_source_projects.html.erb  |  2 +-
 app/views/tags/_show_projects.html.erb        |  2 +-
 app/views/tags/_show_users.html.erb           |  2 +-
 app/views/tags/_tag_search_results.html.erb   |  3 ++
 app/views/tags/index.html.erb                 |  3 +-
 13 files changed, 58 insertions(+), 24 deletions(-)
 create mode 100644 app/views/tags/_show_courses.html.erb

diff --git a/app/controllers/tags_controller.rb b/app/controllers/tags_controller.rb
index 0216d529d..ba01aa990 100644
--- a/app/controllers/tags_controller.rb
+++ b/app/controllers/tags_controller.rb
@@ -64,15 +64,18 @@ class TagsController < ApplicationController
     @forum_tags_num, @attachments_tags_num, @open_source_projects_num = get_tags_size
 
     # 获取搜索结果 
-    @obj,@obj_pages,@results_count,@users_results,
+    @obj,
+    @obj_pages,
+    @results_count,
+    @users_results,
     @projects_results,
-    @@courses_results,
     @issues_results,
     @bids_results,
-    @forums_results,
-    @attachments_results,
-    @contests_tags,
-    @open_source_projects_results = refresh_results(@obj_id,@obj_flag,@selected_tags)
+    @forums_results, 
+    attachments_results,
+    @contests_results,
+    @courses_results,
+    @open_source_projects_results= refresh_results(@obj_id,@obj_flag,@selected_tags)
 
     # 这里是做tag推荐用的， 用来生产推荐的tags
     unless @obj.nil?
@@ -97,14 +100,18 @@ class TagsController < ApplicationController
     $related_tags.delete(@tag)
 
     # 获取搜索结果 
-    @obj,@obj_pages,@results_count,@users_results,
+    @obj,
+    @obj_pages,
+    @results_count,
+    @users_results,
     @projects_results,
-    @@courses_results,
     @issues_results,
     @bids_results,
-    @forums_results,
-    @attachments_results,
-    @contests_results = refresh_results(@obj_id,@show_flag)
+    @forums_results, 
+    attachments_results,
+    @contests_results,
+    @courses_results,
+    @open_source_projects_results= refresh_results(@obj_id,@show_flag)
   end
 
   # 删除已选tag 
@@ -116,14 +123,18 @@ class TagsController < ApplicationController
     $selected_tags.delete(@tag)
 
     # 获取搜索结果 
-    @obj,@obj_pages,@results_count,@users_results,
+    @obj,
+    @obj_pages,
+    @results_count,
+    @users_results,
     @projects_results,
-    @@courses_results,
     @issues_results,
     @bids_results,
     @forums_results, 
-    @attachments_results,
-    @contests_results = refresh_results(@obj_id,@show_flag)   
+    attachments_results,
+    @contests_results,
+    @courses_results,
+    @open_source_projects_results= refresh_results(@obj_id,@show_flag)   
   end
 
   def show_all
diff --git a/app/helpers/courses_helper.rb b/app/helpers/courses_helper.rb
index b1ec023c8..b0cc3ce64 100644
--- a/app/helpers/courses_helper.rb
+++ b/app/helpers/courses_helper.rb
@@ -370,4 +370,9 @@ module CoursesHelper
     end
     return homework_users
   end
+
+  def get_courses_by_tag(tag_name)
+    Course.tagged_with(tag_name).order('updated_at desc')
+  end
+
 end
diff --git a/app/views/tags/_show_attachments.html.erb b/app/views/tags/_show_attachments.html.erb
index 3484d1351..43ed76e02 100644
--- a/app/views/tags/_show_attachments.html.erb
+++ b/app/views/tags/_show_attachments.html.erb
@@ -1,5 +1,5 @@
 <div id="issues">
-<% if attachments_results.size < 0 %>
+<% if attachments_results.try(:size).to_i < 0 %>
 <% else %>
 <hr />
 	<% attachments_results.each do |file| %>
diff --git a/app/views/tags/_show_bids.html.erb b/app/views/tags/_show_bids.html.erb
index 63f351738..e789979e1 100644
--- a/app/views/tags/_show_bids.html.erb
+++ b/app/views/tags/_show_bids.html.erb
@@ -1,5 +1,5 @@
 <div id="issues">
-	<% if bids_results.size > 0 %>
+	<% if bids_results.try(:size).to_i > 0 %>
 	<hr />
 	<% bids_results.each do |bid| %>
 	<p class="font_description2">
diff --git a/app/views/tags/_show_contests.html.erb b/app/views/tags/_show_contests.html.erb
index 79c756522..bc2682fb8 100644
--- a/app/views/tags/_show_contests.html.erb
+++ b/app/views/tags/_show_contests.html.erb
@@ -1,5 +1,5 @@
 <div id="issues">
-	<% if contests_results.size > 0 %>
+	<% if contests_results.try(:size).to_i > 0 %>
 	<hr />
 	<% contests_results.each do |contest| %>
 	<p class="font_description2">
diff --git a/app/views/tags/_show_courses.html.erb b/app/views/tags/_show_courses.html.erb
new file mode 100644
index 000000000..c99e3e22d
--- /dev/null
+++ b/app/views/tags/_show_courses.html.erb
@@ -0,0 +1,14 @@
+<div id="issues">
+	<% if courses_results.try(:size).to_i > 0 %>
+	<hr />
+	<% courses_results.each do |course| %>
+	<p class="font_description2">
+		<strong><%= l(:label_course) %>:<%= link_to "#{course.name}",course_path(course) %></strong>
+		<br />
+		<strong><%= l(:label_new_course_description) %>:</strong><%= course.description %>
+		<%= course.updated_at %>
+	</p>
+	<div class="line_under"></div>
+	<% end %>
+	<% end %>
+</div>
diff --git a/app/views/tags/_show_forums.html.erb b/app/views/tags/_show_forums.html.erb
index 009ec1949..c6dd8a2ed 100644
--- a/app/views/tags/_show_forums.html.erb
+++ b/app/views/tags/_show_forums.html.erb
@@ -1,5 +1,5 @@
 <div id="issues">
-	<% if forums_results.size > 0 %>
+	<% if forums_results.try(:size).to_i > 0 %>
 	<hr />
 	<% forums_results.each do |forum| %>
 	<p class="font_description2">
diff --git a/app/views/tags/_show_issues.html.erb b/app/views/tags/_show_issues.html.erb
index ab98cc351..399caf513 100644
--- a/app/views/tags/_show_issues.html.erb
+++ b/app/views/tags/_show_issues.html.erb
@@ -1,5 +1,5 @@
 <div id="issues">
-	<% if issues_results.size > 0 %>
+	<% if issues_results.try(:size).to_i > 0 %>
 	<hr />
 	<% issues_results.each do |issue| %>
 	<p class="font_description2">
diff --git a/app/views/tags/_show_open_source_projects.html.erb b/app/views/tags/_show_open_source_projects.html.erb
index 2c8bc6374..8ea67f9a7 100644
--- a/app/views/tags/_show_open_source_projects.html.erb
+++ b/app/views/tags/_show_open_source_projects.html.erb
@@ -1,5 +1,5 @@
 <div id="projects">
-<% if projects_results.size > 0 %>
+<% if projects_results.try(:size).to_i > 0 %>
 <hr />
 <% projects_results.each do |prj| %>
 <div>
diff --git a/app/views/tags/_show_projects.html.erb b/app/views/tags/_show_projects.html.erb
index 6186b6869..5dade07ef 100644
--- a/app/views/tags/_show_projects.html.erb
+++ b/app/views/tags/_show_projects.html.erb
@@ -1,5 +1,5 @@
 <div id="projects">
-<% if projects_results.size > 0 %>
+<% if projects_results.try(:size).to_i > 0 %>
 <hr />
 <% projects_results.each do |prj| %>
 <div>
diff --git a/app/views/tags/_show_users.html.erb b/app/views/tags/_show_users.html.erb
index 6818bdd9c..d8b172ba9 100644
--- a/app/views/tags/_show_users.html.erb
+++ b/app/views/tags/_show_users.html.erb
@@ -1,5 +1,5 @@
 <div id="users">
-	<% if users_results.size > 0 %>
+	<% if users_results.try(:size).to_i > 0 %>
 	<hr />
 	<% users_results.each do |user| %>
 	<p class="font_description2">
diff --git a/app/views/tags/_tag_search_results.html.erb b/app/views/tags/_tag_search_results.html.erb
index 1ea452dec..43048be58 100644
--- a/app/views/tags/_tag_search_results.html.erb
+++ b/app/views/tags/_tag_search_results.html.erb
@@ -26,6 +26,9 @@
     <strong><%#= l(:label_attachment)%>
     开源项目：(<%= @results_count %>)</strong>
 	<%= render :partial => "show_open_source_projects",:locals => {:projects_results => open_source_projects_results}%>
+  <% when show_flag == '9'%>
+    <strong><%= l(:label_course)%>(<%= @results_count %>)</strong>
+  <%= render :partial => "show_courses",:locals => {:courses_results => courses_results}%>
 	<% else %>
 	 <strong><%= l(:label_tags_all_objects)%></strong> 
 	 <!--  这里为显示搜有过滤结果预留了默认设置 -->
diff --git a/app/views/tags/index.html.erb b/app/views/tags/index.html.erb
index c74721c3e..a5cda9944 100644
--- a/app/views/tags/index.html.erb
+++ b/app/views/tags/index.html.erb
@@ -23,7 +23,7 @@
 		<%= l(:label_user_plural) %>(<%= @users_tags_num %>) |
 		<%= l(:label_tags_call)%>(<%= @bids_tags_num %>) |
 		<%= l(:field_filename)%>(<%= @attachments_tags_num %>) |
-		开源项目(<%= @open_source_projects_num %>)
+		开源项目(<%= @open_source_projects_num %>) |
 		<%= l(:label_tags_contest)%>(<%= @contests_tags_num %>) |
 	</div>
 	<div id="show_results">
@@ -37,6 +37,7 @@
 								:attachments_results=> @attachments_results,
 								:contests_results 	=> @contests_results,
 								:open_source_projects_results => @open_source_projects_results,
+								:courses_results => @courses_results,
 								:show_flag 			=> @obj_flag}
 		%>
 	</div>