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

permissions: Use proper logging (#2089)

Browse source 
Refactored so we can use the proper logging infrastructure instead of `print`.
This commit is contained in:
Adrian Sampson 2016-06-28 10:29:26 -07:00
parent 3a700eb2a4
commit e33c19a3e9
1 changed files with 43 additions and 42 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
beetsplug/permissions.py
View file

@ -48,12 +48,11 @@ class Permissions(BeetsPlugin):
u'dir': '755'
})
self.register_listener('item_imported', permissions)
self.register_listener('album_imported', permissions)
self.register_listener('item_imported', self.fix)
self.register_listener('album_imported', self.fix)
def permissions(lib, item=None, album=None):
"""Running the permission fixer.
def fix(self, lib, item=None, album=None):
"""Fix the permissions for an imported Item or Album.
"""
# Getting the config.
file_perm = config['permissions']['file'].get()
@ -80,9 +79,10 @@ def permissions(lib, item=None, album=None):
# Checks if the destination path has the permissions configured.
if not check_permissions(util.bytestring_path(path), file_perm):
message = u'There was a problem setting permission on {}'.format(
path)
print(message)
self._log.warn(
u'There was a problem setting permissions on file {}',
path,
)
# Adding directories to the directory chmod queue.
dir_chmod_queue.update(
@ -96,6 +96,7 @@ def permissions(lib, item=None, album=None):
# Checks if the destination path has the permissions configured.
if not check_permissions(util.bytestring_path(path), dir_perm):
message = u'There was a problem setting permission on {}'.format(
path)
print(message)
self._log.warn(
u'There was a problem setting permissions on directory {}',
path,
)