Partial fix for #3514: Handle https://twitter.com/i/web/status/:id URL.
This commit is contained in:
@@ -65,8 +65,10 @@ module Sources::Strategies
|
|||||||
self.class.status_id_from_url(referer_url)
|
self.class.status_id_from_url(referer_url)
|
||||||
end
|
end
|
||||||
|
|
||||||
|
# https://twitter.com/i/web/status/943446161586733056
|
||||||
|
# https://twitter.com/motty08111213/status/943446161586733056
|
||||||
def self.status_id_from_url(url)
|
def self.status_id_from_url(url)
|
||||||
if url =~ %r{^https?://(?:mobile\.)?twitter\.com/\w+/status/(\d+)}
|
if url =~ %r{\Ahttps?://(?:mobile\.)?twitter\.com/(?:i/web|\w+)/status/(\d+)}i
|
||||||
$1.to_i
|
$1.to_i
|
||||||
else
|
else
|
||||||
nil
|
nil
|
||||||
|
|||||||
@@ -125,6 +125,7 @@ class Artist < ApplicationRecord
|
|||||||
"twitpic.com",
|
"twitpic.com",
|
||||||
"twitpic.com/photos", # http://twitpic.com/photos/Type10TK
|
"twitpic.com/photos", # http://twitpic.com/photos/Type10TK
|
||||||
"twitter.com", # https://twitter.com/akkij0358
|
"twitter.com", # https://twitter.com/akkij0358
|
||||||
|
"twitter.com/i/web/status", # https://twitter.com/i/web/status/943446161586733056
|
||||||
"ustream.tv",
|
"ustream.tv",
|
||||||
"ustream.tv/channel", # http://www.ustream.tv/channel/633b
|
"ustream.tv/channel", # http://www.ustream.tv/channel/633b
|
||||||
"ustream.tv/user", # http://www.ustream.tv/user/kazaputi
|
"ustream.tv/user", # http://www.ustream.tv/user/kazaputi
|
||||||
|
|||||||
@@ -138,6 +138,17 @@ module Sources
|
|||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
context "The source site for a https://twitter.com/i/web/status/:id url" do
|
||||||
|
setup do
|
||||||
|
@site = Sources::Site.new("https://twitter.com/i/web/status/943446161586733056")
|
||||||
|
@site.get
|
||||||
|
end
|
||||||
|
|
||||||
|
should "fetch the source data" do
|
||||||
|
assert_equal("https://twitter.com/motty08111213", @site.profile_url)
|
||||||
|
end
|
||||||
|
end
|
||||||
|
|
||||||
context "A tweet" do
|
context "A tweet" do
|
||||||
setup do
|
setup do
|
||||||
@site = Sources::Site.new("https://twitter.com/noizave/status/875768175136317440")
|
@site = Sources::Site.new("https://twitter.com/noizave/status/875768175136317440")
|
||||||
|
|||||||
Reference in New Issue
Block a user