Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: ignored-list in match section is… ignored. #1487

Closed
wants to merge 2 commits into from

Conversation

m-urban
Copy link
Contributor

@m-urban m-urban commented Jun 1, 2015

Hi there,

I just realized that the

match:
    ignored: unmatched_tracks

option didn't work for me as described. After looking into the source, I discovered that the list that is used to match ignored parameters, is filled with the Distance object's values instead of its keys. The PR should fix this.

@m-urban m-urban force-pushed the fix-match-ignore branch 2 times, most recently from 5820740 to 5a854a7 Compare June 1, 2015 16:14
@untitaker
Copy link
Collaborator

Thanks!

@untitaker untitaker closed this in 6e8f066 Jun 1, 2015
m-urban added a commit to m-urban/beets that referenced this pull request Jun 1, 2015
untitaker added a commit that referenced this pull request Jun 1, 2015
Updated changelog with information on fix #1487
@sampsyo
Copy link
Member

sampsyo commented Jun 2, 2015

Oops! Nice catch; thanks.

LordSputnik pushed a commit to LordSputnik/beets that referenced this pull request Jul 6, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants