Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Failure to find artist for certain Twitter URLs #3422

Closed
evazion opened this issue Dec 9, 2017 · 3 comments
Closed

Failure to find artist for certain Twitter URLs #3422

evazion opened this issue Dec 9, 2017 · 3 comments
Labels

Comments

@evazion
Copy link
Member

evazion commented Dec 9, 2017

URL: https://twitter.com/gulp5959/status/929679299002621955

Artist entry: https://danbooru.donmai.us/artists.json?name=gulp5959
Artist URL: https://twitter.com/Gulp5959
Normalized artist URL: http://twitter.com/Gulp5959/

Using the bookmarklet on the tweet above doesn't find the artist. It doesn't match because the normalized URL in the artist entry isn't lowercase.

This happened because inside ArtistUrl#normalize, we do Sources::Site.new("https://twitter.com/Gulp5959").normalize_for_artist_finder! to normalize the URL, which would lowercase it, if it worked. Except it doesn't, because the Twitter source strategy doesn't recognize URLs that don't contain a status ID.

@nonamethanks
Copy link
Member

@evazion it seems this wasn't fully fixed. It is only matching correctly for artists whose link lists have been refreshed - ones with mixed case twitter links still won't match until their links are resaved.

Example: any of this user's tweets https://twitter.com/GiriGiriii/media don't match his existing artist entry (https://danbooru.donmai.us/artists/160859)

Even though this artist was created recently (in the last ~12 hours), it still won't match it. I'm not editing it because otherwise it'd stop being a useful example as the link refreshing can only be reproduced once per artist, so I'll leave that to you so you can observe the bug.

@nonamethanks
Copy link
Member

Ah sorry, pinging @r888888888 too because I just noticed it was him who committed the fix for this.

@r888888888
Copy link
Collaborator

I've reverted and deployed this for now since it was breaking the artist finder.

r888888888 added a commit that referenced this issue Dec 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants