PR添加评论 列表页面

app/controllers/pull_requests_controller.rb

@@ -1,7 +1,7 @@
 
 class PullRequestsController < ApplicationController
   before_filter :find_project_and_repository
-  before_filter :connect_gitlab, :only => [:index, :show, :create, :accept_pull_request, :pull_request_commits, :pull_request_changes, :new, :update_pull_request]
+  before_filter :connect_gitlab, :only => [:index, :show, :create, :accept_pull_request, :pull_request_commits, :pull_request_changes, :new, :update_pull_request, :pull_request_comments]
   layout "base_projects"
   include PullRequestsHelper
   include ApplicationHelper
@@ -87,6 +87,8 @@ class PullRequestsController < ApplicationController
     @commits_count = @commits.count
     @changes = @g.merge_request_changes(@project.gpid, params[:id]).try(:changes)
     @changes_count = @changes.count
+    @comments = @g.merge_request_comments(@project.gpid, params[:id])
+    @comments_count = @comments.count
   end
 
   # Accept a merge request.
@@ -170,6 +172,7 @@ class PullRequestsController < ApplicationController
   def pull_request_commits
     begin
       @commits = @g.merge_request_commits(@project.gpid, params[:id])
+      @commits_count = @commits.count
     rescue Exception => e
       @message = e.message
     end

app/views/pull_requests/_pull_request_comments.html.erb

@@ -0,0 +1,19 @@
+<!--<div class="merge-discussion-content">-->
+  <!--<a href="javascript:void(0);" class="link-blue mr15 fl">Hjqreturn</a><span class="fl fontGrey2">20小时前</span>-->
+  <!--<div class="cl"></div>-->
+  <!--<div class="merge-discussion-detail">回复内容</div>-->
+<!--</div>-->
+<% @comments.each do |comment| %>
+    <div class="merge-discussion-content merge-discussion-automatic">
+      <%= link_to comment.author.try(:username), user_path(get_user_by_login_and(comment.author.try(:username))), :class => "link-blue mr15 fl" %>
+      <span class="fl fontGrey2"><%#= time_tag(comment.author.try(:created_at)) %>前</span>
+      <div class="cl"></div>
+      <div class="merge-discussion-detail"><%= comment.note %></div>
+    </div>
+<% end %>
+
+<div class="merge-discussion-input">
+  
+  <textarea></textarea>
+  <a href="javascript:void(0);" class="BlueCirBtn">提 交</a>
+</div>

app/views/pull_requests/_show.html.erb

@@ -11,9 +11,9 @@
     <div class="cl"></div>
   </div>
   <div class="new-merge-row b_grey"><strong><%= @request.title %></strong><p class="mt10"><%= @request.description %></p></div>
+
   <div class="mt10 mb15">
     请求将 <span class="fontGrey3 fb"><%= @request.source_branch %></span> 合并至 <span class="fontGrey3 fb"><%= @request.target_branch %></span>
-
     <% if @commits_count == 0 && @changes_count == 0 %>
         <div class="merge-commit-option mt15">
           <span class="fontGrey3 fb" style="font-size:13px;"><img src="/images/warning.png" width="16" class="mr5 mt2 fl"><%= @request.source_branch %>没有新内容可以合并至<%= @request.target_branch %></span><br />
@@ -33,28 +33,15 @@
                 </div>
             <% end %>
         <% end %>
-
   </div>
+
         <ul class="merge-record" style="border-top:1px solid #ddd;">
-          <li><%= link_to "留言<span class='project-number-dot'>#{@commits_count}</span>".html_safe, :class => "active" %></li>
+          <li><%= link_to "留言<span class='project-number-dot'>#{@comments_count}</span>".html_safe, pull_request_comments_project_pull_request_path(@request.id, :project_id => @project.id), :remote => true, :class => "active" %></li>
-          <li><%= link_to "提交<span class='project-number-dot'>#{@commits_count}</span>".html_safe, pull_request_commits_project_pull_request_path(@request.id, :project_id => @project.id, :type => 1), :remote => true %></li>
+          <li><%= link_to "提交<span class='project-number-dot'>#{@commits_count}</span>".html_safe, pull_request_commits_project_pull_request_path(@request.id, :project_id => @project.id), :remote => true %></li>
-          <li><%= link_to "改动<span class='project-number-dot'>#{@changes_count}</span>".html_safe, pull_request_changes_project_pull_request_path(@request.id, :project_id => @project.id, :type => 2), :remote => true %></li>
+          <li><%= link_to "改动<span class='project-number-dot'>#{@changes_count}</span>".html_safe, pull_request_changes_project_pull_request_path(@request.id, :project_id => @project.id), :remote => true %></li>
         </ul>
         <div id="merge_record_0">
-          <div class="merge-discussion-content">
+          <%= render :partial => "pull_requests/pull_request_comments" %>
-            <a href="javascript:void(0);" class="link-blue mr15 fl">Hjqreturn</a><span class="fl fontGrey2">20小时前</span>
-            <div class="cl"></div>
-            <div class="merge-discussion-detail">回复内容</div>
-          </div>
-          <div class="merge-discussion-content merge-discussion-automatic">
-            <a href="javascript:void(0);" class="link-blue mr15 fl">Hjqreturn</a><span class="fl fontGrey2">20小时前</span>
-            <div class="cl"></div>
-            <div class="merge-discussion-detail">已关闭请求/重新审核请求</div>
-          </div>
-          <div class="merge-discussion-input">
-            <textarea></textarea>
-            <a href="javascript:void(0);" class="BlueCirBtn">提 交</a>
-          </div>
         </div>
         <div id="merge_record_1" class="undis">
           <%= render :partial => "pull_requests/pull_request_commits" %>

app/views/pull_requests/pull_request_changes.js.erb

@@ -1 +1 @@
-$("#merge_record_1").html('<%= escape_javascript( render :partial => "pull_requests/pull_request_changes",  :locals => {:type => @type} ) %>');
+$("#merge_record_2").html('<%= escape_javascript( render :partial => "pull_requests/pull_request_changes",  :locals => {:type => @type} ) %>');

app/views/pull_requests/pull_request_comments.js.erb

@@ -0,0 +1 @@
+$("#merge_record_0").html('<%= escape_javascript( render :partial => "pull_requests/pull_request_comments") %>');

app/views/pull_requests/pull_request_commits.js.erb

@@ -1 +1 @@
-$("#merge_record_0").html('<%= escape_javascript( render :partial => "pull_requests/pull_request_commits",  :locals => {:type => @type} ) %>');
+$("#merge_record_1").html('<%= escape_javascript( render :partial => "pull_requests/pull_request_commits",  :locals => {:type => @type} ) %>');