Merge pull request #1784 from coderdojo-japan/improve-activity-page-inactive-dojos

活動状況ページで非アクティブ道場も表示し、降順でソート

Author: yasulab

app/controllers/dojos_controller.rb

@@ -113,7 +113,12 @@ def activity
     @inactive_threshold = Dojo::INACTIVE_THRESHOLD_IN_MONTH
 
     @latest_event_by_dojos = []
-    Dojo.active.each do |dojo|
+
+    # アクティブな道場と非アクティブな道場を両方取得
+    # アクティブな道場を先に、非アクティブな道場を後に表示
+    dojos = Dojo.active.to_a + Dojo.inactive.to_a
+
+    dojos.each do |dojo|
       link_in_note = dojo.note.match(URI.regexp)
       # YYYY-MM-DD、YYYY/MM/DD、または YYYY年MM月DD日 形式の日付を抽出
       date_in_note = dojo.note.match(/(\d{4}-\d{1,2}-\d{1,2}|\d{4}\/\d{1,2}\/\d{1,2}|\d{4}年\d{1,2}月\d{1,2}日)/)
@@ -126,6 +131,7 @@ def activity
         note:       dojo.note,
         url:        dojo.url,
         created_at: dojo.created_at,  # 掲載日（/dojos と同じ）
+        is_active:  dojo.active?,   # アクティブ状態を追加
 
         # 直近の開催日（イベント履歴がある場合のみ）
         latest_event_at:  latest_event&.evented_at,
@@ -137,11 +143,24 @@ def activity
       }
     end
 
-    # Sort by latest event date (or created_at if no events) && Dojo's order if same date
-    @latest_event_by_dojos.sort_by! do |dojo|
+    # アクティブな道場と非アクティブな道場を分けてソート
+    active_dojos   = @latest_event_by_dojos.select { |d| d[:is_active] }
+    inactive_dojos = @latest_event_by_dojos.reject { |d| d[:is_active] }
+
+    # それぞれのグループ内でソート
+    active_dojos.sort_by! do |dojo|
       sort_date = dojo[:latest_event_at] || dojo[:note_date] || dojo[:created_at]
       [sort_date, dojo[:order]]
     end
+
+    # 非アクティブな道場は最新の開催日から古い順（降順）にソート
+    inactive_dojos.sort_by! do |dojo|
+      sort_date = dojo[:latest_event_at] || dojo[:note_date] || dojo[:created_at]
+      [-sort_date.to_i, dojo[:order]]  # マイナスを付けて降順にする
+    end
+
+    # アクティブな道場を先に、非アクティブな道場を後に
+    @latest_event_by_dojos = active_dojos + inactive_dojos
   end
 
   private

app/views/dojos/activity.html.erb

@@ -74,18 +74,18 @@
       </tr>
       <% @latest_event_by_dojos.each do |dojo| %>
         <tr>
-          <td>
+          <td class="<%= 'inactive-item' unless dojo[:is_active] %>">
             <small>
               <%= link_to dojo_path(dojo[:id]) do %>
                 <%= dojo[:name] %><br>
                 <small>(ID: <%= dojo[:id] %>)</small>
               <% end %>
             </small>
           </td>
-          <td>
+          <td class="<%= 'inactive-item' unless dojo[:is_active] %>">
             <small><%= dojo[:created_at].strftime("%Y-%m-%d") %></small>
           </td>
-          <td>
+          <td class="<%= 'inactive-item' unless dojo[:is_active] %>">
             <small>
               <% if dojo[:note_date] && dojo[:latest_event_at] %>
                 <!-- 両方の日付がある場合：より新しい方を表示 -->
@@ -128,7 +128,7 @@
               <% end %>
             </small>
           </td>
-          <td>
+          <td class="<%= 'inactive-item' unless dojo[:is_active] %>">
             <small><small>
               <% search_query = URI.encode_www_form(q: "CoderDojo #{dojo[:name].split('@').first.strip}") %>
               <% search_range = URI.encode_www_form(tbs: "qdr:m12") # 過去12ヶ月の検索結果 %>
@@ -138,15 +138,15 @@
               <%= link_to '12ヶ月', [search_url, search_range].join('&'), target: "_blank" %>
             </small></small>
           </td>
