Web   ·   Wiki   ·   Activities   ·   Blog   ·   Lists   ·   Chat   ·   Meeting   ·   Bugs   ·   Git   ·   Translate   ·   Archive   ·   People   ·   Donate
summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDaniel Narvaez <dwnarvaez@gmail.com>2013-04-09 16:06:13 (GMT)
committer Daniel Narvaez <dwnarvaez@gmail.com>2013-04-09 16:13:18 (GMT)
commit7cb4a1d72dddaf729bc1cf4706e5cea319687fa3 (patch)
tree538c0a36b6d25c77a1092b4b58354236ffd8473f
parent7ac2cae7c6a422f3c1deda6e45388a75d08640a5 (diff)
Handle invalid /desktop/sugar/user/color
This is currently breaking the activity list unit test where the profile has not been initialized. But, more in general, it seems like we should not be crashing if our settings has unexpected values.
-rw-r--r--src/sugar3/graphics/xocolor.py2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/sugar3/graphics/xocolor.py b/src/sugar3/graphics/xocolor.py
index c9af6b1..84e93a9 100644
--- a/src/sugar3/graphics/xocolor.py
+++ b/src/sugar3/graphics/xocolor.py
@@ -239,7 +239,7 @@ class XoColor:
'fallback to default', color_string)
client = GConf.Client.get_default()
color_string = client.get_string('/desktop/sugar/user/color')
- randomize = False
+ randomize = is_valid(color_string)
else:
randomize = False