The default revocation reason wasn't autofilling so trying to retrieve it when it wasn't set caused things to blow up.

ticket https://fedorahosted.org/freeipa/ticket/1514
>From ea00b59d3c3ae61cd60d24120a9b9cbe5baa41c6 Mon Sep 17 00:00:00 2001
From: Rob Crittenden <rcrit...@redhat.com>
Date: Thu, 21 Jul 2011 16:46:01 -0400
Subject: [PATCH] Autofill the default revocation reason

https://fedorahosted.org/freeipa/ticket/1514
---
 API.txt                |    2 +-
 ipalib/plugins/cert.py |    1 +
 2 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/API.txt b/API.txt
index 863adb0..dea82e2 100644
--- a/API.txt
+++ b/API.txt
@@ -315,7 +315,7 @@ output: Output('result', <type 'dict'>, Gettext('Dictionary mapping variable nam
 command: cert_revoke
 args: 1,1,1
 arg: Str('serial_number', validate_serial_number, label=Gettext('Serial number', domain='ipa', localedir=None), normalizer=normalize_serial_number)
-option: Int('revocation_reason?', default=0, label=Gettext('Reason', domain='ipa', localedir=None), maxvalue=10, minvalue=0)
+option: Int('revocation_reason?', autofill=True, default=0, label=Gettext('Reason', domain='ipa', localedir=None), maxvalue=10, minvalue=0)
 output: Output('result', None, None)
 command: cert_show
 args: 1,1,1
diff --git a/ipalib/plugins/cert.py b/ipalib/plugins/cert.py
index 643e1cd..2c8ab49 100644
--- a/ipalib/plugins/cert.py
+++ b/ipalib/plugins/cert.py
@@ -536,6 +536,7 @@ class cert_revoke(VirtualCommand):
             minvalue=0,
             maxvalue=10,
             default=0,
+            autofill=True
         ),
     )
 
-- 
1.7.4

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to