-          <td>
+          <td class="<%= 'inactive-item' unless dojo[:is_active] %>">
             <small><small>
               <%= link_to dojo[:url], target: "_blank" do %>
                 Website
 		<small><i class="far fa-external-link"></i></small>
               <% end %>
             </small></small>
           </td>
-          <td class="url-cell">
+          <td class="url-cell <%= 'inactive-item' unless dojo[:is_active] %>">
             <small>
               <span title="<%= dojo[:note] %>">
                 <%= raw Addressable::URI.unescape(truncate_with_preserved_links(dojo[:note], length: 60)) %>

spec/requests/dojos_spec.rb

@@ -346,10 +346,82 @@
       expect(response.body).to include("道場別の活動状況まとめ")
     end
 
-    it "includes only active dojos" do
+    it "includes both active and inactive dojos" do
       get activity_dojos_path
       expect(response.body).to include(@active_dojo.name)
-      expect(response.body).not_to include(@inactive_dojo.name)
+      expect(response.body).to include(@inactive_dojo.name)
+    end
+    
+    it "displays inactive dojos with inactive-item class" do
+      get activity_dojos_path
+      doc = Nokogiri::HTML(response.body)
+      
+      # 非アクティブな道場の行を探す（リンク内にある名前を探す）
+      inactive_link = doc.xpath("//a[contains(., '#{@inactive_dojo.name}')]").first
+      expect(inactive_link).not_to be_nil
+      
+      # そのリンクを含む行（tr）を取得
+      inactive_row = inactive_link.xpath("ancestor::tr").first
+      expect(inactive_row).not_to be_nil
+      
+      # その行のすべてのセルに inactive-item クラスがあることを確認
+      inactive_cells = inactive_row.css('td')
+      expect(inactive_cells.all? { |cell| cell['class']&.include?('inactive-item') }).to be true
+    end
+    
+    it "sorts inactive dojos by latest event date in descending order" do
+      # 複数の非アクティブな道場を作成（異なる作成日で）
+      older_inactive_dojo = create(:dojo,
+        name: "Older Inactive Dojo",
+        created_at: 2.years.ago,  # より古い作成日
+        inactivated_at: 6.months.ago
+      )
+      
+      newer_inactive_dojo = create(:dojo,
+        name: "Newer Inactive Dojo",  
+        created_at: 1.year.ago,  # より新しい作成日
+        inactivated_at: 3.months.ago
+      )
+      
+      # db/static_event_histories.yml の形式に合わせてEventHistory を作成
+      # より古いイベント
+      EventHistory.create!(
+        dojo_id: older_inactive_dojo.id,
+        dojo_name: older_inactive_dojo.name,
+        event_url: 'https://example.com/event1',
+        evented_at: 8.months.ago,
+        participants: 10,
+        service_name: 'static_yml',
+        service_group_id: 'static',
+        event_id: 1
+      )
+      
+      # より新しいイベント
+      EventHistory.create!(
+        dojo_id: newer_inactive_dojo.id,
+        dojo_name: newer_inactive_dojo.name,
+        event_url: 'https://example.com/event2',
+        evented_at: 4.months.ago,  # より最近のイベント
+        participants: 15,
+        service_name: 'static_yml',
+        service_group_id: 'static',
+        event_id: 2
+      )
+      
+      get activity_dojos_path
+      doc = Nokogiri::HTML(response.body)
+      
+      # 全ての道場の名前を取得（テーブルの最初のセルのリンクから）
+      dojo_links = doc.css('table tr td:first-child a')
+      dojo_names = dojo_links.map(&:text).map(&:strip)
+      
+      # 非アクティブな道場の中で、newer_inactive_dojo が older_inactive_dojo より先に表示されることを確認
+      newer_index = dojo_names.find_index { |name| name.include?(newer_inactive_dojo.name) }
+      older_index = dojo_names.find_index { |name| name.include?(older_inactive_dojo.name) }
+      
+      expect(newer_index).not_to be_nil
+      expect(older_index).not_to be_nil
+      expect(newer_index).to be < older_index
     end
 
     it "redirects from old URL /events/latest" do