Refactor i18n so that all community information is available

This makes it easier to use non-chapter communities on the site in future.

app/views/site/communities.html.erb

@@ -9,7 +9,7 @@
 <p><%= t ".local_chapters.list_text" %></p>
 <ul>
   <% @local_chapters.each do |chapter| %>
-    <li><a href="<%= chapter.url %>"><%= t "osm_community_index.local_chapter.#{chapter.id}.name" %></a></li>
+    <li><a href="<%= chapter.url %>"><%= t "osm_community_index.communities.#{chapter.id}.name" %></a></li>
   <% end %>
 </ul>
 

config/initializers/osm_community_index.rb

@@ -1,3 +1,3 @@
 Rails.configuration.after_initialize do
-  OsmCommunityIndex::LocalChapter.add_to_i18n
+  OsmCommunityIndex.add_to_i18n
 end

lib/osm_community_index.rb

@@ -1,2 +1,26 @@
 module OsmCommunityIndex
+  def self.add_to_i18n
+    communities = Community.all
+    files = Dir.glob(Rails.root.join("node_modules/osm-community-index/i18n/*"))
+    files.each do |file|
+      locale = File.basename(file, ".yaml")
+      community_locale_yaml = YAML.safe_load(File.read(file))[locale]
+      # rails wants en-GB but osm-community-index has en_GB
+      locale_rails = locale.tr("_", "-")
+      data = {}
+
+      communities.each do |community|
+        id = community[:id]
+
+        strings = community_locale_yaml[id] || {}
+        # if the name isn't defined then fall back on community,
+        # as per discussion here: https://github.com/osmlab/osm-community-index/issues/483
+        strings["name"] = strings["name"] || community["strings"]["name"] || community["strings"]["community"]
+
+        data.deep_merge!({ "osm_community_index" => { "communities" => { id => strings } } })
+      end
+
+      I18n.backend.store_translations locale_rails, data
+    end
+  end
 end

lib/osm_community_index/local_chapter.rb

@@ -1,28 +0,0 @@
-module OsmCommunityIndex
-  class LocalChapter
-    def self.add_to_i18n
-      local_chapters = Community.where(:type => "osm-lc").where.not(:id => "OSMF")
-      files = Dir.glob(Rails.root.join("node_modules/osm-community-index/i18n/*"))
-      files.each do |file|
-        locale = File.basename(file, ".yaml")
-        community_index_yaml = YAML.safe_load(File.read(file))[locale]
-        # rails wants en-GB but osm-community-index has en_GB
-        locale_rails = locale.tr("_", "-")
-        data = {}
-
-        local_chapters.each do |chapter|
-          id = chapter[:id]
-
-          strings = community_index_yaml[id] || {}
-          # if the name isn't defined then fall back on community,
-          # as per discussion here: https://github.com/osmlab/osm-community-index/issues/483
-          strings["name"] = strings["name"] || chapter["strings"]["name"] || chapter["strings"]["community"]
-
-          data.deep_merge!({ "osm_community_index" => { "local_chapter" => { id => strings } } })
-        end
-
-        I18n.backend.store_translations locale_rails, data
-      end
-    end
-  end
-end