diff options
author | 2020-07-29 13:55:46 +0200 | |
---|---|---|
committer | 2020-07-29 13:55:46 +0200 | |
commit | 0d7c95a80692f15c36b36152034e3bbe54e3c37c (patch) | |
tree | 5f98405211fd9af8be3497b89fbb08fe53fe94f6 /pydis_site/apps/api/models | |
parent | Rename AllowDenyList to FilterList (diff) |
Handle unique validator in DRF, not Django.
I was handling this in a Django vanilla kind of way, which was causing
the constraint to return a 500 instead of a 400. This changes the
approach to use the DRF way, and makes it return 400.
It doesn't actually change the way anything behaves, other than
returning the right status code.
Diffstat (limited to 'pydis_site/apps/api/models')
-rw-r--r-- | pydis_site/apps/api/models/bot/filter_list.py | 11 |
1 files changed, 0 insertions, 11 deletions
diff --git a/pydis_site/apps/api/models/bot/filter_list.py b/pydis_site/apps/api/models/bot/filter_list.py index d279e137..5961aed7 100644 --- a/pydis_site/apps/api/models/bot/filter_list.py +++ b/pydis_site/apps/api/models/bot/filter_list.py @@ -28,14 +28,3 @@ class FilterList(ModelTimestampMixin, ModelReprMixin, models.Model): help_text="Optional comment on this entry.", null=True ) - - class Meta: - """Metaconfig for this model.""" - - # This constraint ensures only one filterlist with the - # same content can exist. This means that we cannot have both an allow - # and a deny for the same item, and we cannot have duplicates of the - # same item. - constraints = [ - models.UniqueConstraint(fields=['content', 'type'], name='unique_filter_list'), - ] |