jp: pubsub options refactoring: There is now only "use_pubsub", and specification are set using "pubsub_flags" argument when instantiating CommandBase. Options are more Python Zen compliant by using explicit arguments for item, draft, url instead of trying to guess with magic keyword and type detection. Pubsub node and item are now always using respecively "-n" and "-i" even when required, this way shell history can be used to change command more easily, and it's globally less confusing for user. if --pubsub-url is used, elements can be overwritten with individual option (e.g. change item id with --item). New "use_draft" argument in CommandBase, to re-use current draft or open a file path as draft. Item can now be specified when using a draft. If it already exists, its content will be added to current draft (with a separator), to avoid loosing data. common.BaseEdit.getItemPath could be simplified thanks to those changes. Pubsub URI handling has been moved to base.py.

--- a/frontends/src/jp/cmd_merge_request.py	Wed Mar 21 19:07:06 2018 +0100
+++ b/frontends/src/jp/cmd_merge_request.py	Wed Mar 21 19:13:22 2018 +0100
@@ -20,7 +20,6 @@
 
 import base
 from sat.core.i18n import _
-from sat_frontends.jp import common
 from sat_frontends.jp.constants import Const as C
 from functools import partial
 import os.path
@@ -46,7 +45,6 @@
         self.host.quit(C.EXIT_OK)
 
     def start(self):
-        common.checkURI(self.args)
         repository = os.path.expanduser(os.path.abspath(self.args.repository))
         extra = {'update': 'true'} if self.args.item else {}
         self.host.bridge.mergeRequestSet(