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)
Did you saw my reply here: https://www.mediamonkey.com/forum/viewtopic.php?p=486799#p486799
and my breakpoint in NotifyWebReq funct: https://www.dropbox.com/s/yv60573my7heubm/Screenshot%202021-09-10%2021.31.10.png?dl=0
Are you saying that opening DevTools and putting breakpoint e.g. to line
[code] downloadPromise = app.utils.web.getURLContentAsync(str, {[/code]
does not stop there for you ?
Or alternatively later at line:
[code] if ((Content) && (Content != ''))[/code]
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: https://www.mediamonkey.com/forum/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)