[RELEASE] xbmclyrics script - automatically grabs lyrics online

(smuto @ aug. 10 2005,13:40 Wrote:xbmclyrics with cache folder q:\albums\lyrics

http://chokemaniac.xbmc.org/viewtopic.php?t=316

with little skin mod

is the same release above?

from readme

Quote:create a cache directory inside the folder where the script is located. just name it "cache" (lowercase, no quotes). [colgate]

this one from chokemaniac's forum creates lyrics directory inside the albums folder

that only different

this script no longer appears to be wrapping text with the latest cvs? or is it just me?

if a line is too long, it repeats upon itself on the same line now instead of wrapping down to the next line.

after a little more research, it seems as though the lines that are getting superimposed are not necessarily too long/wide to fit on screen.

maybe the font needs to be made smaller or more lines made to fit inside the window?

i've e-mailed the author (or at least the address included in the readme.txt). i'll post back if i get a reply.

i have the same problem with latest cvs build.

some lines are displayed over themselves and cannot be read correctly.

would love to see a fix for this.

whew..it's nice to have confirmation at least. so it's not just me

in the meantime if i get adventurous, i'll try editing the script myself.

my wife enjoys this feature a lot and she is getting ticked that 'i just broke it'.

yeah, glad to see it's not just me also. luck to you in fixing this.

here is the reply i got:

Quote:hi,

thanks for the report, but unfortunately i have no idea why that
happens. since the script works on previous xbmc versions i don't
think it's my fault.

enderw

perhaps it has something to do with the skin changes?
i think jmarshall changed some things with the line wrapping?

for a quick and dirty fix. replace the following. there isn't any big change. you might notice some of the paragraphs are split. you could play with this some more to fix that.

this:

Quote: def parse_lyrics(self, lyrics):
try:
query = re.compile('</font><br>(.*)<p><hr size=1', re.ignorecase | re.dotall)
full_lyrics = re.findall(query, lyrics)
self.final = full_lyrics[0].replace("<br />\r\n","\n ")
except:
try:
query = re.compile('</font><br>(.*)<br><br><a href="#', re.ignorecase | re.dotall)
full_lyrics = re.findall(query, lyrics)
self.final = full_lyrics[0].replace("<br />\r\n","\n ")
except:
print lyrics
self.final = "unusual page layout, no parsing built in yet..."
return str(self.final)

to:

Quote: def parse_lyrics(self, lyrics):
try:
query = re.compile('</font><br>(.*)<p><hr size=1', re.ignorecase | re.dotall)
full_lyrics = re.findall(query, lyrics)
self.final = full_lyrics[0].replace("<br />\r","\n ")
except:
try:
query = re.compile('</font><br>(.*)<br><br><a href="#', re.ignorecase | re.dotall)
full_lyrics = re.findall(query, lyrics)
self.final = full_lyrics[0].replace("<br />\r","\n ")
except:
print lyrics
self.final = "unusual page layout, no parsing built in yet..."
return str(self.final)