Adding a lyric source

Post a reply

Smilies
:D :) :( :o :-? 8) :lol: :x :P :oops: :cry: :evil: :roll: :wink:

BBCode is ON
[img] is ON
[url] is ON
Smilies are ON

Topic review
   

Expand view Topic review: Adding a lyric source

Re: Adding a lyric source

by Ludek » Wed Sep 29, 2021 12:43 pm

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)

Re: Adding a lyric source

by Zonky » Wed Sep 29, 2021 6:12 am

So, I still want to debug this and still no solution for the Chrome debugger not stopping at the point of interest...

Anyone has got any hint for me?

Re: Adding a lyric source

by Zonky » Sat Sep 18, 2021 7:29 am

So, still no success with the above given track and Genius lyrics search :(
Still getting some html code mixed with the lyrics itselft.

I tried to debug this after noticing I need Chrome installed (running exclusively Firefox) to get the debug running :D
Nevertheless I added a breakpoint in source helpers\lyricsSearch.js at line ~986

Code: Select all

rGenius.host = 'https://genius.com/%artist%-%title%-lyrics';
After opening the tracks' properties I switched to Lyrics tab and executed the lyrics search. I assumed it stops at the line I previously added a breakpoint - but it didn't.

So any hint available how I could stop at the time lyrics search is running or better when Genius search gets called? FYI it does not stop either when setting up a breakpoint at any line in lyricsSearch.js.

Of course Genius as a source for lyrics lookup is active in settings.

Re: Adding a lyric source

by Ludek » Fri Sep 10, 2021 2:40 pm

Re: Adding a lyric source

by Zonky » Thu Sep 09, 2021 12:27 pm

Peke wrote: Tue Sep 07, 2021 8:08 pm Hi,
Are you using 5.0.2.2500 beta as in that version all HTML tags should be stripped.
Yes, I do!

Can I debug this at the time the lyrics get fetched? As mentioned above, I'm running the beta, debug version, so I assume the Chrome console should do the job, right.

Did not try it myself because of lack of some information how to debug inside MM.

Re: Adding a lyric source

by Peke » Tue Sep 07, 2021 8:08 pm

Hi,
Are you using 5.0.2.2500 beta as in that version all HTML tags should be stripped.

Re: Adding a lyric source

by Zonky » Tue Sep 07, 2021 12:57 pm

drakinite wrote: Fri Sep 03, 2021 4:47 pm Approved! :slight_smile:
Guess I've found a bug when seeking for lyrics for the below mp3 file.
Genius returns in this case:

Code: Select all

[Verse 1]
<a href="/4794116/London-grammar-nightcall/Im-giving-you-a-nightcall-to-tell-you-how-i-feel-im-gonna-drive-you-through-the-night-down-the-hills-im-gonna-tell-you-something-you-dont-want-to-hear-im-gonna-show-you-where-its-dumped-but-have-no-fear" class="ReferentFragment__ClickTarget-oqvzi6-0 evuxZm
MP3 properties:

Code: Select all

Title: "Nightcall (Raaja Bones & Fyfe Dangerfield Remix)"
Artists: "London Grammar"
Album: "Nightcall"
Album Artists: "London Grammar"
Do you need more details for investigation if all?

Re: Adding a lyric source

by IanRTaylorUK » Sun Sep 05, 2021 3:42 am

Thanks Zonky,

So I am better to use Lyricator with MMW4 for block updating with it 11(?) websites than looking up individual files with 7 sources.

Re: Adding a lyric source

by Zonky » Sun Sep 05, 2021 2:23 am

IanRTaylorUK wrote: Sat Sep 04, 2021 3:53 pm However, one thing I am not certain about. Is this add-on intended to be used on a single selection or cannot it be used to multi-select?
I've recently asked this question and was advised that this is not the case and there is no intention to do this since this might violate the websites policies for mass grabbing lyrics which will probably lead to a lockout of MM.

MM4 did not care about this, so I guess it's just a matter of time until MM5 will give us a bulk update of lyrics again. Otherwise the current lyrics feature is not very convenient...

Re: Adding a lyric source

by IanRTaylorUK » Sat Sep 04, 2021 3:53 pm

Looks like a really useful add-on.

I have previously used Lyricator in MMW4 and so only have a few MP3 left without lyrics.

I tried the add-on but not finding lyrics - probably because there aren't any.

However, one thing I am not certain about. Is this add-on intended to be used on a single selection or cannot it be used to multi-select?

Re: Adding a lyric source

by jess12345 » Sat Sep 04, 2021 8:17 am

Been testing this add-on Tonight... Finding alot of lyrics I couldn't get before so thanks mate!! But it seems to not be grabbing all the lyrics, just part of them... But good job for your first add-on bro. I can't do anything like that so just wanted to say good job.

Re: Adding a lyric source

by drakinite » Fri Sep 03, 2021 4:47 pm

Approved! :slight_smile:

Re: Adding a lyric source

by fizzjob » Fri Sep 03, 2021 3:37 pm

TIV73 wrote: Fri Sep 03, 2021 2:03 pm Congratulations on your first addon!
Well, not my first, but my first for MM5. ;)
drakinite wrote: Fri Sep 03, 2021 3:04 pm Congrats; great work! However, before I can approve it: You forgot to fix that one error I pointed out earlier. The addon breaks when lyrics cannot be found, because you need to call requestNext() on line 16. :slight_smile:

On the Submit Addon page, you should be able to edit the submitted version by clicking on Version Information on the right, then selecting a new file like so: http://files.drakinite.net/2021-09-03_15-58-51.mp4
Fixed and submitted, thanks for the reminder.

Re: Adding a lyric source

by drakinite » Fri Sep 03, 2021 3:04 pm

fizzjob wrote: Fri Sep 03, 2021 1:44 pm edit: ...and it's submitted! First time ever doing anything with node.js and npm. I'm learning all around today! :D
Congrats; great work! However, before I can approve it: You forgot to fix that one error I pointed out earlier. The addon breaks when lyrics cannot be found, because you need to call requestNext() on line 16. :slight_smile:

On the Submit Addon page, you should be able to edit the submitted version by clicking on Version Information on the right, then selecting a new file like so: http://files.drakinite.net/2021-09-03_15-58-51.mp4

Re: Adding a lyric source

by TIV73 » Fri Sep 03, 2021 2:03 pm

Congratulations on your first addon!

Top