puniko/akkoma
1
1
Fork 0
forked from AkkomaGang/akkoma
Code Issues Pull requests Projects Releases Packages Wiki Activity

title parse improvement

Browse source
This commit is contained in:
Alexander Strizhakov 2020-01-29 11:13:34 +03:00
parent f1d5c0f079
commit 1f4fbe9d98
No known key found for this signature in database
GPG key ID: 022896A53AEF1381
3 changed files with 244 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/pleroma/web/rich_media/parsers/meta_tags_parser.ex
View file

@ -48,6 +48,6 @@ defp maybe_put_title(meta, html) when meta != %{} do
defp maybe_put_title(meta, _), do: meta
defp get_page_title(html) do
Floki.find(html, "title") |> List.first() |> Floki.text()
Floki.find(html, "html head title") |> List.first() |> Floki.text()
end
end

228
test/fixtures/nypd-facial-recognition-children-teenagers4.html vendored Normal file
View file

File diff suppressed because one or more lines are too long

15
test/web/rich_media/parsers/twitter_card_test.exs
View file

@ -85,4 +85,19 @@ test "respect only first title tag on the page" do
image: image_path
}}
end
test "takes first founded title in html head if there is html markup error" do
html = File.read!("test/fixtures/nypd-facial-recognition-children-teenagers4.html")
assert TwitterCard.parse(html, %{}) ==
{:ok,
%{
site: nil,
title:
"She Was Arrested at 14. Then Her Photo Went to a Facial Recognition Database. - The New York Times",
"app:id:googleplay": "com.nytimes.android",
"app:name:googleplay": "NYTimes",
"app:url:googleplay": "nytimes://reader/id/100000006583622"
}}
end
end