From 83a0ea14f1305cd0a62ee2caeb0142fe077d886c Mon Sep 17 00:00:00 2001 From: Andy Allan Date: Wed, 3 Aug 2022 10:11:30 +0100 Subject: [PATCH] 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 | 2 +- config/initializers/osm_community_index.rb | 2 +- lib/osm_community_index.rb | 24 +++++++++++++++++++ lib/osm_community_index/local_chapter.rb | 28 ---------------------- 4 files changed, 26 insertions(+), 30 deletions(-) delete mode 100644 lib/osm_community_index/local_chapter.rb diff --git a/app/views/site/communities.html.erb b/app/views/site/communities.html.erb index aee536c05..0e5dec3e7 100644 --- a/app/views/site/communities.html.erb +++ b/app/views/site/communities.html.erb @@ -9,7 +9,7 @@

<%= t ".local_chapters.list_text" %>

diff --git a/config/initializers/osm_community_index.rb b/config/initializers/osm_community_index.rb index 1400ef028..0a32bf2eb 100644 --- a/config/initializers/osm_community_index.rb +++ b/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 diff --git a/lib/osm_community_index.rb b/lib/osm_community_index.rb index 519306d8f..1e868580d 100644 --- a/lib/osm_community_index.rb +++ b/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 diff --git a/lib/osm_community_index/local_chapter.rb b/lib/osm_community_index/local_chapter.rb deleted file mode 100644 index d83f13471..000000000 --- a/lib/osm_community_index/local_chapter.rb +++ /dev/null @@ -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 -- 2.39.5