set default value of max_length to prefix length, and change help text to make it clear this field should be an integer without a leading slash. closes #132

svn path=/trunk/; revision=4087

1 files changed, 3 insertions, 9 deletions

diff --git a/rpkid/rpki/gui/app/forms.py b/rpkid/rpki/gui/app/forms.py
index 41e154e5..aad9185d 100644
--- a/rpkid/rpki/gui/app/forms.py
+++ b/rpkid/rpki/gui/app/forms.py
@@ -94,16 +94,10 @@ def PrefixRoaForm(prefix, *args, **kwargs):
     class _wrapper(forms.Form):
         asns = forms.CharField(max_length=200, required=False,
                 help_text='Comma-separated list of ASNs')
-        max_length = forms.IntegerField(required=False,
-                min_value=prefix_range.prefixlen(),
+        max_length = forms.IntegerField(min_value=prefix_range.prefixlen(),
                 max_value=prefix_range.datum_type.bits,
-                help_text='optional, defaults to /%d' % prefix_range.prefixlen())
-
-        def clean_max_length(self):
-            v = self.cleaned_data.get('max_length')
-            if not v:
-                v = prefix_range.prefixlen()
-            return v
+                initial=prefix_range.prefixlen(),
+                help_text='must be in range %d-%d' % (prefix_range.prefixlen(), prefix_range.datum_type.bits))
 
         def clean_asns(self):
             try: