From: Adam Hoyle <atomoil@gmail.com>
Date: Sun, 17 Oct 2021 22:34:26 +0000 (+0100)
Subject: Pay heed to rubocop
X-Git-Tag: live~1451^2~29
X-Git-Url: https://git.openstreetmap.org./rails.git/commitdiff_plain/f234bf3a462effeffdebbc18d67abec4f848b6b7?ds=inline

Pay heed to rubocop
---

diff --git a/lib/osm_community_index/local_chapter.rb b/lib/osm_community_index/local_chapter.rb
index 6b203afe8..fc87f64d2 100644
--- a/lib/osm_community_index/local_chapter.rb
+++ b/lib/osm_community_index/local_chapter.rb
@@ -1,6 +1,5 @@
 module OsmCommunityIndex
   class LocalChapter
-
     attr_reader :id, :name, :url
 
     @localised_chapters = {}
@@ -15,8 +14,6 @@ module OsmCommunityIndex
       @localised_chapters[locale] ||= load_local_chapters(locale)
     end
 
-    protected
-
     def self.load_local_chapters(locale)
       community_index = OsmCommunityIndex.community_index
       localised_strings = OsmCommunityIndex.localised_strings(locale)
@@ -33,7 +30,5 @@ module OsmCommunityIndex
       end
       local_chapters
     end
-
   end
-
-end
\ No newline at end of file
+end
diff --git a/lib/osm_community_index/osm_community_index.rb b/lib/osm_community_index/osm_community_index.rb
index 987f94614..caa47304c 100644
--- a/lib/osm_community_index/osm_community_index.rb
+++ b/lib/osm_community_index/osm_community_index.rb
@@ -12,8 +12,6 @@ module OsmCommunityIndex
       @localised_strings[locale] ||= locale_hash_from_json(locale)
     end
 
-    protected
-
     def self.community_index_from_json
       json_file = Rails.root.join("node_modules/osm-community-index/dist/resources.json")
       JSON.parse(File.read(json_file))
@@ -23,17 +21,13 @@ module OsmCommunityIndex
       locale = locale_in.to_s.tr("-", "_")
       # try the passed in locale
       json = load_locale_json(locale)
-      unless json.nil?
-        return json
-      end
+      return json unless json.nil?
 
       # now try it without it's country part (eg 'en' instead of 'en_GB')
       shortened_locale = locale.split("_").first
-      unless shortened_locale === locale
+      unless shortened_locale == locale
         json = load_locale_json(shortened_locale)
-        unless json.nil?
-          return json
-        end
+        return json unless json.nil?
       end
 
       # if nothing else works, then return "en"
@@ -42,11 +36,9 @@ module OsmCommunityIndex
 
     def self.load_locale_json(locale)
       json_path = Rails.root.join("node_modules/osm-community-index/i18n/#{locale}.yaml")
-      if File.exist?(json_path)
-        return YAML.safe_load(File.read(json_path))[locale]
-      end
+      return YAML.safe_load(File.read(json_path))[locale] if File.exist?(json_path)
+
       nil
     end
-
   end
-end
\ No newline at end of file
+end