refactor: is_activeカラムの参照をinactivated_atベースに変更

実装内容:
- Dojoモデル: active/inactiveスコープの内部実装を変更
- order_by_active_statusスコープを追加して可読性向上
- sync_active_status関連コードを削除（移行期間終了）
- reactivate!メソッドからis_active更新を削除
- コントローラー: ソート条件を新しいスコープに変更
- Rakeタスク: is_active設定行を削除

これにより既存のインターフェースを維持しながら、内部実装を
inactivated_atベースに統一。is_activeカラム削除の準備完了。

refs #1734

Author: yasulab

app/controllers/dojos_controller.rb

@@ -28,15 +28,15 @@ def index
     end
 
     @dojos = []
-    dojos_scope.includes(:prefecture).order(is_active: :desc, order: :asc).each do |dojo|
+    dojos_scope.includes(:prefecture).order_by_active_status.order(order: :asc).each do |dojo|
       # 年が選択されている場合は、その年末時点でのアクティブ状態を判定
       # 選択されていない場合は、現在の is_active を使用
       is_active_at_selected_time = if @selected_year
         # その年末時点でアクティブだったかを判定
         # inactivated_at が nil（まだアクティブ）または選択年より後に非アクティブ化
         dojo.inactivated_at.nil? || dojo.inactivated_at > Time.zone.local(@selected_year).end_of_year
       else
-        dojo.is_active
+        dojo.active?
       end
 
       @dojos << {

app/models/dojo.rb

@@ -14,8 +14,13 @@ class Dojo < ApplicationRecord
   before_save { self.email = self.email.downcase }
 
   scope :default_order, -> { order(prefecture_id: :asc, order: :asc) }
-  scope :active,        -> { where(is_active: true ) }
-  scope :inactive,      -> { where(is_active: false) }
+  scope :active,        -> { where(inactivated_at: nil) }
+  scope :inactive,      -> { where.not(inactivated_at: nil) }
+  
+  # ソート用スコープ: アクティブな道場を先に表示
+  scope :order_by_active_status, -> {
+    order(Arel.sql('CASE WHEN inactivated_at IS NULL THEN 0 ELSE 1 END'))
+  }
 
   # 新しいスコープ: 特定の日時点でアクティブだったDojoを取得
   scope :active_at, ->(date) { 
@@ -103,27 +108,10 @@ def reactivate!
       end
     end
 
-    update!(
-      is_active: true,
-      inactivated_at: nil
-    )
+    update!(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

lib/tasks/dojos.rake

@@ -44,7 +44,6 @@ namespace :dojos do
       d.url            = dojo['url']
       d.prefecture_id  = dojo['prefecture_id']
       d.order          = dojo['order'] || search_order_number_by(dojo['name'])
-      d.is_active      = dojo['is_active'].nil?  ? true  : dojo['is_active']
       d.is_private     = dojo['is_private'].nil? ? false : dojo['is_private']
       d.inactivated_at = dojo['inactivated_at'] ? Time.zone.parse(dojo['inactivated_at']) : nil
       d.created_at     = d.new_record? ? Time.zone.now : dojo['created_at'] || d.created_at