diff --git a/app/controllers/courses_controller.rb b/app/controllers/courses_controller.rb
index c73a1e0fc..9ae57db3d 100644
--- a/app/controllers/courses_controller.rb
+++ b/app/controllers/courses_controller.rb
@@ -641,7 +641,7 @@ class CoursesController < ApplicationController
   def isCourseTeacher(id,course)
     result = false
     user = User.find(id)
-    if user.nil? && user.allowed_to?(:as_teacher,course)#@teachers && @teachers.count != 0 && @teachers.find_by_user_id(id) != nil
+    if !user.nil? && user.allowed_to?(:as_teacher,course)
       result = true
     end
     result
diff --git a/app/views/courses/_member_list.html.erb b/app/views/courses/_member_list.html.erb
index a8a9e8085..edd04bc97 100644
--- a/app/views/courses/_member_list.html.erb
+++ b/app/views/courses/_member_list.html.erb
@@ -13,11 +13,8 @@
       <% end %>
 		<!--teacher's code disapeared  moified by huang-->
 
-      <% unless member.user.user_extensions.identity ==0 %>
-          <!-- added by bai 加学号 -->
-          <%if @canShowCode%>
-              <%= content_tag "p", "#{l(:label_bidding_user_studentcode)}#{' : '}#{member.user.user_extensions.student_id}", :class => "nomargin avatar_name" %>
-          <% end %>
+      <%if @canShowCode%>
+        <%= content_tag "p", "#{l(:label_bidding_user_studentcode)}#{' : '}#{member.user.user_extensions.student_id}", :class => "nomargin avatar_name" %>
       <% end %>
       <div class ="clear avatar_name"> &nbsp;
         <p>