From 3251cdf5d6d4ce51e52d96f52f266ab37f1fa695 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Florian=20Sch=C3=A4fer?= <florian@schaeferban.de>
Date: Sun, 8 Mar 2015 20:39:51 +0100
Subject: [PATCH] Added fixes for rubocop complaints

---
 app/helpers/browse_helper.rb       | 22 ++++++++++------------
 test/helpers/browse_helper_test.rb |  2 +-
 2 files changed, 11 insertions(+), 13 deletions(-)

diff --git a/app/helpers/browse_helper.rb b/app/helpers/browse_helper.rb
index 096f74e2f..8a0721c58 100644
--- a/app/helpers/browse_helper.rb
+++ b/app/helpers/browse_helper.rb
@@ -72,9 +72,9 @@ module BrowseHelper
     elsif wdt = wikidata_links(key, value)
       # IMPORTANT: Note that wikidata_links() returns an array of hashes, unlike for example wikipedia_link(),
       # which just returns one such hash.
-      wdt.map{|w|
-        link_to(w[:title], w[:url], :title => t('browse.tag_details.wikidata_link', :page => w[:title].strip))
-      }.join(";").html_safe
+      wdt.map do |w|
+        link_to(w[:title], w[:url], :title => t("browse.tag_details.wikidata_link", :page => w[:title].strip))
+      end.join(";").html_safe
     elsif url = wiki_link("tag", "#{key}=#{value}")
       link_to h(value), url, :title => t("browse.tag_details.wiki_link.tag", :key => key, :value => value)
     elsif url = telephone_link(key, value)
@@ -169,16 +169,14 @@ module BrowseHelper
         :url => "//www.wikidata.org/wiki/#{value}?uselang=#{I18n.locale}",
         :title => value
       }]
-    elsif (
-      # Key has to be one of the accepted wikidata-tags
-      key =~ /(architect|artist|brand|operator|subject):wikidata/ &&
-      # Value has to be a semicolon-separated list of wikidata-IDs (whitespaces allowed before and after semicolons)
-      value =~ /^[Qq][1-9][0-9]*(\s*;\s*[Qq][1-9][0-9]*)*$/
-    )
+    # Key has to be one of the accepted wikidata-tags
+    elsif key =~ /(architect|artist|brand|operator|subject):wikidata/ &&
+          # Value has to be a semicolon-separated list of wikidata-IDs (whitespaces allowed before and after semicolons)
+          value =~ /^[Qq][1-9][0-9]*(\s*;\s*[Qq][1-9][0-9]*)*$/
       # Splitting at every semicolon to get a separate hash for each wikidata-ID
-      return value.split(";").map { |id|
-        id = {:title => id, :url => "//www.wikidata.org/wiki/#{id.strip}?uselang=#{I18n.locale}" }
-      }
+      return value.split(";").map do |id|
+        { :title => id, :url => "//www.wikidata.org/wiki/#{id.strip}?uselang=#{I18n.locale}" }
+      end
     end
     nil
   end
diff --git a/test/helpers/browse_helper_test.rb b/test/helpers/browse_helper_test.rb
index 19b290516..130bf8590 100644
--- a/test/helpers/browse_helper_test.rb
+++ b/test/helpers/browse_helper_test.rb
@@ -206,7 +206,7 @@ class BrowseHelperTest < ActionView::TestCase
     assert_equal "Q1568346", links[2][:title]
     I18n.locale = "en"
 
-    # and now with whitespaces…
+    # and now with whitespaces...
     links = wikidata_links("subject:wikidata", "Q6542248 ;\tQ180\n ;\rQ364\t\n\r ;\nQ4006")
     assert_equal 4, links.length
     assert_equal "//www.wikidata.org/wiki/Q6542248?uselang=en", links[0][:url]
-- 
2.43.2