add support for lololyrics.com

Bug #614917 reported by Anton Petersson
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
CoverGloobus
Fix Committed
Low
Alexandr Grigorcea

Bug Description

http://www.lololyrics.com has a very good coverage of techno lyrics, which covergloobus currently is lacking

direct link to search (since it normally uses ajax):
http://www.lololyrics.com/includes/getsearch.php?q=searchtext

Revision history for this message
Fredrik Kilegran (atomicoz) wrote :

FYI: we have a simple API I quickly hacked up for another plugin prior to going on vacation It returns both the lyric and cover.

More info can be found at http://api.lololyrics.com

Revision history for this message
Alexandr Grigorcea (cahr-gr) wrote :

I have some issues with their API, python xml parser are unable to parse entities like ß

Revision history for this message
Fredrik Kilegran (atomicoz) wrote :

Alexandr: If you still have the code, try just adding 'rawutf8=1' to the request, and you'll get it in raw utf-8, no html entities or html linebreaks.

http://api.lololyrics.com/old/getLyric?rawutf8=1&artist=Laserkraft+3D&track=Nein+Mann

(Sorry didn't see your reply until now, feel free to contact me through launchpad if you need anything else, also subscribed to this issue)

Revision history for this message
Alexandr Grigorcea (cahr-gr) wrote :

I've pushed it to trunk, should work, but will be nice if someone will test it a little bit more cause I don't have this kind of music

Changed in covergloobus:
assignee: nobody → Alexandr Grigorcea (cahr-gr)
importance: Undecided → Low
status: New → Fix Committed
Revision history for this message
Fredrik Kilegran (atomicoz) wrote :

Nice job, just grabbed the trunk version and seems to work fine with songs with foreign characters, etc :)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.