Can't set the WOAR value correctly.

inetmarketeers commented 3 years ago

The problem

So I have tried to set this value as an addition field

        theTrack.AdditionalFields["WOAR"] = textEdit3.Text;

Environment

Visual Studio 19 using ATL 3.16

Details

The value does not get saved at least when I open the file again the WOAR value is null. I

Zeugma440 commented 3 years ago

Hey there !

I've tested it myself and it seems to work fine on my end.

The issue you have might be caused by :

Incorrect writing code (you just sent one single line here)
Incorrect reading code
Using WOAR with ID3v2.2 (shouldn't happen if you're using ATL default settings)
Trying to write to a file format that doesn't support additional fields (e.g. S3M or MIDI)

Could you give a little more details so that I can help you find the issue ? I advise you to start opening your audio file with an editor and check if the field has actually been written.

inetmarketeers commented 3 years ago

Sure, here is my code

        Settings.ID3v2_tagSubVersion = 3;
        Track theTrack = new Track(filename);
        theTrack.AlbumArtist = textEdit2.Text;
        theTrack.AdditionalFields["WOAR"] = textEdit3.Text;
        theTrack.Popularity = 255;
        theTrack.Save();

Popularity is saved correctly. the textEdit3.Text will have for example https://therankingstore.com

However when I read the same file with the component the key value is like this:

"ÿþh\0t\0t\0p\0s\0:\0/\0/\0w\0w\0w\0.\0t\0h\0e\0r\0a\0n\0k\0i\0n\0g\0s\0t\0o\0r\0e\0.\0c\0o\0m\0/"

Zeugma440 commented 3 years ago

Thanks for the details.

I'll try with that and let you know tomorrow.

Zeugma440 commented 3 years ago

There was indeed an issue when writing URL link frames with the ID3v2.3 format.

The fix will be available in the next release.

inetmarketeers commented 3 years ago

Do you have any timeframe for that release?

From: Zeugma440 notifications@github.com Sent: 27 January 2021 07:52 To: Zeugma440/atldotnet atldotnet@noreply.github.com Cc: inetmarketeers chris@inetmarketeers.com; Author author@noreply.github.com Subject: Re: [Zeugma440/atldotnet] Can't set the WOAR value correctly. (#85)

There was indeed an issue when writing URL link frames with the ID3v2.3 format.

The fix will be available in the next release.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Zeugma440/atldotnet/issues/85#issuecomment-768103679 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AB3AAP457QA2DWII733UQODS37AYVANCNFSM4WT6A3VA . https://github.com/notifications/beacon/AB3AAP3RKHNFPNBW7XP7NXDS37AYVA5CNFSM4WT6A3VKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFXEFJ7Y.gif

inetmarketeers commented 3 years ago

Or is there a workaround?

From: Zeugma440 notifications@github.com Sent: 27 January 2021 07:52 To: Zeugma440/atldotnet atldotnet@noreply.github.com Cc: inetmarketeers chris@inetmarketeers.com; Author author@noreply.github.com Subject: Re: [Zeugma440/atldotnet] Can't set the WOAR value correctly. (#85)

There was indeed an issue when writing URL link frames with the ID3v2.3 format.

The fix will be available in the next release.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Zeugma440/atldotnet/issues/85#issuecomment-768103679 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AB3AAP457QA2DWII733UQODS37AYVANCNFSM4WT6A3VA . https://github.com/notifications/beacon/AB3AAP3RKHNFPNBW7XP7NXDS37AYVA5CNFSM4WT6A3VKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFXEFJ7Y.gif

Zeugma440 commented 3 years ago

Timeframe : Today or tomorrow Workaround : Use ID3v2.4 😁 😁

inetmarketeers commented 3 years ago

Fantastic

From: Zeugma440 notifications@github.com Sent: 27 January 2021 11:53 To: Zeugma440/atldotnet atldotnet@noreply.github.com Cc: inetmarketeers chris@inetmarketeers.com; Author author@noreply.github.com Subject: Re: [Zeugma440/atldotnet] Can't set the WOAR value correctly. (#85)

Timeframe : Today or tomorrow Workaround : Use ID3v2.4 😁 😁

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Zeugma440/atldotnet/issues/85#issuecomment-768234528 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AB3AAP2Y2SJHHEV6YUQL4ATS375A5ANCNFSM4WT6A3VA . https://github.com/notifications/beacon/AB3AAP6VIG64XMG2SVIWELLS375A5A5CNFSM4WT6A3VKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOFXFFIIA.gif

Zeugma440 commented 3 years ago

That's done ! Fix is available on today's v3.17.

Enjoy~

Zeugma440 / atldotnet