TextField constructor needs a strip=False option

[Rob van der Linde]

In Django 1.9 an option strip=True was added to CharField, this is fine but it also seems to affect TextField and there is no way to pass a strip=False option to TextField constructor to turn that off.

The issue seems to be the formfield method, this has lead to people subclassing TextField to "fix" this addition to Django 1.9, for example:

class NonStrippingTextField(TextField):
    """A TextField that does not strip whitespace at the beginning/end of
    it's value.  Might be important for markup/code."""

    def formfield(self, **kwargs):
        kwargs['strip'] = False
        return super(NonStrippingTextField, self).formfield(**kwargs)

Other people fix it in the form and override the admin form, this has become a bit messy, it seems to be necessary to introduce a strip argument to the TextField constructor, to avoid having to create custom field types.

[Simon Charette]

Just to make it clear the strip=True option was added to forms.CharField and not db.CharField like the report seems to imply. Moving this option to the database layer has been discussed and rejected when the option was added to the form layer so I doubt there's consensus on adding an option to specialize TextField here given striping spaces should be the desired behaviour in most cases just like with db.CharField.

Other people fix it in the form and override the admin form

Can't this be simply done with ​formfield_overrides or formfield_for_dbfield?

[Rob van der Linde]

Ah I see, I'll have to find a Wagtail specific fix, though I must say I do find this feature added to Django 1.9 extremely annoying.