artist_url.rb: remove legacy artist url normalization code.
* `legacy_normalize` came from c6012535, which is no longer a problem.
* `normalize_for_search` is only used for "[mass edit]" links
in artist entries. These links are a shortcut for performing a
`-artist_name source:<artist_url> -> artist_name` mass edit to tag
untagged artists, but this won't work for most sites these days.
This commit is contained in:
@@ -45,35 +45,6 @@ class ArtistUrl < ApplicationRecord
|
|||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
def self.legacy_normalize(url)
|
|
||||||
if url.nil?
|
|
||||||
nil
|
|
||||||
else
|
|
||||||
url = url.gsub(%r!^https://!, "http://")
|
|
||||||
url = url.gsub(%r!^http://blog\d+\.fc2!, "http://blog.fc2")
|
|
||||||
url = url.gsub(%r!^http://blog-imgs-\d+\.fc2!, "http://blog.fc2")
|
|
||||||
url = url.gsub(%r!^http://blog-imgs-\d+-\w+\.fc2!, "http://blog.fc2")
|
|
||||||
url = url.gsub(%r!^http://img\d+\.pixiv\.net!, "http://img.pixiv.net")
|
|
||||||
url = url.gsub(%r!^http://i\d+\.pixiv\.net/img\d+!, "http://img.pixiv.net")
|
|
||||||
url = url.gsub(%r!/+\Z!, "")
|
|
||||||
url + "/"
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
def self.normalize_for_search(url)
|
|
||||||
if url =~ /\.\w+\Z/ && url =~ /\w\/\w/
|
|
||||||
url = File.dirname(url)
|
|
||||||
end
|
|
||||||
|
|
||||||
url = url.gsub(%r!^https://!, "http://")
|
|
||||||
url = url.gsub(%r!^http://blog\d+\.fc2!, "http://blog*.fc2")
|
|
||||||
url = url.gsub(%r!^http://blog-imgs-\d+\.fc2!, "http://blog*.fc2")
|
|
||||||
url = url.gsub(%r!^http://blog-imgs-\d+-\w+\.fc2!, "http://blog*.fc2")
|
|
||||||
url = url.gsub(%r!^http://img\d+\.pixiv\.net!, "http://img*.pixiv.net")
|
|
||||||
url = url.gsub(%r!^http://i\d+\.pixiv\.net/img\d+!, "http://*.pixiv.net/img*")
|
|
||||||
url
|
|
||||||
end
|
|
||||||
|
|
||||||
def parse_prefix
|
def parse_prefix
|
||||||
case url
|
case url
|
||||||
when /^-/
|
when /^-/
|
||||||
|
|||||||
@@ -60,10 +60,6 @@ class ArtistVersion < ApplicationRecord
|
|||||||
new_urls = url_array
|
new_urls = url_array
|
||||||
old_urls = version.present? ? version.url_array : []
|
old_urls = version.present? ? version.url_array : []
|
||||||
|
|
||||||
latest_urls = latest_urls.map {|url| ArtistUrl.legacy_normalize(url)}
|
|
||||||
new_urls = new_urls.map {|url| ArtistUrl.legacy_normalize(url)}
|
|
||||||
old_urls = old_urls.map {|url| ArtistUrl.legacy_normalize(url)}
|
|
||||||
|
|
||||||
added_urls = new_urls - old_urls
|
added_urls = new_urls - old_urls
|
||||||
removed_urls = old_urls - new_urls
|
removed_urls = old_urls - new_urls
|
||||||
|
|
||||||
|
|||||||
@@ -33,9 +33,6 @@
|
|||||||
<% else %>
|
<% else %>
|
||||||
<del><%= h(url.url) %></del>
|
<del><%= h(url.url) %></del>
|
||||||
<% end %>
|
<% end %>
|
||||||
<% if CurrentUser.user.is_moderator? %>
|
|
||||||
[<%= link_to("mass edit", edit_moderator_tag_path(:antecedent => "-#{artist.name} source:#{ArtistUrl.normalize_for_search(url.url)}", :consequent => artist.name)) %>]
|
|
||||||
<% end %>
|
|
||||||
</li>
|
</li>
|
||||||
<% end %>
|
<% end %>
|
||||||
</ul>
|
</ul>
|
||||||
|
|||||||
Reference in New Issue
Block a user