]> source.dussan.org Git - redmine.git/commitdiff
Fixed that issue counts are 0 on project overview (#26376).
authorJean-Philippe Lang <jp_lang@yahoo.fr>
Sat, 8 Jul 2017 07:20:01 +0000 (07:20 +0000)
committerJean-Philippe Lang <jp_lang@yahoo.fr>
Sat, 8 Jul 2017 07:20:01 +0000 (07:20 +0000)
git-svn-id: http://svn.redmine.org/redmine/trunk@16766 e93f8b46-1217-0410-a6f0-8f06a7374b81

app/controllers/projects_controller.rb
app/views/projects/show.html.erb
test/functional/projects_controller_test.rb

index 3e3fc69f323634975cf859ec02dd9bae8c514423..bded5f2518ee5c4cc177f42b72e53279a39a361a 100644 (file)
@@ -147,7 +147,6 @@ class ProjectsController < ApplicationController
     end
 
     @users_by_role = @project.users_by_role
-    @subprojects = @project.children.visible.to_a
     @news = @project.news.limit(5).includes(:author, :project).reorder("#{News.table_name}.created_on DESC").to_a
     @trackers = @project.rolled_up_trackers.visible
 
@@ -160,6 +159,8 @@ class ProjectsController < ApplicationController
       @total_hours = TimeEntry.visible.where(cond).sum(:hours).to_f
     end
 
+    @subprojects = @project.children.visible.to_a
+
     @key = User.current.rss_key
 
     respond_to do |format|
index 7de31023c1660bcb6d987101fa07e020d0396351..2368a29c42bf37519fbf3c3380236bc39b886622 100644 (file)
@@ -59,7 +59,7 @@
           <td>
             <%= link_to (@total_issues_by_tracker[tracker].to_i - @open_issues_by_tracker[tracker].to_i), project_issues_path(@project, :set_filter => 1, :tracker_id => tracker.id, :status_id => 'c') %>
           </td>
-          <td>
+          <td class="total">
             <%= link_to @total_issues_by_tracker[tracker].to_i, project_issues_path(@project, :set_filter => 1, :tracker_id => tracker.id, :status_id => '*') %>
           </td>
         </tr>
index b0feee8271618ade72785151c5066357d1111923..f34627e5b760affc9e25a4028ae80efda25b6a89 100644 (file)
@@ -543,6 +543,16 @@ class ProjectsControllerTest < Redmine::ControllerTest
     assert_select 'a', :text => /Private child/
   end
 
+  def test_show_by_member_on_leaf_project_should_display_issue_counts
+    @request.session[:user_id] = 2
+    get :show, :params => {
+        :id => 'onlinestore'
+      }
+    assert_response :success
+    # Make sure there's a > 0 issue count
+    assert_select 'table.issue-report td.total a', :text => %r{\A[1-9]\d*\z}
+  end
+
   def test_settings
     @request.session[:user_id] = 2 # manager
     get :settings, :params => {