From: Adam Hoyle <atomoil@gmail.com>
Date: Tue, 24 Aug 2021 20:12:48 +0000 (+0100)
Subject: Tidy
X-Git-Tag: live~1792^2~40
X-Git-Url: https://git.openstreetmap.org./rails.git/commitdiff_plain/94f9c2b86f4dc1ca65f3b259a36987763fc4daa8

Tidy
---

diff --git a/app/models/communities.rb b/app/models/communities.rb
index cd8d6c0fe..05e330b26 100644
--- a/app/models/communities.rb
+++ b/app/models/communities.rb
@@ -1,70 +1,55 @@
-
 class Communities
-
-  require 'yaml'
+  require "yaml"
 
   @local_chapters = {}
 
   def self.local_chapters(locale)
-    puts "locale is "+ locale.to_s
-    @local_chapters[locale] = self.local_chapter_for(locale)
+    @local_chapters[locale] = local_chapter_for(locale)
   end
 
-  protected
-
   def self.local_chapter_for(locale)
-
-    @local_chapters_index = self.load_local_chapters
-
-    locale_dict = self.locale_dict_for(locale)
-
+    @local_chapters_index = load_local_chapters
+    locale_dict = locale_dict_for(locale)
     localised_chapters = []
     @local_chapters_index.each do |chapter|
       id = chapter[:id]
-      name = locale_dict.dig(id,"name") || chapter[:name]
+      name = locale_dict.dig(id, "name") || chapter[:name]
       url = chapter[:url]
-      localised_chapters.push({ id: id, name: name, url: url })
+      localised_chapters.push({ :id => id, :name => name, :url => url })
     end
-    puts localised_chapters
     localised_chapters
   end
 
   def self.load_local_chapters
-
-    json_file = File.expand_path("node_modules/osm-community-index/dist/resources.json", Dir.pwd);
+    json_file = File.expand_path("node_modules/osm-community-index/dist/resources.json", Dir.pwd)
     community_index = JSON.parse(File.read(json_file))
-
     local_chapters = []
-    community_index['resources'].each do |id, resource|
+    community_index["resources"].each do |id, resource|
       resource.each do |key, value|
-        if key == "type" and value == "osm-lc" and id != "OSMF"
-          strings = resource['strings']
-          chapter_name = strings['community'] ||strings['name']
-          url = strings['url']
-          local_chapters.push({ id: id, name: chapter_name, url: url})
-        end
+        next unless key == "type" && value == "osm-lc" && id != "OSMF"
+
+        strings = resource["strings"]
+        chapter_name = strings["community"] || strings["name"]
+        url = strings["url"]
+        local_chapters.push({ :id => id, :name => chapter_name, :url => url })
       end
     end
-    return local_chapters
+    local_chapters
   end
 
-
-  def self.locale_dict_for(localeIn)
-    locale = localeIn.to_s.gsub("-","_")
-    full_local_path = File.expand_path("node_modules/osm-community-index/i18n/"+locale+".yaml", Dir.pwd)
+  def self.locale_dict_for(locale_in)
+    locale = locale_in.to_s.tr("-", "_")
+    full_local_path = File.expand_path("node_modules/osm-community-index/i18n/#{locale}.yaml", Dir.pwd)
     locale_dict = {}
-    if File.exists?(full_local_path)
-      locale_dict = YAML.load(File.read(full_local_path))[locale]
+    if File.exist?(full_local_path)
+      locale_dict = YAML.safe_load(File.read(full_local_path))[locale]
     else
       shortened_locale = locale.split("_").first
       if shortened_locale != locale
-        shortened_local_path = File.expand_path("node_modules/osm-community-index/i18n/"+shortened_locale+".yaml", Dir.pwd)
-        if File.exists?(shortened_local_path)
-          locale_dict = YAML.load(File.read(shortened_local_path))[shortened_locale]
-        end
+        shortened_local_path = File.expand_path("node_modules/osm-community-index/i18n/#{shortened_locale}.yaml", Dir.pwd)
+        locale_dict = YAML.safe_load(File.read(shortened_local_path))[shortened_locale] if File.exist?(shortened_local_path)
       end
     end
-    return locale_dict
+    locale_dict
   end
-
 end
diff --git a/app/views/site/communities.html.erb b/app/views/site/communities.html.erb
index b92b3140c..d09c36caa 100644
--- a/app/views/site/communities.html.erb
+++ b/app/views/site/communities.html.erb
@@ -16,4 +16,4 @@
   </ul>
   <h2><%= t ".other_groups.title", :locale => @locale %></h2>
   <p><%= t ".other_groups.about_html", :locale => @locale %></p>
-<% end %>
\ No newline at end of file
+<% end %>