search

LMS-Community / plugin-Qobuz

Squeebox plugin to stream from qobuz.com
31 stars 16 forks source link

Logging category added two times #13

Closed ljungberg closed 7 years ago

ljungberg commented 7 years ago

plugin.qobuz is added two times to the Log

(once in Plugin.pm with level 'ERROR' and once in Settings.pm with level 'INFO')

When looking at web GUI 'Settings'->'Advanced' choosing 'Logging' in drop down menu there is one entry for plugin.qobuz with level 'INFO'. I think the addLoggCategory call in Settings.pm overwrites the first one.

My guess is that you would want something like:

my $log = logger('plugin.qobuz');

in Settings.pm

mherger commented 7 years ago

Fixed in commit 4542094f765ee6d8cf213e4f00662c0740953bd1. Thanks!