ome / omero-web

Django-based OMERO.web client
https://www.openmicroscopy.org/omero
16 stars 29 forks source link

Deprecate omeroweb.custom_forms.NonASCIIForm #473

Closed sbesson closed 1 year ago

sbesson commented 1 year ago

Fixes #468

The original intent of the NonASCIIForm was to support Unicode strings by overriding the CharField validation. Unicode is now natively supported by the Python/Django stack. This commit proposes to drop our intermediate custom class in favor of upstream django.forms.Form. This should incidentally reduce the maintenance burden when upgrading between Django major versions. Testing:

All places using the legacy base form should be reviewed with this change using both ASCII strings and Unicode strings as inputs. Below is a list of endpoints to review