external_script and static_string modules code QA

1 files changed, 19 insertions, 19 deletions

diff --git a/py3status/modules/external_script.py b/py3status/modules/external_script.py
index c2a6977..4b3f2e8 100644
--- a/py3status/modules/external_script.py
+++ b/py3status/modules/external_script.py
@@ -1,17 +1,16 @@
 # -*- coding: utf-8 -*-
 """
-Display output of given script
+Display output of given script.
 
-Display output of any executable script set by script_path
-Pay attention. The output must be one liner, or will break your i3status
-The script should not have any parameters, but it could work
+Display output of any executable script set by 'script_path'.
+Pay attention. The output must be one liner, or will break your i3status !
+The script should not have any parameters, but it could work.
 
 Configuration parameters:
     - cache_timeout : how often we refresh this module in seconds
-    - script_path   : script you want to show output of (compulsory)
     - color         : color of printed text
-    - on_click      : read goo.gl/u10n0x
     - format        : see placeholders below
+    - script_path   : script you want to show output of (compulsory)
 
 Format of status string placeholders:
     {output} - output of script given by "script_path"
@@ -19,9 +18,9 @@ Format of status string placeholders:
 i3status.conf example:
 
 external_script {
-    script_path = "/usr/bin/whoami"
-    format = "my name is {output}"
     color = "#00FF00"
+    format = "my name is {output}"
+    script_path = "/usr/bin/whoami"
 
 @author frimdo ztracenastopa@centrum.cz
 """
@@ -31,31 +30,32 @@ from time import time
 
 
 class Py3status:
+    """
+    """
+    # available configuration parameters
     cache_timeout = 15
-    format = '{output}'
     color = None
+    format = '{output}'
     script_path = None
 
     def external_script(self, i3s_output_list, i3s_config):
-
         if self.script_path:
-
-            return_value = subprocess.check_output(self.script_path, shell=True, universal_newlines=True)
+            return_value = subprocess.check_output(self.script_path,
+                                                   shell=True,
+                                                   universal_newlines=True)
             response = {
                 'cached_until': time() + self.cache_timeout,
-                'full_text': self.format.format(
-                    output=return_value.rstrip()),
-                'color': self.color
-                }
-
+                'color': self.color,
+                'full_text': self.format.format(output=return_value.rstrip())
+            }
         else:
             response = {
                 'cached_until': time() + self.cache_timeout,
-                'full_text': ""
+                'full_text': ''
             }
-
         return response
 
+
 if __name__ == "__main__":
     from time import sleep
     x = Py3status()