From 1171a1d0ae778cd5746ae73e62da116e0dbcac73 Mon Sep 17 00:00:00 2001 From: enkore Date: Sat, 16 Mar 2013 12:58:01 +0100 Subject: [PATCH] Change Module.test API to something more intuitive (I mean, what the hell has crossed my mind to name a method *test* and let it return FALSE if successful? :D) --- i3pystatus/core/modules.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/i3pystatus/core/modules.py b/i3pystatus/core/modules.py index 802ed64..b94535d 100644 --- a/i3pystatus/core/modules.py +++ b/i3pystatus/core/modules.py @@ -25,12 +25,10 @@ class Module(SettingsBase): def test(self): """ - Return None* if settings seem ok (i.e. login credentials and such are valid) + Return True if settings seem ok (i.e. login credentials and such are valid) or a string describing the problem - - * or some other value evaluating to false in a boolean context """ - return + return True def on_click(self, button): if button == 1: # Left mouse button