treehouse
/
mastodon
22
8
Fork 5
Code Issues 32 Pull Requests Projects 1 Releases Wiki Activity

Micro-optimization: only split `acct` into two Strings (#19901) 

* Since `acct` is split by `@` and assigned to `username` and `domain`, we only need to split `acct` into two Strings.
lolsob-rspec
Postmodern 2022-11-07 07:17:55 -08:00 committed by GitHub
parent 3f108bf80b
commit 41d43a1e1f
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/models/account_alias.rb
View File

@ -29,7 +29,7 @@ class AccountAlias < ApplicationRecord
end end
def pretty_acct def pretty_acct
username, domain = acct.split('@') username, domain = acct.split('@', 2)
domain.nil? ? username : "#{username}@#{Addressable::IDNA.to_unicode(domain)}" domain.nil? ? username : "#{username}@#{Addressable::IDNA.to_unicode(domain)}"
end end