From c5e9c66d00e481493bbc3f63f98e57ac68962ce0 Mon Sep 17 00:00:00 2001 From: Felix Domke Date: Wed, 11 Mar 2009 08:06:04 +0100 Subject: [PATCH] Patch by Moritz Venn: The current implementation of ConfigNumber.isChanged (the one inherited from ConfigElement) does not work properly as it will - at least for unsaved values - always returns True. This is obvious if you just take a look at the datatypes since value is an int and default is a string (because ConfigNumber is a modified ConfigText). To resolve this issue one can either compare self.tostring(self.value) or self.text to self.default but the former seems to be the more logical approach. The attached patch does override the method in ConfigNumber with the proposed fix. --- lib/python/Components/config.py | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/lib/python/Components/config.py b/lib/python/Components/config.py index 4cc40633..24d39cba 100755 --- a/lib/python/Components/config.py +++ b/lib/python/Components/config.py @@ -1029,6 +1029,13 @@ class ConfigNumber(ConfigText): value = property(getValue, setValue) _value = property(getValue, setValue) + def isChanged(self): + sv = self.saved_value + strv = self.tostring(self.value) + if sv is None and strv == self.default: + return False + return strv != sv + def conform(self): pos = len(self.text) - self.marked_pos self.text = self.text.lstrip("0") -- 2.30.2