Add 'Until' column to contributors index

Author: joshuay03

app/assets/stylesheets/screen.scss

@@ -330,7 +330,7 @@ TABLES GENERALS
   font-size: 1.1em;
 }
 
-#table-wrap table td.contributor-since {
+#table-wrap table td.contributor-timestamp {
   white-space: nowrap;
   text-align: left;
 }

app/models/contributor.rb

@@ -39,8 +39,8 @@ def self._all_with_ncommits(joins, where=nil)
       order('ncommits DESC, contributors.url_id ASC')
   end
 
-  def self.set_first_contribution_timestamps(only_new)
-    scope = only_new ? 'first_contribution_at IS NULL' : '1 = 1'
+  def self.set_first_contribution_timestamps(only_missing)
+    scope = only_missing ? 'first_contribution_at IS NULL' : '1 = 1'
 
     connection.execute(<<-SQL)
       UPDATE contributors
@@ -56,6 +56,23 @@ def self.set_first_contribution_timestamps(only_new)
     SQL
   end
 
+  def self.set_last_contribution_timestamps(only_missing)
+    scope = only_missing ? 'last_contribution_at IS NULL' : '1 = 1'
+
+    connection.execute(<<-SQL)
+      UPDATE contributors
+      SET last_contribution_at = last_contributions.committer_date
+      FROM (
+        SELECT contributor_id, MAX(commits.committer_date) AS committer_date
+          FROM contributions
+          INNER JOIN commits ON commits.id = commit_id
+          GROUP BY contributor_id
+      ) AS last_contributions
+      WHERE id = last_contributions.contributor_id
+      AND #{scope}
+    SQL
+  end
+
   # The contributors table may change if new name equivalences are added and IDs
   # in particular are expected to move. So, we just put the parameterized name
   # in URLs, which is unique anyway.

app/models/repo.rb

@@ -85,7 +85,7 @@ def sync
         if ncommits > 0 || nreleases > 0 || rebuild_all
           sync_names
           sync_ranks
