beetbox / beetbox/beets

merging albums with --set fields overwrites whole album

Open
#5,077 3 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
15.7k
Forks
2.1k
Avg merge
4d 21h
Merged PRs (30d)
31

Description

### Problem

when importing a track with:
- for instance --set=title=whatever
- then making so we hit the "this album is already in the library" dialog and choosing merge

the --set=title=whatever flag ends up affecting all existing tracks in the album as well.

fairly long, self-contained MWE:

```bash
#!/usr/bin/env bash
set -eux
basedir=$(mktemp -d)

# create a standalone config
cat > $basedir/conf.yaml < $basedir/src.wav

# set the "file editor" to a script that sets the "album" and "albumartist" fields
export EDITOR="sed -E -i 's/album:.*\$/album: album/;s/albumartist:.*\$/albumartist: artist/'"

# import 'full album'
# eDit, Accept
beet -c $basedir/conf.yaml imp --set=title=title1 $basedir/src.wav <

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.