Renamed '--varsfilter' to '--filter' and added '--xml'

pym/cl_client_cmd.py

@@ -91,7 +91,7 @@ class client_cmd(share_cmd):
         optionsNotDomain = self.getOptionsNotDomain(optObj)
         if not args:
             options = optionsNotDomain + [optObj.color, optObj.v,
-                                          optObj.varsfilter]
+                                          optObj.filter, optObj.xml]
             if not filter(lambda x: x, options):
                 errMsg = _("no such argument")+":"+" %s" %USAGE.split(" ")[-1]
                 self.optobj.error(errMsg)
@@ -109,6 +109,17 @@ class client_cmd(share_cmd):
             errMsg = _("incorrect argument") + ":" + " %s" %" ".join(args)
             self.optobj.error(errMsg)
             return False
+        if not optObj.v:
+            if optObj.filter:
+                errMsg = _("incorrect option") + ":" + " %s" %"--filter" +\
+                ": " + _("use with option '-v'")
+                self.optobj.error(errMsg)
+                return False
+            if optObj.xml:
+                errMsg = _("incorrect option") + ":" + " %s" %"--xml" +\
+                ": " + _("use with option '-v'")
+                self.optobj.error(errMsg)
+                return False
         return optObj, args
 
     def addDomain(self, domainName):

pym/cl_passwd_cmd.py

@@ -58,6 +58,17 @@ class passwd_cmd(share_cmd):
             errMsg = _("invalid argument") + ":" + " %s" %" ".join(args)
             self.optobj.error(errMsg)
             return False
+        if not optObj.v:
+            if optObj.filter:
+                errMsg = _("incorrect option") + ":" + " %s" %"--filter" +\
+                ": " + _("use with option '-v'")
+                self.optobj.error(errMsg)
+                return False
+            if optObj.xml:
+                errMsg = _("incorrect option") + ":" + " %s" %"--xml" +\
+                ": " + _("use with option '-v'")
+                self.optobj.error(errMsg)
+                return False
         return optObj, args
 
     def setUserPasswordToServer(self):

pym/cl_share_cmd.py

@@ -29,20 +29,21 @@ class share_cmd(color_print, _error):
     def printVars(self, optObj):
         """Печать переменных"""
         if optObj.v:
-            self.logicObj.printVars()
-        # Печать фильтрованных переменных
-        elif optObj.varsfilter:
-            optCmd = optObj.varsfilter
-            varsFilter=None
+            varsFilter = None
             varsNames = []
-            if ',' in optCmd:
-                varsNames = optCmd.split(",")
-            elif '*' in optCmd:
-                varsFilter = optCmd.replace("*", ".*")
-            else:
-                varsNames.append(optCmd)
-            self.logicObj.printVars(varsFilter, varsNames)
-        return True
+            format = "default"
+            # Фильтрование переменных
+            if optObj.filter:
+                optCmd = optObj.filter
+                if ',' in optCmd:
+                    varsNames = optCmd.split(",")
+                elif '*' in optCmd:
+                    varsFilter = optCmd.replace("*", ".*")
+                else:
+                    varsNames.append(optCmd)
+            if optObj.xml:
+                format = "xml"
+            self.logicObj.printVars(varsFilter, varsNames, outFormat=format)
 
     def setVars(self, optObj):
         """Установка переменных"""

pym/cl_sync_cmd.py

@@ -92,7 +92,7 @@ class sync_cmd(share_cmd):
         """Проверка опций командной строки"""
         optionsRequired = self.getOptionsRequired(optObj)
         if not args:
-            options = [optObj.color, optObj.v, optObj.varsfilter]
+            options = [optObj.color, optObj.v, optObj.filter, optObj.xml]
             if not filter(lambda x: x, options):
                 errMsg = _("no such argument")+":"+" %s" %USAGE.split(" ")[-1]
                 self.optobj.error(errMsg)
@@ -113,6 +113,17 @@ class sync_cmd(share_cmd):
             errMsg = _("incorrect argument") + ":" + " %s" %" ".join(args)
             self.optobj.error(errMsg)
             return False
+        if not optObj.v:
+            if optObj.filter:
+                errMsg = _("incorrect option") + ":" + " %s" %"--filter" +\
+                ": " + _("use with option '-v'")
+                self.optobj.error(errMsg)
+                return False
+            if optObj.xml:
+                errMsg = _("incorrect option") + ":" + " %s" %"--xml" +\
+                ": " + _("use with option '-v'")
+                self.optobj.error(errMsg)
+                return False
         return optObj, args
 
     def setUserName(self, userName):

scripts/cl-client

@@ -42,7 +42,7 @@ if __name__ == "__main__":
     # Печать переменных
     obj.printVars(opts)
     # Если нет печати переменных выполняем логику программы
-    if not opts.v and not opts.varsfilter:
+    if not opts.v and not opts.filter and not opts.xml:
         if args:
             domainName = args[0]
             # Ввод в домен

scripts/cl-passwd

@@ -42,7 +42,7 @@ if __name__ == "__main__":
     # Печать переменных
     obj.printVars(opts)
     # Если нет печати переменных выполняем логику программы
-    if not opts.v and not opts.varsfilter:
+    if not opts.v and not opts.filter and not opts.xml:
         # Изменение пароля пользователя
         if not obj.setUserPasswordToServer():
             sys.exit(1)

scripts/cl-sync

@@ -49,7 +49,7 @@ if __name__ == "__main__":
     # Печать переменных
     obj.printVars(opts)
     # Если нет печати переменных выполняем логику программы
-    if not opts.v and not opts.varsfilter:
+    if not opts.v and not opts.filter and not opts.xml:
         if userName:
             sync = not opts.nosync
             if opts.login: