diff --git a/app/models/artist_url.rb b/app/models/artist_url.rb index cec0b5c1d..3c05a5d04 100644 --- a/app/models/artist_url.rb +++ b/app/models/artist_url.rb @@ -7,6 +7,7 @@ class ArtistUrl < ActiveRecord::Base if url.nil? nil else + url = url.gsub(/^https:\/\//, "http://") url = url.gsub(/^http:\/\/blog\d+\.fc2/, "http://blog.fc2") url = url.gsub(/^http:\/\/blog-imgs-\d+\.fc2/, "http://blog.fc2") url = url.gsub(/^http:\/\/blog-imgs-\d+-\w+\.fc2/, "http://blog.fc2") @@ -22,6 +23,7 @@ class ArtistUrl < ActiveRecord::Base url = File.dirname(url) end + url = url.gsub(/^https:\/\//, "http://") url = url.gsub(/^http:\/\/blog\d+\.fc2/, "http://blog*.fc2") url = url.gsub(/^http:\/\/blog-imgs-\d+\.fc2/, "http://blog*.fc2") url = url.gsub(/^http:\/\/blog-imgs-\d+-\w+\.fc2/, "http://blog*.fc2") diff --git a/app/views/admin/users/edit.html.erb b/app/views/admin/users/edit.html.erb index e388df586..cb70f5b58 100644 --- a/app/views/admin/users/edit.html.erb +++ b/app/views/admin/users/edit.html.erb @@ -15,6 +15,8 @@ +<%= render "users/secondary_links" %> + <% content_for(:page_title) do %> Edit User - <%= Danbooru.config.app_name %> <% end %> diff --git a/test/unit/artist_test.rb b/test/unit/artist_test.rb index 5d58a64db..3a0cda4de 100644 --- a/test/unit/artist_test.rb +++ b/test/unit/artist_test.rb @@ -94,10 +94,13 @@ class ArtistTest < ActiveSupport::TestCase should "find matches by url" do a1 = FactoryGirl.create(:artist, :name => "rembrandt", :url_string => "http://rembrandt.com/test.jpg") a2 = FactoryGirl.create(:artist, :name => "subway", :url_string => "http://subway.com/test.jpg") + a3 = FactoryGirl.create(:artist, :name => "minko", :url_string => "https://minko.com/test.jpg") assert_equal(["rembrandt"], Artist.find_all_by_url("http://rembrandt.com/test.jpg").map(&:name)) assert_equal(["rembrandt"], Artist.find_all_by_url("http://rembrandt.com/another.jpg").map(&:name)) assert_equal([], Artist.find_all_by_url("http://nonexistent.com/test.jpg").map(&:name)) + assert_equal(["minko"], Artist.find_all_by_url("https://minko.com/test.jpg").map(&:name)) + assert_equal(["minko"], Artist.find_all_by_url("http://minko.com/test.jpg").map(&:name)) end should "not allow duplicates" do diff --git a/test/unit/artist_url_test.rb b/test/unit/artist_url_test.rb index 5575201db..eaf4da618 100644 --- a/test/unit/artist_url_test.rb +++ b/test/unit/artist_url_test.rb @@ -22,6 +22,12 @@ class ArtistUrlTest < ActiveSupport::TestCase assert_equal("http://monet.com/", url.url) assert_equal("http://monet.com/", url.normalized_url) end + + should "normalise https" do + url = FactoryGirl.create(:artist_url, :url => "https://google.com") + assert_equal("https://google.com", url.url) + assert_equal("http://google.com/", url.normalized_url) + end should "normalize fc2 urls" do url = FactoryGirl.create(:artist_url, :url => "http://blog55.fc2.com/monet")