summaryrefslogtreecommitdiffstats
path: root/settings_manager.py
diff options
context:
space:
mode:
authorYi Li <adamliyi@msn.com>2016-11-22 16:34:18 +0800
committerYi Li <adamliyi@msn.com>2016-11-22 16:44:24 +0800
commit5ebab487e36a6b203e49139269eec3e7944c6c12 (patch)
treedd2224695d2c682b051cde75c7db128de65842ed /settings_manager.py
parentf68b86490896ef53db8a61071073da249e01f795 (diff)
downloadphosphor-settingsd-5ebab487e36a6b203e49139269eec3e7944c6c12.tar.gz
phosphor-settingsd-5ebab487e36a6b203e49139269eec3e7944c6c12.zip
Fix incorrect argument for "PropertiesChanged" signal handler
We cannot pass 'fname' as an argument to "PropertiesChanged" signal handler, without changing the signal emitting code. Resolves openbmc/openbmc#824 Change-Id: Ia093914fdba9e3299f9cb409b91e093f74393773 Signed-off-by: Yi Li <adamliyi@msn.com>
Diffstat (limited to 'settings_manager.py')
-rw-r--r--settings_manager.py10
1 files changed, 5 insertions, 5 deletions
diff --git a/settings_manager.py b/settings_manager.py
index 458af5e..f4907f7 100644
--- a/settings_manager.py
+++ b/settings_manager.py
@@ -87,7 +87,7 @@ class HostSettingsObject(DbusProperties):
self.path = path
self.name = name
self.settings = settings
- fname = name[name.rfind("/")+1:] + '-'
+ self.fname = name[name.rfind("/")+1:] + '-'
# Needed to ignore the validation on default networkconfig values as
# opposed to user giving the same.
@@ -108,7 +108,8 @@ class HostSettingsObject(DbusProperties):
if setting['type'] is 'instance_query':
continue
self.set_settings_property(
- setting['name'], setting['type'], setting['default'], fname)
+ setting['name'], setting['type'], setting['default'],
+ self.fname)
# Done with consuming factory settings.
self.adminmode = False
@@ -147,10 +148,9 @@ class HostSettingsObject(DbusProperties):
# Signal handler for when one ore more settings properties were updated.
# This will sync the changes to the BMC.
def settings_signal_handler(
- self, interface_name, changed_properties, invalidated_properties,
- fname):
+ self, interface_name, changed_properties, invalidated_properties):
for name, value in changed_properties.items():
- self.set_system_settings(name, value, fname)
+ self.set_system_settings(name, value, self.fname)
# Placeholder signal. Needed to register the settings interface.
@dbus.service.signal(DBUS_NAME, signature='s')
OpenPOWER on IntegriCloud