#1684 某课程主页--不输入搜索条件，直接点击搜索按钮返回404页面

app/controllers/courses_controller.rb

@@ -130,57 +130,64 @@ class CoursesController < ApplicationController
   def search
     courses_all = Course.all_course
     name = params[:name]
-    (redirect_to courses_url, :notice => l(:label_sumbit_empty);return) if name.blank?
-    @courses = courses_all.visible
-    if params[:name].present?
-      @courses_all = @courses.like(params[:name])
+    #(redirect_to courses_url, :notice => l(:label_sumbit_empty);return) if name.blank?
+    if name.blank?
+      @courses = []
+      @courses_all = []
+      @course_count = 0
+      @course_pages = Paginator.new @course_count, per_page_option, params['page']
     else
-      @courses_all =  @courses;
-    end
-    @course_count = @courses_all.count
-    @course_pages = Paginator.new @course_count, per_page_option, params['page']
+      @courses = courses_all.visible
+      if params[:name].present?
+        @courses_all = @courses.like(params[:name])
+      else
+        @courses_all =  @courses;
+      end
+      @course_count = @courses_all.count
+      @course_pages = Paginator.new @course_count, per_page_option, params['page']
 
-    # 课程的动态数
-    @course_activity_count=Hash.new
-    @courses_all.each do |course|
-      @course_activity_count[course.id]=0
-    end
+      # 课程的动态数
+      @course_activity_count=Hash.new
+      @courses_all.each do |course|
+        @course_activity_count[course.id]=0
+      end
 
-    case params[:course_sort_type]
-      when '0'
-        @courses =  @courses_all.order("created_at desc")
-        @s_type = 0
-        @courses =  @courses.offset(@course_pages.offset).limit(@course_pages.per_page)
+      case params[:course_sort_type]
+        when '0'
+          @courses =  @courses_all.order("created_at desc")
+          @s_type = 0
+          @courses =  @courses.offset(@course_pages.offset).limit(@course_pages.per_page)
 
-        @course_activity_count=get_course_activity @courses,@course_activity_count
+          @course_activity_count=get_course_activity @courses,@course_activity_count
 
-      when '1'
-        @courses = @courses_all.order("course_ac_para desc")
-        @s_type = 1
-        @courses =  @courses.offset(@course_pages.offset).limit(@course_pages.per_page)
+        when '1'
+          @courses = @courses_all.order("course_ac_para desc")
+          @s_type = 1
+          @courses =  @courses.offset(@course_pages.offset).limit(@course_pages.per_page)
 
-        @course_activity_count=get_course_activity @courses,@course_activity_count
+          @course_activity_count=get_course_activity @courses,@course_activity_count
 
-      when '2'
-        @courses = @courses_all.order("watchers_count desc")
-        @s_type = 2
-        @courses =  @courses.offset(@course_pages.offset).limit(@course_pages.per_page)
+        when '2'
+          @courses = @courses_all.order("watchers_count desc")
+          @s_type = 2
+          @courses =  @courses.offset(@course_pages.offset).limit(@course_pages.per_page)
 
-        @course_activity_count=get_course_activity @courses,@course_activity_count
+          @course_activity_count=get_course_activity @courses,@course_activity_count
 
-      when '3'
-        @course_activity_count=get_course_activity @courses_all,@course_activity_count_array
-        @courses=handle_course @courses_all,@course_activity_count
-        @s_type = 3
-        @courses =  @courses[@course_pages.offset, @course_pages.per_page]
+        when '3'
+          @course_activity_count=get_course_activity @courses_all,@course_activity_count_array
+          @courses=handle_course @courses_all,@course_activity_count
+          @s_type = 3
+          @courses =  @courses[@course_pages.offset, @course_pages.per_page]
 
-      else
-        @s_type = 0
-        @courses = @courses_all.order("created_at desc")
-        @courses =  @courses.offset(@course_pages.offset).limit(@course_pages.per_page)
+        else
+          @s_type = 0
+          @courses = @courses_all.order("created_at desc")
+          @courses =  @courses.offset(@course_pages.offset).limit(@course_pages.per_page)
 
-         @course_activity_count=get_course_activity @courses,@course_activity_count
+          @course_activity_count=get_course_activity @courses,@course_activity_count
 
+      end
     end
 
     respond_to do |format|