whipper icon indicating copy to clipboard operation
whipper copied to clipboard

Edit tags before rip the CD without needing to use Musicbrainz database.

Open henriqueelder opened this issue 6 years ago • 12 comments

Hello, would be nice if we could Edit the CD and tracks informations manually without needing the Musicbrainz database because sometimes we found wrong filenames and CD titles on Musicbrainz.

henriqueelder avatar Dec 04 '18 14:12 henriqueelder

I would also like this feature. The use cases that I have for it are:

  1. I have a few music CD-Rs that don't belong in MusicBrainz but I would like to add the metadata before ripping.

  2. For some of my CDs the metadata is in MusicBrainz and is correct but I would like to tailor the metadata for my needs. For example shortening lengthy titles or adding a series name to the title so all albums in a series are together when sorted.

  3. I make a minor correction in MusicBrainz but have to wait 7 days for the change to be applied. I would like to make the same change to track list whipper uses so I can immediately rip the Cd instead of waiting a week.

I know it is possible to use other programs to edit metadata afterwards but then the cue sheet titles won't match. If whipper opened the metadata in a text editor for editing it would also be possible to store this text file in the output directory so if I ever need to rerip the CD I could reuse the text file with my modifications.

srussel avatar Apr 25 '19 01:04 srussel

Just created an account here.

This feature would really be cool to have. The use cases are obviously, I can absolutely confirm srussels thoughts on this.

Nord-Licht avatar Dec 17 '19 16:12 Nord-Licht

This patch is a hack I have been using for a few months to work around this issue. It adds an "--edit" option that writes the metadata to a text file, opens it in an editor for the user to update, then parses it back into whipper before continuing with the rip.

whipper-edit-metadata.txt

srussel avatar Jan 09 '21 06:01 srussel

This patch is a hack I have been using for a few months to work around this issue. It adds an "--edit" option that writes the metadata to a text file, opens it in an editor for the user to update, then parses it back into whipper before continuing with the rip.

whipper-edit-metadata.txt

@srussel I can't seem to apply that to the latest version -- would you mind submitting a PR or putting it up on a fork?

Californian avatar May 28 '21 17:05 Californian

@Californian https://github.com/srussel/whipper/commits/edit-metadata

srussel avatar May 30 '21 07:05 srussel

Very cool hack tbh, it would be cool to have it done like this in official whipper.

JuniorJPDJ avatar Jun 23 '21 23:06 JuniorJPDJ

@Californian https://github.com/srussel/whipper/commits/edit-metadata

Yusss thank you!

Californian avatar Jun 24 '21 03:06 Californian

@srussel I tested https://github.com/srussel/whipper/commits/edit-metadata with CD which occurs #453 . I can edit metadata with env EDITOR=pluma whipper cd -d /dev/sr1 rip --edit.But edited title was not used and fails to rip.

oech3 avatar Apr 03 '22 08:04 oech3

@oech3 I have rebased my patch https://github.com/srussel/whipper/tree/edit-metadata It works for me. If it still does not work, try putting some print statements in readMetadata(). That is where the text file is parsed and updates the whipper metadata.

srussel avatar Apr 10 '22 03:04 srussel

@oech3 I found I was only updating the title, not the disambiguated title, which depending on whether you were using %d or %D for the disc title may not have worked. I have updated my patch to support editing both.

srussel avatar Apr 10 '22 05:04 srussel

@srussel Thankyou.I can rip now. I failed to rip due to output_directory/soundtrack/'Various Artists - VERYLONGTITLE' was not editable.

oech3 avatar Apr 10 '22 08:04 oech3

thanks for the work @srussel ! this helped me out a bunch today. hopefully this gets merged w main branch soon enough.

privacy313 avatar Oct 24 '22 11:10 privacy313