-          sync_first_contribution_timestamps
+          sync_contribution_timestamps
         end
 
         RepoUpdate.create!(
@@ -188,8 +188,9 @@ def sync_ranks
     end
   end
 
-  def sync_first_contribution_timestamps
+  def sync_contribution_timestamps
     Contributor.set_first_contribution_timestamps(!rebuild_all)
+    Contributor.set_last_contribution_timestamps(!rebuild_all)
   end
 
   # Determines whether the names mapping has been updated. This is useful because

app/views/contributors/_contributor.html.erb

@@ -1,7 +1,8 @@
-<tr class="<%= cycle 'even', 'odd' %>">
+<tr id="<%= contributor.name.downcase.tr(' ', '-') %>" class="<%= cycle 'even', 'odd' %>">
   <td class="contributor-rank">#<%= contributor.rank %></td>
   <td class="contributor-name"><%= link_to_contributor contributor %></td>
-  <td class="contributor-since"><%= date contributor.first_contribution_at %></td>
+  <td class="contributor-timestamp"><%= date contributor.first_contribution_at %></td>
+  <td class="contributor-timestamp"><%= date contributor.last_contribution_at %></td>
   <td class="no-commits">
     <% path = if @time_window
          contributor_commits_in_time_window_path(contributor, @time_window)

app/views/contributors/index.html.erb

@@ -14,6 +14,7 @@
       <th></th>
       <th>Name</th>
       <th>Since</th>
+      <th>Until</th>
       <th>Commits</th>
     </tr>
     <%= render @contributors %>

db/migrate/20150326181907_add_first_contribution_at_to_contributors.rb

@@ -1,7 +1,7 @@
 class AddFirstContributionAtToContributors < ActiveRecord::Migration[4.2]
   def up
     add_column :contributors, :first_contribution_at, :datetime
-    Contributor.try(:fill_missing_first_contribution_timestamps)
+    Contributor.set_first_contribution_timestamps
   end
 
   def down

db/migrate/20240824030051_add_last_contribution_at_to_contributors.rb

@@ -0,0 +1,10 @@
+class AddLastContributionAtToContributors < ActiveRecord::Migration[7.1]
+  def up
+    add_column :contributors, :last_contribution_at, :datetime
+    Contributor.set_last_contribution_timestamps
+  end
+
+  def down
+    remove_column :contributors, :last_contribution_at
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema[7.0].define(version: 2016_05_12_095609) do
+ActiveRecord::Schema[7.1].define(version: 2024_08_24_030051) do
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
 
@@ -42,6 +42,7 @@
     t.string "url_id", null: false
     t.integer "rank"
     t.datetime "first_contribution_at", precision: nil
+    t.datetime "last_contribution_at"
     t.index ["name"], name: "index_contributors_on_name", unique: true
     t.index ["url_id"], name: "index_contributors_on_url_id", unique: true
   end

test/controllers/contributors_controller_test.rb

@@ -3,7 +3,7 @@
 class ContributorsControllerTest < ActionController::TestCase
   def test_index_main_listing
     # Order by ncommits DESC, url_id ASC.
-    expected = [[:jeremy, 3], [:david, 2], [:jose, 1], [:vijay, 1], [:xavier, 1]]
+    expected = [[:jeremy, 3], [:david, 2], [:jose, 2], [:vijay, 1], [:xavier, 1]]
 
     get :index
 
@@ -13,13 +13,27 @@ def test_index_main_listing
     assert_equal expected.size, actual.size
 
     expected.zip(actual).each do |e, a|
-      assert_equal contributors(e.first).name, a.name
+      contributor = contributors(e.first)
+
+      assert_equal contributor.name, a.name
       assert_equal e.second, a.ncommits
+
+      assert_select "tr##{contributor.name.downcase.tr(' ', '-') }" do |elements|
+        assert_select 'td.contributor-rank', "##{contributor.rank.to_s}"
+        assert_select 'td.contributor-name', /#{contributor.name}/
+        assert_select 'td.contributor-timestamp' do |elements|
+          assert_equal 2, elements.size
+          assert_equal contributor.first_contribution_at.strftime("%d %b %Y"), elements[0].text.strip
+          assert_equal contributor.last_contribution_at.strftime("%d %b %Y"), elements[1].text.strip
+        end
+        assert_select "td.no-commits", e.second.to_s
+      end
     end
   end
 
   def test_index_by_release
     releases = {
+      'v4.0.0'  => [[:jose, 1]],
       'v3.2.0'  => [[:jeremy, 1], [:jose, 1], [:vijay, 1]],
       'v0.14.4' => [[:david, 1]]
     }
@@ -48,11 +62,11 @@ def test_in_time_window
     date_range = '20121201-20121231'
 
     time_windows = {
-      'all-time'   => [[:jeremy, 3], [:david, 2], [:jose, 1], [:vijay, 1], [:xavier, 1]],
+      'all-time'   => [[:jeremy, 3], [:david, 2], [:jose, 2], [:vijay, 1], [:xavier, 1]],
       'today'      => [[:jeremy, 1]],
       'this-week'  => [[:jeremy, 1], [:xavier, 1]],
       'this-month' => [[:david, 1], [:jeremy, 1], [:xavier, 1]],
-      'this-year'  => [[:jeremy, 3], [:david, 1], [:jose, 1], [:vijay, 1], [:xavier, 1]],
+      'this-year'  => [[:jeremy, 3], [:jose, 2], [:david, 1], [:vijay, 1], [:xavier, 1]],
       since        => [[:jeremy, 1], [:xavier, 1]],
       date_range   => [[:david, 1], [:jeremy, 1], [:xavier, 1]],
     }

test/controllers/releases_controller_test.rb

@@ -5,9 +5,8 @@ def test_index
     get :index
     assert_response :success
 
-    assert_select 'span.listing-total', 'Showing 5 releases'
-
     expected = %w(
+      v4_0_0
       v3_2_0
       v2_3_2_1
       v2_3_2
@@ -24,7 +23,7 @@ def test_index
       assert_equal e.contributors.count, a.ncontributors
     end
 
-    assert_select 'span.listing-total', 'Showing 5 releases'
+    assert_select 'span.listing-total', "Showing #{expected.size} releases"
     assert_select 'li.current', 'Releases'
   end
 end