Use reports_count so that counter_cache works automatically.

diff --git a/app/models/report.rb b/app/models/report.rb
index fa90cf04e949530fba904661cd40e862acfcafbd..477b4b1824842d37997e8cc2e05054dbf5dc4706 100644 (file)
--- a/app/models/report.rb
+++ b/app/models/report.rb
@@ -1,4 +1,4 @@
 class Report < ActiveRecord::Base
 class Report < ActiveRecord::Base

-  belongs_to :issue
+  belongs_to :issue, :counter_cache => true

   belongs_to :user, :class_name => "User", :foreign_key => :reporter_user_id
 end
   belongs_to :user, :class_name => "User", :foreign_key => :reporter_user_id
 end

diff --git a/app/views/issues/index.html.erb b/app/views/issues/index.html.erb
index 0eb2108b903072d117d7c49670cb6d525a90fed6..f9f2c28f96ef9656903d554ee477ba6ca770d31f 100644 (file)
--- a/app/views/issues/index.html.erb
+++ b/app/views/issues/index.html.erb
@@ -19,7 +19,7 @@ Search for a particular issue(s):  <br/>
   <thead>
     <tr>
       <td style="width:40px;"><b> <%= sortable("status") %></b></td>
   <thead>
     <tr>
       <td style="width:40px;"><b> <%= sortable("status") %></b></td>

-      <td style="width:160px;"><b> <%= sortable("report_count", "Number of Reports") %></b></td>
+      <td style="width:160px;"><b> <%= sortable("reports_count", "Number of Reports") %></b></td>

       <td style="width:141px;"><b> <%= sortable("updated_at","Last updated at") %></b></td>
       <td style="width:140px;"><b> <%= sortable("updated_by","Last updated by") %></b></td>
       <td style="width:203px;"><b> Link to reports </b></td>
       <td style="width:141px;"><b> <%= sortable("updated_at","Last updated at") %></b></td>
       <td style="width:140px;"><b> <%= sortable("updated_by","Last updated by") %></b></td>
       <td style="width:203px;"><b> Link to reports </b></td>


@@ -31,7 +31,7 @@ Search for a particular issue(s):  <br/>
     <% @issues.each do |issue| %>
       <tr>
         <td><%= issue.status.humanize %></td>
     <% @issues.each do |issue| %>
       <tr>
         <td><%= issue.status.humanize %></td>

-        <td style="text-align:center;"><%= issue.report_count %></td>
+        <td style="text-align:center;"><%= issue.reports_count %></td>

         <td><%= l(issue.updated_at.to_datetime, :format => :friendly) %></td>
         <td><% if issue.user_updated %> <%= issue.user_updated.display_name %> <% else %> - <% end %></td>
         <td><%= reports_url(issue) %></td>
         <td><%= l(issue.updated_at.to_datetime, :format => :friendly) %></td>
         <td><% if issue.user_updated %> <%= issue.user_updated.display_name %> <% else %> - <% end %></td>
         <td><%= reports_url(issue) %></td>

diff --git a/db/migrate/20160822153153_add_report_count_to_issues.rb b/db/migrate/20160822153153_add_reports_count_to_issues.rb
similarity index 59%
rename from db/migrate/20160822153153_add_report_count_to_issues.rb
rename to db/migrate/20160822153153_add_reports_count_to_issues.rb
index 17a1151e740c6aacb2c118642617eaf5494a61cf..a7ccd228f8f31da51c25479218d171e6aae35bf5 100644 (file)
--- a/db/migrate/20160822153153_add_report_count_to_issues.rb
+++ b/db/migrate/20160822153153_add_reports_count_to_issues.rb
@@ -1,6 +1,6 @@
-class AddReportCountToIssues < ActiveRecord::Migration
+class AddReportsCountToIssues < ActiveRecord::Migration

   def change
   def change

-    add_column :issues, :report_count, :integer, :default => 0
+    add_column :issues, :reports_count, :integer, :default => 0

     add_foreign_key :issues, :users, :column => :updated_by, :name => "issues_updated_by_fkey", :on_delete => :cascade
     add_index :issues, :updated_by
   end
     add_foreign_key :issues, :users, :column => :updated_by, :name => "issues_updated_by_fkey", :on_delete => :cascade
     add_index :issues, :updated_by
   end