Fix new rubocop warnings
authorTom Hughes <tom@compton.nu>
Wed, 3 Jun 2020 16:47:35 +0000 (17:47 +0100)
committerTom Hughes <tom@compton.nu>
Wed, 3 Jun 2020 16:47:35 +0000 (17:47 +0100)
app/controllers/api/amf_controller.rb
app/helpers/browse_tags_helper.rb

index db8982d43278237df950f2889b5078b84aef69f2..c05a130981cdf46bb97cea3af2dca125362822dd 100644 (file)
@@ -868,7 +868,7 @@ module Api
     # not just the id, hence this abstraction
 
     def getuser(token)
-      if token =~ /^(.+)\:(.+)$/
+      if token =~ /^(.+):(.+)$/
         User.authenticate(:username => Regexp.last_match(1), :password => Regexp.last_match(2))
       else
         User.authenticate(:token => token)
index 4f73eb9d7c8018f932471ac71b98e6f8f22645df..fc25325a4b0bf07472d2a2a824608901843650b7 100644 (file)
@@ -155,7 +155,7 @@ module BrowseTagsHelper
     #
     # Also accepting / as a visual separator although not given in RFC 3966,
     # because it is used as a visual separator in OSM data in some countries.
-    if value.match?(%r{^\s*\+[\d\s\(\)/\.-]{6,25}\s*(;\s*\+[\d\s\(\)/\.-]{6,25}\s*)*$})
+    if value.match?(%r{^\s*\+[\d\s()/.-]{6,25}\s*(;\s*\+[\d\s()/.-]{6,25}\s*)*$})
       return value.split(";").map do |phone_number|
         # for display, remove leading and trailing whitespace
         phone_number = phone_number.strip