diff --git a/app/controllers/courses_controller.rb b/app/controllers/courses_controller.rb
index 8c870d81d..7302af89d 100644
--- a/app/controllers/courses_controller.rb
+++ b/app/controllers/courses_controller.rb
@@ -60,7 +60,7 @@ class CoursesController < ApplicationController
     end
   end
 
-  
+
   def join_private_courses
     respond_to do |format|
       format.js
@@ -73,14 +73,14 @@ class CoursesController < ApplicationController
     c = cs.edit_course params,@course,User.current
     @course = c[:course]
     if @course.errors.full_messages.count <= 0
-        respond_to do |format|
-          format.html {
-            # render :layout => 'base_courses'
-            flash[:notice] = l(:notice_successful_update)
-            redirect_to settings_course_url(@course)
-          }
-          format.api  { render_api_ok }
-        end
+      respond_to do |format|
+        format.html {
+          # render :layout => 'base_courses'
+          flash[:notice] = l(:notice_successful_update)
+          redirect_to settings_course_url(@course)
+        }
+        format.api  { render_api_ok }
+      end
     else
       respond_to do |format|
         format.html {
@@ -308,7 +308,7 @@ class CoursesController < ApplicationController
   def member
     ## 有角色参数的才是课程，没有的就是项目
     if (User.current.admin? || @course.is_public == 1 || (@course.is_public == 0 && User.current.member_of_course?(@course)))
-      
+
       @render_file = 'new_member_list'
       @score_sort_by = "desc"
       @canShowCode = User.current.allowed_to?(:as_teacher,@course) && params[:role] != '1'
@@ -450,24 +450,24 @@ class CoursesController < ApplicationController
       end
     else
       respond_to do |format|
-              format.html {
-                # render :layout => 'base_courses'
-                flash[:notice] = l(:notice_successful_create)
-                if params[:continue]
-                  redirect_to new_course_url(attrs, :course => '0')
-                elsif params[:course_continue]
-                  redirect_to new_course_url(:course => '1')
-                else
-                  redirect_to settings_course_url(@course, :course_type => 1)
-                end
-              }
-              format.api { render :action => 'show', :status => :created, :location => url_for(:controller => 'courses', :action => 'show', :id => @course.id) }
-            end
+        format.html {
+          # render :layout => 'base_courses'
+          flash[:notice] = l(:notice_successful_create)
+          if params[:continue]
+            redirect_to new_course_url(attrs, :course => '0')
+          elsif params[:course_continue]
+            redirect_to new_course_url(:course => '1')
+          else
+            redirect_to settings_course_url(@course, :course_type => 1)
+          end
+        }
+        format.api { render :action => 'show', :status => :created, :location => url_for(:controller => 'courses', :action => 'show', :id => @course.id) }
+      end
     end
   end
 
   def course
-     @school_id = params[:school_id]
+    @school_id = params[:school_id]
     per_page_option = 10
     if @school_id == "0" or @school_id.nil?
       @courses_all = Course.active.visible.
@@ -551,8 +551,8 @@ class CoursesController < ApplicationController
 
   def index
     if !User.current.admin?
-       render_404
-       return
+      render_404
+      return
     end
     @course_type = params[:course_type]
     @school_id = params[:school_id]
@@ -601,7 +601,7 @@ class CoursesController < ApplicationController
 
     respond_to do |format|
       format.html {
-         render :layout => 'base'
+        render :layout => 'base'
       }
       format.atom {
         courses = Course.visible.order('created_on DESC').limit(Setting.feeds_limit.to_i).all
@@ -701,7 +701,7 @@ class CoursesController < ApplicationController
       if Rails.env.production? && Setting.course_cahce_enabled?
         events = Rails.cache.read(key) || []
       end
-      if events.empty? 
+      if events.empty?
         @authors.each do |author|
           @activity = Redmine::Activity::Fetcher.new(User.current, :course => @course,
                                                      :with_subprojects => false,
@@ -719,7 +719,7 @@ class CoursesController < ApplicationController
         Rails.cache.write(key, events) if Rails.env.production? && Setting.course_cahce_enabled?
       end
     else
-    # @author = @course.teacher
+      # @author = @course.teacher
       @activity = Redmine::Activity::Fetcher.new(User.current, :course => @course,
                                                  :with_subprojects => false,
                                                  :author => @author)
@@ -781,7 +781,6 @@ class CoursesController < ApplicationController
       end
       @jour = paginateHelper @jours,10
       @state = false
-      @page = page.nil? ? "1" : page
       respond_to do |format|
         format.html{render :layout => 'base_courses'}
         format.api
@@ -838,7 +837,7 @@ class CoursesController < ApplicationController
   def can_show_course
     @first_page = FirstPage.find_by_page_type('project')
     if @first_page.try(:show_course) == 2
-       render_404
+      render_404
     end
   end