From 014f3ff1c63eeff6d6bc36cbdce7f082676d6925 Mon Sep 17 00:00:00 2001
From: Rob Crittenden <rcritten@redhat.com>
Date: Fri, 8 May 2009 14:10:53 -0400
Subject: [PATCH] Improve revocation_reason argument

---
 ipalib/plugins/cert.py | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/ipalib/plugins/cert.py b/ipalib/plugins/cert.py
index 01d3266b9..cd6e31893 100644
--- a/ipalib/plugins/cert.py
+++ b/ipalib/plugins/cert.py
@@ -151,7 +151,14 @@ class cert_revoke(Command):
     takes_args = ['serial_number']
 
     # FIXME: The default is 0.  Is this really an Int param?
-    takes_options = [Int('revocation_reason?', default=0)]
+    takes_options = (
+        Int('revocation_reason?',
+            doc='Reason for revoking the certificate (0-10)',
+            minvalue=0,
+            maxvalue=10,
+            default=0,
+        ),
+    )
 
 
     def execute(self, serial_number, **kw):