From 61086f9905f2260ef48308639de706c5b3c4769e Mon Sep 17 00:00:00 2001
From: Yohei Yasukawa <yohei@yasslab.jp>
Date: Wed, 25 Dec 2024 21:57:17 +0900
Subject: [PATCH] Detect and alert if in-active dojo found in a get_dojo_list
 task

---
 get_dojo_list.rb  | 21 +++++++++++----------
 get_dojo_list.txt |  5 +++--
 2 files changed, 14 insertions(+), 12 deletions(-)

diff --git a/get_dojo_list.rb b/get_dojo_list.rb
index 549ef2b5a..594ede0b8 100755
--- a/get_dojo_list.rb
+++ b/get_dojo_list.rb
@@ -2,7 +2,6 @@
 
 require 'net/http'
 require 'json'
-require 'pry'
 
 # Google Spreadsheet などから対象となった Dojo 名の列をコピーし、
 # get_dojo_list.txt にペースト後、本スクリプトを実行すると、
@@ -39,10 +38,10 @@
     .split('、').first               # Ex: 東大阪、八尾
     .strip
 
-  # Search dojo data by its KANJI name from DOJO_DB
-  dojo_data = DOJO_DB.find do |dojo|
-    binding.pry if dojo_name.nil?
-    dojo[:name].start_with? dojo_name.downcase
+  # Search dojo data by its KANJI name from DOJO_DB (including inactive dojos).
+  # MEMO: Use `.reverse` to find a latest dojo in case of overriding inactive dojo's name.
+  found_dojo = DOJO_DB.reverse.find do |dojo|
+    dojo[:name] == dojo_name.downcase
       .gsub('ishigaki',       '石垣')
       .gsub('hitachinaka',    'ひたちなか')
       .gsub('kodaira',        'こだいら')
@@ -56,23 +55,25 @@
       .gsub('harumi',         '晴海')
       .gsub('町田',           'まちだ')
       .gsub('小平',           'こだいら')
+      .gsub('八戸',           '八戸@吹上')
       .gsub('吉備岡山',       '吉備')
       .gsub('浦和@urawa minecraft club', '浦和@Urawa Minecraft Club')
   end
 
-  dojo_data.nil? ?
-    not_found << dojo_name :
-    dojo_list << dojo_data
+  (found_dojo && found_dojo[:is_active]) ?
+    dojo_list << found_dojo :
+    not_found << dojo_name
 end
 
 dojo_list.sort_by!{ |dojo| dojo[:order] }
-result <<  dojo_list.map{ |dojo| "  <li><a href='#{dojo[:url]}'>#{dojo[:name]}</a><small>（#{dojo[:prefecture]}）</small></li>" }.join("\n")
+result <<  dojo_list.map{ |dojo| "#{dojo[:is_active]}  <li><a href='#{dojo[:url]}'>#{dojo[:name]}</a><small>（#{dojo[:prefecture]}）</small></li>" }.join("\n")
 result << "\n</ul>\n"
 puts result
 
+#binding.irb
 # 検索して見つからなかった Dojo 一覧があれば出力
 if not_found.any?
   puts '--- NOTE ---'
   puts "道場数: #{dojo_list.count}"
-  not_found.each {|dojo_name| puts "Not found: #{dojo_name}" }
+  not_found.each {|dojo_name| puts "Not-found or In-active: #{dojo_name}" }
 end
diff --git a/get_dojo_list.txt b/get_dojo_list.txt
index 4ae4700c6..62e84964d 100644
--- a/get_dojo_list.txt
+++ b/get_dojo_list.txt
@@ -30,7 +30,8 @@ CoderDojo 稲沢正明寺
 CoderDojo岐阜
 CoderDojo伊勢
 CoderDojo なかもず
-CoderDojo 奈良・生駒
+CoderDojo 奈良
+CoderDojo 生駒
 CoderDojo南紀田辺
 CoderDojo 淡路島
 CoderDojo 鳥取
@@ -50,7 +51,7 @@ CoderDojo 富山
 CoderDojoとなみ
 CoderDojo Yabuki 
 CoderDojo仙台若林
-CoderDojo徳島
+CoderDojo登米
 CoderDojo 山形
 CoderDojo Tsuruoka (鶴岡)
 CoderDojo 札幌