Comment 2 for bug 1291414

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on horizon (master)

Change abandoned by Juan Manuel Ollé (<email address hidden>) on branch: master
Review: https://review.openstack.org/80323
Reason: This is wrong, overloading _clean_fields if a very bad idea, it depends on django internal implementation.

From an OO design point of view, extending CharField is the correct way to fix this issue.

Patch 6 is the correct way to fix this.

Thanks all of you for your time