X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/12c1d5e6c05813a0697724277b4d8529a1a7c240..9a563403d10df21ce3a5f456b807e70361026fcc:/lib/osm_community_index.rb?ds=inline diff --git a/lib/osm_community_index.rb b/lib/osm_community_index.rb index ca9d2d3b3..a5c2fba72 100644 --- a/lib/osm_community_index.rb +++ b/lib/osm_community_index.rb @@ -6,7 +6,7 @@ module OsmCommunityIndex files = Rails.root.glob("node_modules/osm-community-index/i18n/*.yaml") files.each do |file| locale = File.basename(file, ".yaml") - community_locale_yaml = YAML.safe_load(File.read(file))[locale] + community_locale_yaml = YAML.safe_load_file(file)[locale] # rails wants language-COUNTRY but osm-community-index uses underscores locale_rails = locale.tr("_", "-") @@ -14,9 +14,7 @@ module OsmCommunityIndex 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"] + strings["name"] = resolve_name(community, community_locale_yaml) obj.deep_merge!("osm_community_index" => { "communities" => { id => strings } }) end @@ -24,4 +22,23 @@ module OsmCommunityIndex I18n.backend.store_translations locale_rails, data end end + + def self.resolve_name(community, community_locale_yaml) + # If theres an explicitly translated name then use that + translated_name = community_locale_yaml.dig(community.id, "name") + return translated_name if translated_name + + # If not, then look up the default translated name for this type of community, and interpolate the template + template = community_locale_yaml.dig("_defaults", community.type, "name") + community_name = community_locale_yaml.dig("_communities", community.strings["communityID"]) + # Change the `{community}` placeholder to `%{community}` and use Ruby's Kernel.format to fill it in. + translated_name = format(template.gsub("{", "%{"), { :community => community_name }) if template && community_name + return translated_name if translated_name + + # Otherwise fall back to the (English-language) resource name + return community.strings["name"] if community.strings["name"] + + # Finally use the (English-language) community name + community.strings["community"] + end end