Merge pull request #1726 from coderdojo-japan/add-inactivated-at-to-dojo

Add `inactivated_at` to `Dojo` model to replace `is_active` boolean column

Author: yasulab

app/models/dojo.rb

@@ -16,6 +16,12 @@ class Dojo < ApplicationRecord
   scope :default_order, -> { order(prefecture_id: :asc, order: :asc) }
   scope :active,        -> { where(is_active: true ) }
   scope :inactive,      -> { where(is_active: false) }
+  
+  # 新しいスコープ: 特定の日時点でアクティブだったDojoを取得
+  scope :active_at, ->(date) { 
+    where('created_at <= ?', date)
+      .where('inactivated_at IS NULL OR inactivated_at > ?', date) 
+  }
 
   validates :name,        presence: true, length: { maximum: 50 }
   validates :email,       presence: false
@@ -74,8 +80,50 @@ def annual_count(period)
       ]
     end
   end
+  
+  # インスタンスメソッド
+  def active?
+    inactivated_at.nil?
+  end
+  
+  def active_at?(date)
+    created_at <= date && (inactivated_at.nil? || inactivated_at > date)
+  end
+  
+  # 再活性化メソッド
+  def reactivate!
+    if inactivated_at.present?
+      # 非活動期間を note に記録
+      inactive_period = "#{inactivated_at.strftime('%Y-%m-%d')}〜#{Date.today}"
+      
+      if note.present?
+        self.note += "\n非活動期間: #{inactive_period}"
+      else
+        self.note = "非活動期間: #{inactive_period}"
+      end
+    end
+    
+    update!(
+      is_active: true,
+      inactivated_at: nil
+    )
+  end
+  
+  # is_activeとinactivated_atの同期（移行期間中）
+  before_save :sync_active_status
 
   private
+  
+  def sync_active_status
+    if is_active_changed?
+      if is_active == false && inactivated_at.nil?
+        self.inactivated_at = Time.current
+      elsif is_active == true && inactivated_at.present?
+        # is_activeがtrueに変更された場合、inactivated_atをnilに
+        self.inactivated_at = nil
+      end
+    end
+  end
 
   # Now 6+ tags are available since this PR:
   # https://github.com/coderdojo-japan/coderdojo.jp/pull/1697

app/models/high_charts_builder.rb

@@ -65,9 +65,20 @@ def build_annual_participants(source, lang = 'ja')
     private
 
     def annual_chart_data_from(source)
-      years           = source.map(&:first)
-      increase_nums   = source.map(&:last)
-      cumulative_sums = increase_nums.size.times.map {|i| increase_nums[0..i].sum }
+      # sourceがハッシュの場合は配列に変換
+      source_array = source.is_a?(Hash) ? source.to_a : source
+      
+      years           = source_array.map(&:first)
+      counts          = source_array.map(&:last)
+      
+      # 増加数を計算（前年との差分）
+      increase_nums = counts.each_with_index.map do |count, i|
+        i == 0 ? count : count - counts[i - 1]
+      end
+      
+      # annual_dojos_with_historical_dataからの値は既にその時点での総数
+      # （累積値として扱う）
+      cumulative_sums = counts
 
       {
         years: years,

app/models/stat.rb

@@ -32,8 +32,19 @@ def annual_sum_of_participants
   end
 
   def annual_dojos_chart(lang = 'ja')
-    # MEMO: トップページの道場数と一致するように Active Dojo を集計対象としている
-    HighChartsBuilder.build_annual_dojos(Dojo.active.annual_count(@period), lang)
+    # 各年末時点でアクティブだったDojoを集計（過去の非アクティブDojoも含む）
+    # YAMLマスターデータには既にinactivated_atが含まれているため、常にこの方式を使用
+    HighChartsBuilder.build_annual_dojos(annual_dojos_with_historical_data, lang)
+  end
+  
+  # 各年末時点でアクティブだったDojo数を集計（過去の非アクティブDojoも含む）
+  def annual_dojos_with_historical_data
+    (@period.first.year..@period.last.year).each_with_object({}) do |year, hash|
+      end_of_year = Time.zone.local(year).end_of_year
+      # その年の終わりにアクティブだったDojoの数を集計
+      count = Dojo.active_at(end_of_year).sum(:counter)
+      hash[year.to_s] = count
+    end
   end
 
   def annual_event_histories_chart(lang = 'ja')