diff --git a/app/controllers/admin_controller.rb b/app/controllers/admin_controller.rb
index be622b919..52e1f5fd0 100644
--- a/app/controllers/admin_controller.rb
+++ b/app/controllers/admin_controller.rb
@@ -61,7 +61,7 @@ class AdminController < ApplicationController
if params[:syllabus_id] && params[:course_id]
course = Course.where("id = #{params[:course_id].to_i}").first
unless course.nil?
- course.update_attribute('syllabus_id', params[:syllabus_id].to_i)
+ course.update_column('syllabus_id', params[:syllabus_id].to_i)
@flag = true
end
end
@@ -124,11 +124,11 @@ class AdminController < ApplicationController
end
end
- #修改课程名称
+ #修改班级名称
def update_course_name
@course = Course.where("id = #{params[:course_id].to_i}").first
unless @course.nil?
- @course.update_attribute("name", params[:name])
+ @course.update_column("name", params[:name])
respond_to do |format|
format.js
end
diff --git a/app/controllers/quality_analysis_controller.rb b/app/controllers/quality_analysis_controller.rb
index 73672d11a..03ed7945a 100644
--- a/app/controllers/quality_analysis_controller.rb
+++ b/app/controllers/quality_analysis_controller.rb
@@ -114,7 +114,7 @@ class QualityAnalysisController < ApplicationController
end
end
rescue => e
- puts e
+ @message = e.message
end
respond_to do |format|
if @current_build_status == "success"
@@ -227,9 +227,6 @@ class QualityAnalysisController < ApplicationController
end
@ha.store(key,value)
end
-
- issue_date = open(@sonar_address + "/api/resources/index?resource=#{@resource_id}&depth=0&metrics=blocker_violations,critical_violations,major_violations,minor_violations,info_violations,violations").read
- @sonar_issues = JSON.parse(issue_date).first
end
rescue => e
puts e
diff --git a/app/helpers/quality_analysis_helper.rb b/app/helpers/quality_analysis_helper.rb
index 2a7d350e7..5940dd255 100644
--- a/app/helpers/quality_analysis_helper.rb
+++ b/app/helpers/quality_analysis_helper.rb
@@ -3,16 +3,16 @@ module QualityAnalysisHelper
def sqale_rating_status val
arr = []
- if val < 5
+ if val <= 5
arr << "很好"
arr << "b_green2"
- elsif val. > 5 && val < 10
+ elsif val. > 5 && val <= 10
arr << "较好"
arr << "b_slow_yellow"
- elsif val > 10 && val < 20
+ elsif val > 10 && val <= 20
arr << "中等"
arr << "b_yellow"
- elsif val > 20 && val < 50
+ elsif val > 20 && val <= 50
arr << "较差"
arr << "b_slow_red"
elsif val > 20
@@ -23,10 +23,10 @@ module QualityAnalysisHelper
def complexity_status val
arr = []
- if val < 10
+ if val <= 10
arr << "良好"
arr << "b_green2"
- elsif val > 10 && val < 15
+ elsif val > 10 && val <= 15
arr << "较高"
arr << "b_yellow"
elsif val > 15
@@ -37,10 +37,10 @@ module QualityAnalysisHelper
def duplicated_lines_density_status val
arr = []
- if val < 30
+ if val <= 30
arr << "良好"
arr << "b_green2"
- elsif val > 30 && val < 50
+ elsif val > 30 && val <= 50
arr << "较高"
arr << "b_yellow"
elsif val > 50
@@ -51,10 +51,10 @@ module QualityAnalysisHelper
def comment_lines_density_status val
arr = []
- if val < 20
+ if val <= 20
arr << "较低"
arr << "b_yellow"
- elsif val > 20 && val < 50
+ elsif val > 20 && val <= 50
arr << "正常"
arr << "b_green2"
elsif val > 50
@@ -64,15 +64,15 @@ module QualityAnalysisHelper
end
def score_sqale_rating val
- if val > 0 && val < 5
+ if val >= 0 && val <= 5
"5"
- elsif val > 5 && val < 10
+ elsif val > 5 && val <= 10
"4"
- elsif val > 10 && val < 20
+ elsif val > 10 && val <= 20
"3"
- elsif val > 20 && val < 50
+ elsif val > 20 && val <= 50
"2"
- elsif val > 20
+ elsif val > 50
"1"
end
end
diff --git a/app/views/admin/_courselist_detail_tr.html.erb b/app/views/admin/_courselist_detail_tr.html.erb
index bc1eebeed..83b7904ce 100644
--- a/app/views/admin/_courselist_detail_tr.html.erb
+++ b/app/views/admin/_courselist_detail_tr.html.erb
@@ -17,7 +17,8 @@
<%= course.is_delete == 0 ? '正常' : '归档' %>
- |
+ |
+
<%= format_date(course.created_at) %>
|
diff --git a/app/views/admin/non_syllabus_courses.html.erb b/app/views/admin/non_syllabus_courses.html.erb
index 6cde56675..1be979eeb 100644
--- a/app/views/admin/non_syllabus_courses.html.erb
+++ b/app/views/admin/non_syllabus_courses.html.erb
@@ -27,10 +27,10 @@
-
+ |
序号
|
-
+ |
班级
|
@@ -42,6 +42,9 @@
|
<%=l(:field_is_public)%>
|
+
+ 状态
+ |
<%=l(:field_created_on)%>
|
@@ -63,7 +66,7 @@
<%= course.id %>
|
-
+ |
<%= course.name%>
@@ -77,6 +80,9 @@
|
<%= checked_image course.is_public? %>
|
+
+ <%= course.is_delete == 0 ? '正常' : '归档' %>
+ |
<%= format_date(course.created_at) %>
|
diff --git a/app/views/quality_analysis/_result_list.html.erb b/app/views/quality_analysis/_result_list.html.erb
index b0b8abafe..3e2a7d373 100644
--- a/app/views/quality_analysis/_result_list.html.erb
+++ b/app/views/quality_analysis/_result_list.html.erb
@@ -17,10 +17,10 @@
<% if @quality_analyses.count >0 %>
<% @quality_analyses.each do |qa| %>
- - <%=link_to "#{qa.author_login}:#{qa.rep_identifier}", project_quality_analysis_path(:resource_id => qa.sonar_name, :branch => (qa.branch.nil? ? "master" : qa.branch)), :class => "analysis-result-name fl fontBlue2" %>
- - <%= qa.branch %>
- - <%= qa.language %>
- - <%= qa.path %>
+ - <%=link_to "#{qa.author_login}:#{qa.rep_identifier}", project_quality_analysis_path(:resource_id => qa.sonar_name, :branch => (qa.branch.nil? ? "master" : qa.branch)), :class => "analysis-result-name fl fontBlue2 hidden" %>
+ - <%= qa.branch %>
+ - <%= qa.language %>
+ - <%= qa.path %>
<% if User.current.try(:login) == qa.author_login %>
-
<%=link_to "编辑", edit_project_quality_analysi_path(qa, :project_id => @project.id), :remote => true, :class => "fontBlue2" %>
diff --git a/app/views/quality_analysis/_show.html.erb b/app/views/quality_analysis/_show.html.erb
index 347f33cb5..0f5aebc2d 100644
--- a/app/views/quality_analysis/_show.html.erb
+++ b/app/views/quality_analysis/_show.html.erb
@@ -58,17 +58,17 @@
可定性评价为:质量<%= @ha["sqale_rating"].nil? ? "很好" : sqale_rating_status(@ha["sqale_rating"].to_i)[0] %>
技术债务
-
+
<% if @ha["sqale_index"].to_i == 0 %>
- 0
+ 0天
<% else %>
<%= /[0-9]*/.match(@ha["sqale_index"])[0] %>天
- <%= / [0-9]*/.match(@ha["sqale_index"]).nil? ? 0 : / [0-9]*/.match(@ha["sqale_index"]) %>小时
+ <%= / [0-9]*/.match(@ha["sqale_index"]).nil? ? "" : (/ [0-9]*/.match(@ha["sqale_index"]).to_s + "小时") %>
<% end %>
查看详情
- 质量问题
-
+ 质量问题
+
<%= @ha["violations"].nil? ? 0 : @ha["violations"] %>
diff --git a/app/views/quality_analysis/index.html.erb b/app/views/quality_analysis/index.html.erb
index 7925f6120..06455cebe 100644
--- a/app/views/quality_analysis/index.html.erb
+++ b/app/views/quality_analysis/index.html.erb
@@ -1,14 +1,6 @@
<% if @name_flag %>
<%= render :partial => "result_list" %>
<% else %>
- <% if params[:current_build_status].nil? || params[:current_build_status] == "success" %>
- <%= render :partial => "show", :locals => {:branch => @branch} %>
- <% else %>
- <% if params[:build_console_result] %>
- 运行结果超时
- <% else %>
- <%= render :partial => "console_output" %>
- <% end %>
- <% end %>
+ <%= render :partial => "show", :locals => {:branch => @branch} %>
<% end %>
diff --git a/app/views/users/_join_course_course_message.html.erb b/app/views/users/_join_course_course_message.html.erb
index d0718c11e..933de0b0d 100644
--- a/app/views/users/_join_course_course_message.html.erb
+++ b/app/views/users/_join_course_course_message.html.erb
@@ -18,8 +18,7 @@
真实姓名:<%= User.find(ma.course_message_id).realname %>
申请课程:<%= Course.find(ma.course_id).name%>
课程描述:
- <%= Course.find(ma.course_id).description.html_safe if Course.find(ma.course_id).description %>
- 申请职位:<%= ma.content == '9' ? "教师" : "教辅"%>
+ <%= Course.find(ma.course_id).description.html_safe if Course.find(ma.course_id).description %> 申请职位:<%= ma.content == '9' ? "教师" : "教辅"%>
-
diff --git a/app/views/users/_user_message_course.html.erb b/app/views/users/_user_message_course.html.erb
index 065cb0b27..c27002d24 100644
--- a/app/views/users/_user_message_course.html.erb
+++ b/app/views/users/_user_message_course.html.erb
@@ -1,5 +1,5 @@
<% if ma.class == CourseMessage %>
- <% if ma.course_message_type == "News" %>
+ <% if ma.course_message_type == "News" && ma.course_message %>
- <%=link_to image_tag(url_to_avatar(ma.course_message.author), :width => "30", :height => "30"),user_path(ma.course_message.author), :target => '_blank' %>
- <%=link_to ma.course_message.author, user_path(ma.course_message.author), :class => "newsBlue homepageNewsPublisher", :target => '_blank' %>">发布了通知:
@@ -19,7 +19,7 @@
- <%= time_tag(ma.created_at).html_safe %>
<% end %>
- <% if ma.course_message_type == "Comment" %>
+ <% if ma.course_message_type == "Comment" && ma.course_message %>
- <%=link_to image_tag(url_to_avatar(ma.course_message.author), :width => "30", :height => "30"), user_path(ma.course_message.author), :target => '_blank' %>
- <%=link_to ma.course_message.author, user_path(ma.course_message.author), :class => "newsBlue homepageNewsPublisher", :target => '_blank' %>">评论了通知:
@@ -39,7 +39,7 @@
- <%= time_tag(ma.created_at).html_safe %>
<% end %>
- <% if ma.course_message_type == "HomeworkCommon" && ma.status.nil?%>
+ <% if ma.course_message_type == "HomeworkCommon" && ma.status.nil? && ma.course_message %>
- <%=link_to image_tag(url_to_avatar(ma.course_message.user), :width => "30", :height => "30"), user_path(ma.course_message.user), :target => '_blank' %>
- <%=link_to ma.course_message.user.lastname + ma.course_message.user.firstname + "老师", user_path(ma.course_message.user), :class => "newsBlue homepageNewsPublisher", :target => '_blank' %>
@@ -98,7 +98,7 @@
- <%= time_tag(ma.created_at).html_safe %>
<% end %>
- <% if ma.course_message_type == "HomeworkCommon" && ma.status == 1 %>
+ <% if ma.course_message_type == "HomeworkCommon" && ma.status == 1 && ma.course_message %>
<% end %>
- <% if ma.course_message_type == "HomeworkCommon" && ma.status == 2 %>
+ <% if ma.course_message_type == "HomeworkCommon" && ma.status == 2 && ma.course_message %>
<% end %>
- <% if ma.course_message_type == "HomeworkCommon" && ma.status == 3 %>
+ <% if ma.course_message_type == "HomeworkCommon" && ma.status == 3 && ma.course_message %>
<% end %>
- <% if ma.course_message_type == "HomeworkCommon" && ma.status == 4 %>
+ <% if ma.course_message_type == "HomeworkCommon" && ma.status == 4 && ma.course_message %>
<% end %>
- <% if ma.course_message_type == "ApplyResource" && ma.status == 0 %>
+ <% if ma.course_message_type == "ApplyResource" && ma.status == 0 && ma.course_message %>
<%= render :partial => 'apply_resource_course_message', :locals => {:ma => ma} %>
<% end %>
- <% if ma.course_message_type == "ApplyResource" && ma.status == 1 %>
+ <% if ma.course_message_type == "ApplyResource" && ma.status == 1 && ma.course_message %>
<%= render :partial => 'apply_resource_course_message_reply', :locals => {:ma => ma} %>
<% end %>
- <% if ma.course_message_type == "HomeworkCommon" && ma.status == 6 %>
+ <% if ma.course_message_type == "HomeworkCommon" && ma.status == 6 && ma.course_message %>
<% end %>
- <% if ma.course_message_type == "Poll" %>
+ <% if ma.course_message_type == "Poll" && ma.course_message %>
- <%=link_to image_tag(url_to_avatar(ma.course_message.user), :width => "30", :height => "30"), user_path(ma.course_message.user), :target => '_blank' %>
- <%=link_to ma.course_message.user, user_path(ma.course_message.user), :class => "newsBlue homepageNewsPublisher", :target => '_blank' %>">发布了问卷:
@@ -287,7 +287,7 @@
- <%= time_tag(ma.created_at).html_safe %>
<% end %>
- <% if ma.course_message_type == "Message" %>
+ <% if ma.course_message_type == "Message" && !ma.nil? && !ma.course_message.nil? %>
<% end %>
- <% if ma.course_message_type == "JournalsForMessage" %>
+ <% if ma.course_message_type == "JournalsForMessage" && ma.course_message %>
<% if ma.course_message.jour_type == 'Course' %>
<% if params[:type] != 'homework' %>
@@ -459,7 +459,7 @@
<% end %>
<% end %>
- <% if ma.course_message_type == "StudentWork" && !ma.course_message.homework_common.nil? && !User.current.allowed_to?(:as_teacher, ma.course_message.homework_common.course) %>
+ <% if ma.course_message_type == "StudentWork" && !ma.course_message.homework_common.nil? && !User.current.allowed_to?(:as_teacher, ma.course_message.homework_common.course) && ma.course_message %>
- <%= link_to image_tag(url_to_avatar(ma.course_message.homework_common.user), :width => "30", :height => "30"), user_path(ma.course_message.homework_common.user), :target => '_blank' %>
-
@@ -493,7 +493,7 @@
<% end %>
- <% if ma.course_message_type == "StudentWork" && ma.status == 1 %>
+ <% if ma.course_message_type == "StudentWork" && ma.status == 1 && ma.course_message %>
<% end %>
- <% if ma.course_message_type == "StudentWork" && ma.status == 2 %>
+ <% if ma.course_message_type == "StudentWork" && ma.status == 2 && ma.course_message %>
<% end %>
- <% if ma.course_message_type == "Course" %>
+ <% if ma.course_message_type == "Course" && ma.course_message %>
<% end %>
- <% if ma.course_message_type == "JoinCourseRequest" %>
+
+
+ <% if ma.course_message_type == "JoinCourseRequest" && !ma.nil? %>
<%= render :partial => 'join_course_course_message', :locals => {:ma => ma} %>
@@ -710,7 +712,7 @@
<% end %>
- <% if ma.course_message_type == "Exercise" && ma.status == 2 %>
+ <% if ma.course_message_type == "Exercise" && ma.status == 2 && ma.course_message %>
- <%=link_to image_tag(url_to_avatar(ma.course_message.user), :width => "30", :height => "30"), user_path(ma.course_message.user), :target => '_blank' %>
-
@@ -747,7 +749,7 @@
<% end %>
- <% if ma.course_message_type == "Exercise" && ma.status == 3 %>
+ <% if ma.course_message_type == "Exercise" && ma.status == 3 && ma.course_message %>
- <%=link_to image_tag(url_to_avatar(ma.course_message.user), :width => "30", :height => "30"), user_path(ma.course_message.user), :target => '_blank' %>
-
diff --git a/lib/tasks/update_homework.rake b/lib/tasks/update_homework.rake
index 7c550b673..d97004c8c 100644
--- a/lib/tasks/update_homework.rake
+++ b/lib/tasks/update_homework.rake
@@ -20,8 +20,8 @@ namespace :update_homework do
project_time=ForgeActivity.where("project_id=?",project.id).last.updated_at if ForgeActivity.where("project_id=?",project.id).last
puts "the time is =========================> #{time}"
puts "the project_time is =========================> #{project_time}"
- #if time.strftime('%Y-%m-%d %H:%M:%S') < project_time.strftime('%Y-%m-%d %H:%M:%S')
- if format_time(time) < format_time(project_time)
+ if time.strftime('%Y-%m-%d %H:%M:%S') < project_time.strftime('%Y-%m-%d %H:%M:%S')
+ #if format_time(time) < format_time(project_time)
time = project_time
end
begin
@@ -33,9 +33,10 @@ namespace :update_homework do
unless changesets[0].blank?
puts "the time is =========================> #{time}"
puts "the changesets_latest_coimmit is =========================> #{changesets_latest_coimmit.created_at}"
- #if time.strftime('%Y-%m-%d %H:%M:%S') < changesets_latest_coimmit.created_at.strftime('%Y-%m-%d %H:%M:%S')
- if format_time(time) < format_time(changesets_latest_coimmit.created_at)
- time = changesets_latest_coimmit.created_at
+ puts "the distance time is =========================> #{changesets_latest_coimmit.created_at.to_time > Time.now}"
+ if time.strftime('%Y-%m-%d %H:%M:%S') < changesets_latest_coimmit.created_at.to_time.strftime('%Y-%m-%d %H:%M:%S')
+ #if format_time(time) < format_time(changesets_latest_coimmit.created_at)
+ time = changesets_latest_coimmit.created_at.to_time
end
end
rescue Exception => e
diff --git a/public/stylesheets/project.css b/public/stylesheets/project.css
index bde560613..9a3ba2e47 100644
--- a/public/stylesheets/project.css
+++ b/public/stylesheets/project.css
@@ -1273,11 +1273,11 @@ a.pages-big{ width:50px;}
.analysis-result-list {padding:5px;}
.analysis-result-list:nth-of-type(odd){background:#fff;}/*奇数行*/
.analysis-result-list:nth-of-type(even){background:#f5f5f5;}/*偶数行*/
-.analysis-result-name {width:200px; cursor:pointer;}
-.analysis-result-version {width:90px; text-align:right; cursor:pointer;}
-.analysis-result-loc {width:60px; text-align:right; cursor:pointer;}
-.analysis-result-debt {width:160px; text-align:right; cursor:pointer;}
-.analysis-result-time {width:150px; text-align:right; cursor:pointer;}
+.analysis-result-name {width:200px;}
+.analysis-result-version {width:90px; text-align:right;}
+.analysis-result-loc {width:60px; text-align:right;}
+.analysis-result-debt {width:160px; text-align:right;}
+.analysis-result-time {width:150px; text-align:right;}
.analysis-name-icon {background:url(../images/code-analysis-icon.png) -2px -148px no-repeat; width:16px; height:16px; display:inline-block; vertical-align:middle;}
/*未登录回复提示*/
|