Adding a lyric source

To discuss development of addons / skins / customization of MediaMonkey.

Moderators: jiri, drakinite, Addon Administrators

Ludek
Posts: 4958
Joined: Fri Mar 09, 2007 9:00 am

Re: Adding a lyric source

Post by Ludek »

Did you saw my reply here: viewtopic.php?p=486799#p486799
and my breakpoint in NotifyWebReq funct: https://www.dropbox.com/s/yv60573my7heu ... 0.png?dl=0


Are you saying that opening DevTools and putting breakpoint e.g. to line

Code: Select all

 downloadPromise = app.utils.web.getURLContentAsync(str, {
does not stop there for you ?

Or alternatively later at line:

Code: Select all

 if ((Content) && (Content != ''))
But maybe better for you is to put breakpoint directly in cleanupLyrics() funct that does clean the lyrics to be without HTML elements/tags.
Supposing that you are using cleanupLyrics in rGenius.onSuccess ?
Alernatively put breakpoint at the beginning of rGenius.onSuccess

EDIT: When I was reviewing the Genius sources here: viewtopic.php?p=486709#p486709
I found that NotifyWebReq and LoadWebPageFailed functions redundant. They are already in the original lyricsSearch.js, not sure why the script author added it there, but I think they should be removed from the script (not overriding the native variants)
Post Reply