From 20ee4a73e7b9e0ccdc7725ff4b87404d5543992e Mon Sep 17 00:00:00 2001 From: Jan Cholasta Date: Wed, 10 Aug 2016 09:02:47 +0200 Subject: [PATCH] client: add missing output params to client-side commands Add output params for the otptoken-add-yubikey, vault-add, vault-mod, vault-archive and vault-retrieve commands. This fixes the commands not having any output in CLI. https://fedorahosted.org/freeipa/ticket/6182 Reviewed-By: Alexander Bokovoy Reviewed-By: David Kupka --- ipaclient/plugins/otptoken_yubikey.py | 6 ++++++ ipaclient/plugins/vault.py | 24 ++++++++++++++++++++++++ 2 files changed, 30 insertions(+) diff --git a/ipaclient/plugins/otptoken_yubikey.py b/ipaclient/plugins/otptoken_yubikey.py index bfa244c88..549376a0f 100644 --- a/ipaclient/plugins/otptoken_yubikey.py +++ b/ipaclient/plugins/otptoken_yubikey.py @@ -106,6 +106,12 @@ class otptoken_add_yubikey(Command): for option in super(otptoken_add_yubikey, self).get_options(): yield option + def get_output_params(self): + for param in self.api.Command.otptoken_add.output_params(): + yield param + for param in super(otptoken_add_yubikey, self).get_output_params(): + yield param + def _iter_output(self): return self.api.Command.otptoken_add.output() diff --git a/ipaclient/plugins/vault.py b/ipaclient/plugins/vault.py index 1e715fdb9..c0ded21d5 100644 --- a/ipaclient/plugins/vault.py +++ b/ipaclient/plugins/vault.py @@ -223,6 +223,12 @@ class vault_add(Local): for option in super(vault_add, self).get_options(): yield option + def get_output_params(self): + for param in self.api.Command.vault_add_internal.output_params(): + yield param + for param in super(vault_add, self).get_output_params(): + yield param + def _iter_output(self): return self.api.Command.vault_add_internal.output() @@ -423,6 +429,12 @@ class vault_mod(Local): for option in super(vault_mod, self).get_options(): yield option + def get_output_params(self): + for param in self.api.Command.vault_mod_internal.output_params(): + yield param + for param in super(vault_mod, self).get_output_params(): + yield param + def _iter_output(self): return self.api.Command.vault_mod_internal.output() @@ -607,6 +619,12 @@ class vault_archive(Local): for option in super(vault_archive, self).get_options(): yield option + def get_output_params(self): + for param in self.api.Command.vault_archive_internal.output_params(): + yield param + for param in super(vault_archive, self).get_output_params(): + yield param + def _iter_output(self): return self.api.Command.vault_archive_internal.output() @@ -855,6 +873,12 @@ class vault_retrieve(Local): for option in super(vault_retrieve, self).get_options(): yield option + def get_output_params(self): + for param in self.api.Command.vault_retrieve_internal.output_params(): + yield param + for param in super(vault_retrieve, self).get_output_params(): + yield param + def _iter_output(self): return self.api.Command.vault_retrieve_internal.output()