From 1a03bd322df65ecb9302f0cd70ec2b9bbfa3812e Mon Sep 17 00:00:00 2001 From: Jan Cholasta Date: Wed, 29 Jun 2016 18:11:41 +0200 Subject: cert: fix CLI output of cert_remove_hold cert_remove_hold uses output params instead of exceptions to convey unsuccessful result. Move the output params to the client side before the command is fixed to use exceptions. https://fedorahosted.org/freeipa/ticket/4739 Reviewed-By: David Kupka --- ipaclient/plugins/cert.py | 14 +++++++++++++- 1 file changed, 13 insertions(+), 1 deletion(-) (limited to 'ipaclient/plugins') diff --git a/ipaclient/plugins/cert.py b/ipaclient/plugins/cert.py index de4318b68..37e894efb 100644 --- a/ipaclient/plugins/cert.py +++ b/ipaclient/plugins/cert.py @@ -23,7 +23,7 @@ from ipaclient.frontend import MethodOverride from ipalib import errors from ipalib import x509 from ipalib import util -from ipalib.parameters import File +from ipalib.parameters import File, Flag, Str from ipalib.plugable import Registry from ipalib.text import _ @@ -54,6 +54,18 @@ class cert_show(MethodOverride): return super(cert_show, self).forward(*keys, **options) +@register(override=True) +class cert_remove_hold(MethodOverride): + has_output_params = ( + Flag('unrevoked', + label=_('Unrevoked'), + ), + Str('error_string', + label=_('Error'), + ), + ) + + @register(override=True) class cert_find(MethodOverride): takes_options = ( -- cgit