[RELEASE] CU Lyrics (Script) Addon

woodyearl Wrote:Figures I do something wrong.
Thanks!

Actually, as I take a closer look, they are listed as Capital. AC/DC.

woodyearl Wrote:Actually, as I take a closer look, they are listed as Capital. AC/DC.

the script doesnt see it as capitals, strange

can you upload that file somewhere for me to download?

EDIT: now that I looked closer, the script does see it as capitals but that changes somewhere along the line. I would appreciate the file so that I can test.

amet, the changing of case is because lyricwiki most time requires it be camel case.

thats why there is an excepetion option, so when it finds AC/DC it won't change it.

this was in the original scraper, looks like it's still there in yours

just noticed using the old drive letters P:\\, so change those to special://

Amet Wrote:the script doesnt see it as capitals, strange

can you upload that file somewhere for me to download?

EDIT: now that I looked closer, the script does see it as capitals but that changes somewhere along the line. I would appreciate the file so that I can test.

http://www.mediafire.com/?k5djatz3eow

Nice release Amet. Most files work but I had some issues that required me to copy the 'language' folder from XBMC into the resources folder of CU Lyrics to make it work. I also get messages like below with every search:

DEBUG: CGUIMediaWindow::GetDirectory (?)
DEBUG: ParentPath = []
ERROR: DIRECTORY::CDirectory::GetDirectory - Error getting ?
ERROR: CGUIMediaWindow::GetDirectory(?) failed

I also have run into capitalization issue. ZZ Top won't return any results because it changes the 2nd 'Z' to lowercase. The paste below is what CU Lyrics sends:

http://lyricwiki.org/index.php?title=Zz%...ction=edit

If you modify the above post to 'ZZ' in a browser, it works. Too bad Lyricwiki is so case-sensitive.

Those issues aside this script looks quite nice. Great work!

Thanks for the hard work,

I'm having a different problem.
Using confluance, I cannot escape out of the script
I am able to navigate the up/down arrows but cannot navigate to the 'X' button
The standard back and home buttons are ineffective

If I start the script in the 'scripts' window I am only able to change to fullscreen vis but not change back
If I start the script in fullscreen vis I cannot escape anywhere

Much appreciated, my most used script forsure

*edit
on song change the script does not attempt to fetch the new song.

woodyearl Wrote:http://www.mediafire.com/?k5djatz3eow

paco Wrote:Nice release Amet. Most files work but I had some issues that required me to copy the 'language' folder from XBMC into the resources folder of CU Lyrics to make it work. I also get messages like below with every search:

DEBUG: CGUIMediaWindow::GetDirectory (?)
DEBUG: ParentPath = []
ERROR: DIRECTORY::CDirectory::GetDirectory - Error getting ?
ERROR: CGUIMediaWindow::GetDirectory(?) failed

I also have run into capitalization issue. ZZ Top won't return any results because it changes the 2nd 'Z' to lowercase. The paste below is what CU Lyrics sends:

http://lyricwiki.org/index.php?title=Zz%...ction=edit

If you modify the above post to 'ZZ' in a browser, it works. Too bad Lyricwiki is so case-sensitive.

Those issues aside this script looks quite nice. Great work!

Updated first post with version 0.6.0, it should take care of capitalization issue

Let me know if it works for you.
Thanks for testing
Zeljko

Loto_Bak Wrote:Thanks for the hard work,

I'm having a different problem.
Using confluance, I cannot escape out of the script
I am able to navigate the up/down arrows but cannot navigate to the 'X' button
The standard back and home buttons are ineffective

If I start the script in the 'scripts' window I am only able to change to fullscreen vis but not change back
If I start the script in fullscreen vis I cannot escape anywhere

Much appreciated, my most used script forsure

I only have Win to test here, "esc" button works to stop the script. I will test on ATV when I get home

Loto_Bak Wrote:*edit
on song change the script does not attempt to fetch the new song.

I will test that as well, thanks for letting me know
Zeljko

Nuka1195 Wrote:amet, the changing of case is because lyricwiki most time requires it be camel case.

thats why there is an excepetion option, so when it finds AC/DC it won't change it.

this was in the original scraper, looks like it's still there in yours

Nuka1195 Wrote:just noticed using the old drive letters P:\\, so change those to special://

Thanks for the pointers, I have updated the scraper to hopefully solve the issue.

Zeljko