From 447864774e4b820542a3c79f1727292d4debbea3 Mon Sep 17 00:00:00 2001 From: Stein Magnus Jodal Date: Wed, 9 Oct 2013 23:51:01 +0200 Subject: [PATCH] core: Add volume arg to volume_changed() event It was already called with the argument, and both the MPD and HTTP frontends handled it/expected it. It was just the default implementation in CoreListener that lacked the argument. --- mopidy/core/listener.py | 5 ++++- tests/core/listener_test.py | 2 +- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/mopidy/core/listener.py b/mopidy/core/listener.py index c93fc39e..5afb3f4f 100644 --- a/mopidy/core/listener.py +++ b/mopidy/core/listener.py @@ -132,11 +132,14 @@ class CoreListener(object): """ pass - def volume_changed(self): + def volume_changed(self, volume): """ Called whenever the volume is changed. *MAY* be implemented by actor. + + :param volume: the new volume in the range [0..100] + :type volume: int """ pass diff --git a/tests/core/listener_test.py b/tests/core/listener_test.py index bf3a235d..d1773a12 100644 --- a/tests/core/listener_test.py +++ b/tests/core/listener_test.py @@ -49,7 +49,7 @@ class CoreListenerTest(unittest.TestCase): self.listener.options_changed() def test_listener_has_default_impl_for_volume_changed(self): - self.listener.volume_changed() + self.listener.volume_changed(70) def test_listener_has_default_impl_for_seeked(self): self.listener.seeked(0)