address comment raised in #109, now uses separate format strings for on/off, rather than status strings

1 files changed, 6 insertions, 17 deletions

diff --git a/py3status/modules/dpms.py b/py3status/modules/dpms.py
index 262c82b..f939111 100644
--- a/py3status/modules/dpms.py
+++ b/py3status/modules/dpms.py
@@ -7,12 +7,8 @@ of DPMS (Display Power Management Signaling)
 by clicking on 'DPMS' in the status bar.
 
 Configuration parameters:
-    - format: output format string
-    - status_off: string to display when DPMS is enabled
-    - status_on: string to display when DPMS is disabled
-
-Format of status string placeholders:
-    {status} - DPMS status (defined by status_on/off parameters)
+    - format_off: string to display when DPMS is enabled
+    - format_on: string to display when DPMS is disabled
 
 @author Andre Doser <dosera AT tf.uni-freiburg.de>
 """
@@ -24,9 +20,8 @@ class Py3status:
     """
     """
     # available configuration parameters
-    format = "DPMS"
-    status_off = "Off"
-    status_on = "On"
+    format_off = "DPMS"
+    format_on = "DPMS"
 
     def dpms(self, i3s_output_list, i3s_config):
         """
@@ -36,16 +31,10 @@ class Py3status:
         self.run = system('xset -q | grep -iq "DPMS is enabled"') == 0
 
         response = {
-            'full_text': self.format.format(
-                status = self.status_on if self.run else self.status_off
-            )
+            'full_text': self.format_on if self.run else self.format_off,
+            'color': i3s_config['color_good'] if self.run else i3s_config['color_bad']
         }
 
-        if self.run:
-            response['color'] = i3s_config['color_good']
-        else:
-            response['color'] = i3s_config['color_bad']
-
         return response
 
     def on_click(self, i3s_output_list, i3s_config, event):