Use if modifier instead of an if block

This commit is contained in:
Maciej Szlosarczyk 2018-08-17 11:24:00 +03:00
parent 98f55fd48a
commit b57adca624
No known key found for this signature in database
GPG key ID: 41D62D42D3B0D765

View file

@ -36,11 +36,9 @@ module Versions
def user_from_id_role_username(str)
registrar = Registrar.find_by(name: str)
user = registrar.api_users.first if registrar
str_match = str.match(/^(\d+)-(ApiUser:|api-|AdminUser:)/)
if str_match
user ||= User.find_by(id: str_match[1])
end
str_match = str.match(/^(\d+)-(ApiUser:|api-|AdminUser:)/)
user ||= User.find_by(id: str_match[1]) if str_match
user
end