--- a/sat_frontends/jp/cmd_forums.py	Wed Jun 27 07:51:29 2018 +0200
+++ b/sat_frontends/jp/cmd_forums.py	Wed Jun 27 20:14:46 2018 +0200
@@ -33,23 +33,36 @@
 
 
 class Edit(base.CommandBase, common.BaseEdit):
-    use_items=False
+    use_items = False
 
     def __init__(self, host):
-        base.CommandBase.__init__(self, host, 'edit', use_pubsub=True, use_draft=True, use_verbose=True, help=_(u'edit forums'))
+        base.CommandBase.__init__(
+            self,
+            host,
+            "edit",
+            use_pubsub=True,
+            use_draft=True,
+            use_verbose=True,
+            help=_(u"edit forums"),
+        )
         common.BaseEdit.__init__(self, self.host, FORUMS_TMP_DIR)
-        self.need_loop=True
+        self.need_loop = True
 
     def add_parser_options(self):
-        self.parser.add_argument("-k", "--key", type=base.unicode_decoder, default=u'',
-                                 help=_(u"forum key (DEFAULT: default forums)"))
+        self.parser.add_argument(
+            "-k",
+            "--key",
+            type=base.unicode_decoder,
+            default=u"",
+            help=_(u"forum key (DEFAULT: default forums)"),
+        )
 
     def getTmpSuff(self):
         """return suffix used for content file"""
-        return u'json'
+        return u"json"
 
     def forumsSetCb(self):
-        self.disp(_(u'forums have been edited'), 1)
+        self.disp(_(u"forums have been edited"), 1)
         self.host.quit()
 
     def publish(self, forums_raw):
@@ -60,9 +73,12 @@
             self.args.key,
             self.profile,
             callback=self.forumsSetCb,
-            errback=partial(self.errback,
-                            msg=_(u"can't set forums: {}"),
-                            exit_code=C.EXIT_BRIDGE_ERRBACK))
+            errback=partial(
+                self.errback,
+                msg=_(u"can't set forums: {}"),
+                exit_code=C.EXIT_BRIDGE_ERRBACK,
+            ),
+        )
 
     def forumsGetCb(self, forums_json):
         content_file_obj, content_file_path = self.getTmpFile()
@@ -71,19 +87,21 @@
             # we loads and dumps to have pretty printed json
             forums = json.loads(forums_json)
             # cf. https://stackoverflow.com/a/18337754
-            f = codecs.getwriter('utf-8')(content_file_obj)
+            f = codecs.getwriter("utf-8")(content_file_obj)
             json.dump(forums, f, ensure_ascii=False, indent=4)
             content_file_obj.seek(0)
         self.runEditor("forums_editor_args", content_file_path, content_file_obj)
 
     def forumsGetEb(self, failure_):
         # FIXME: error handling with bridge is broken, need to be properly fixed
-        if failure_.condition == u'item-not-found':
-            self.forumsGetCb(u'')
+        if failure_.condition == u"item-not-found":
+            self.forumsGetCb(u"")
         else:
-            self.errback(failure_,
-                         msg=_(u"can't get forums structure: {}"),
-                         exit_code=C.EXIT_BRIDGE_ERRBACK)
+            self.errback(
+                failure_,
+                msg=_(u"can't get forums structure: {}"),
+                exit_code=C.EXIT_BRIDGE_ERRBACK,
+            )
 
     def start(self):
         self.host.bridge.forumsGet(
@@ -92,55 +110,66 @@
             self.args.key,
             self.profile,
             callback=self.forumsGetCb,
-            errback=self.forumsGetEb)
+            errback=self.forumsGetEb,
+        )
 
 
 class Get(base.CommandBase):
-
     def __init__(self, host):
-        extra_outputs = {'default': self.default_output}
-        base.CommandBase.__init__(self, host, 'get', use_output=C.OUTPUT_COMPLEX, extra_outputs=extra_outputs, use_pubsub=True, use_verbose=True, help=_(u'get forums structure'))
-        self.need_loop=True
+        extra_outputs = {"default": self.default_output}
+        base.CommandBase.__init__(
+            self,
+            host,
+            "get",
+            use_output=C.OUTPUT_COMPLEX,
+            extra_outputs=extra_outputs,
+            use_pubsub=True,
+            use_verbose=True,
+            help=_(u"get forums structure"),
+        )
+        self.need_loop = True
 
     def add_parser_options(self):
-        self.parser.add_argument("-k", "--key", type=base.unicode_decoder, default=u'',
-                                 help=_(u"forum key (DEFAULT: default forums)"))
+        self.parser.add_argument(
+            "-k",
+            "--key",
+            type=base.unicode_decoder,
+            default=u"",
+            help=_(u"forum key (DEFAULT: default forums)"),
+        )
 
     def default_output(self, forums, level=0):
         for forum in forums:
             keys = list(forum.keys())
             keys.sort()
             try:
-                keys.remove(u'title')
+                keys.remove(u"title")
             except ValueError:
                 pass
             else:
-                keys.insert(0, u'title')
+                keys.insert(0, u"title")
             try:
-                keys.remove(u'sub-forums')
+                keys.remove(u"sub-forums")
             except ValueError:
                 pass
             else:
-                keys.append(u'sub-forums')
+                keys.append(u"sub-forums")
 
             for key in keys:
                 value = forum[key]
-                if key == 'sub-forums':
-                    self.default_output(value, level+1)
+                if key == "sub-forums":
+                    self.default_output(value, level + 1)
                 else:
-                    if self.host.verbosity < 1 and key != u'title':
+                    if self.host.verbosity < 1 and key != u"title":
                         continue
                     head_color = C.A_LEVEL_COLORS[level % len(C.A_LEVEL_COLORS)]
-                    self.disp(A.color(level * 4 * u' ',
-                                      head_color,
-                                      key,
-                                      A.RESET,
-                                      u': ',
-                                      value))
+                    self.disp(
+                        A.color(level * 4 * u" ", head_color, key, A.RESET, u": ", value)
+                    )
 
     def forumsGetCb(self, forums_raw):
         if not forums_raw:
-            self.disp(_(u'no schema found'), 1)
+            self.disp(_(u"no schema found"), 1)
             self.host.quit(1)
         forums = json.loads(forums_raw)
         self.output(forums)
@@ -153,14 +182,18 @@
             self.args.key,
             self.profile,
             callback=self.forumsGetCb,
-            errback=partial(self.errback,
-                            msg=_(u"can't get forums: {}"),
-                            exit_code=C.EXIT_BRIDGE_ERRBACK))
-
+            errback=partial(
+                self.errback,
+                msg=_(u"can't get forums: {}"),
+                exit_code=C.EXIT_BRIDGE_ERRBACK,
+            ),
+        )
 
 
 class Forums(base.CommandBase):
     subcommands = (Get, Edit)
 
     def __init__(self, host):
-        super(Forums, self).__init__(host, 'forums', use_profile=False, help=_(u'Forums structure edition'))
+        super(Forums, self).__init__(
+            host, "forums", use_profile=False, help=_(u"Forums structure edition")
+        )