Backup_Repos/beets
1
0
Fork 0
mirror of https://github.com/beetbox/beets.git synced 2025-12-30 12:32:33 +01:00
Code Issues Projects Releases Wiki Activity

Use item[field] instead of getattr(item, field)

Browse source
This commit is contained in:
Thomas Scholtes 2014-09-14 15:47:20 +02:00
parent 19d3ca822e
commit 492cf38927
2 changed files with 7 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
beets/autotag/match.py
View file

@ -65,10 +65,10 @@ def current_metadata(items):
fields = ['artist', 'album', 'albumartist', 'year', 'disctotal',
'mb_albumid', 'label', 'catalognum', 'country', 'media',
'albumdisambig']
for key in fields:
values = [getattr(item, key) for item in items if item]
likelies[key], freq = plurality(values)
consensus[key] = (freq == len(values))
for field in fields:
values = [item[field] for item in items if item]
likelies[field], freq = plurality(values)
consensus[field] = (freq == len(values))
# If there's an album artist consensus, use this for the artist.
if consensus['albumartist'] and likelies['albumartist']:

7
beetsplug/zero.py
View file

@ -85,12 +85,11 @@ class ZeroPlugin(BeetsPlugin):
return
for field, patterns in self.patterns.items():
try:
value = getattr(item, field)
except AttributeError:
if field not in item.keys():
log.error(u'[zero] no such field: {0}'.format(field))
continue
value = item[field]
if self.match_patterns(value, patterns):
log.debug(u'[zero] {0}: {1} -> None'.format(field, value))
setattr(item, field, None)
item[field] = None