* Added automatic 'required-field' indicator to the form rendering

 * Made 'Legal Name' field in the IprDetails model required
 - Legacy-Id: 107

Author: levkowetz

ietf/ipr/models.py

@@ -77,7 +77,7 @@ class Admin:
 
 class IprDetail(models.Model):
     ipr_id = models.AutoField(primary_key=True)
-    p_h_legal_name = models.CharField("Legal Name", blank=True, maxlength=255)
+    p_h_legal_name = models.CharField("Legal Name", maxlength=255)
     document_title = models.CharField(blank=True, maxlength=255)
     rfc_number = models.IntegerField(null=True, editable=False, blank=True)	# always NULL
     id_document_tag = models.IntegerField(null=True, editable=False, blank=True)	# always NULL

ietf/templates/ipr/formfield.html

@@ -5,4 +5,5 @@
       {% endfor %}
    </ul>
    {% endif %}
-   {{ field }}{{ help_text }}
+   {% if field.required %}<span class="required">*</span>{% endif %}{{ text }}
+   {{ help_text }}

ietf/templates/ipr/style.html

@@ -22,6 +22,6 @@
    input[type="checkbox"] { width: auto; }
    input[type="submit"] { width: auto; }
    textarea	{ width: 72ex; height: 5em; font-family: sans-serif; font-size: 11pt; font-weight: normal; }
-   .required	{ color: red; }
+   .required	{ color: red; float: right; padding-top: 1ex; }
    .errorlist	{ background: red; padding: 0 0 0 2px; border: 0px; margin: 0px; }
    </style>

ietf/utils.py

@@ -266,6 +266,6 @@ def __getitem__(self, name):
                 return field
             bf = forms.forms.BoundField(self, field, name)
             errors = [escape(error) for error in bf.errors]
-            rendering = loader.render_to_string(self._template, { "errors": errors, "label": bf.label, "field": unicode(bf), "help_text": field.help_text })
+            rendering = loader.render_to_string(self._template, { "errors": errors, "label": bf.label, "text": unicode(bf), "help_text": field.help_text, "field":field })
             return rendering
     return FormattingForm