Merge pull request #104 from novafloss/103_fix_disabled_attribute

Does not set the disabled attribute in widget

Author: moumoutte

demo/tests/test_forms.py

@@ -226,6 +226,15 @@ def test_readonly_field(self):
         self.assertEquals(field.required, False)
         self.assertEquals(field.widget.attrs['disabled'], True)
 
+    def test_no_disabled_attr(self):
+        self.text_field.accesses.create(access_id=u'human', level=EDITABLE)
+        form_class = self.form.get_django_form_class(role=u'human')
+        form = form_class()
+        self.assertIn('text-input', form.fields)
+        field = form.fields['text-input']
+        self.assertEquals(field.required, False)
+        self.assertNotIn('disabled', field.widget.attrs)
+
     def test_hidden_field(self):
         self.text_field.accesses.create(access_id=u'human', level=HIDDEN)
         form_class = self.form.get_django_form_class(role=u'human')

formidable/forms/field_builder.py

@@ -57,7 +57,12 @@ def get_widget_kwargs(self):
         return {'attrs': self.get_widget_attrs()}
 
     def get_widget_attrs(self):
-        return {'disabled': self.get_disabled()}
+        attrs = {}
+
+        if self.get_disabled():
+            attrs['disabled'] = True
+
+        return attrs
 
     def get_disabled(self):
         if self.access: