aboutsummaryrefslogtreecommitdiffstats
path: root/pydis_site/apps/api/models
diff options
context:
space:
mode:
authorGravatar Leon Sandøy <[email protected]>2020-07-29 13:55:46 +0200
committerGravatar Leon Sandøy <[email protected]>2020-07-29 13:55:46 +0200
commit0d7c95a80692f15c36b36152034e3bbe54e3c37c (patch)
tree5f98405211fd9af8be3497b89fbb08fe53fe94f6 /pydis_site/apps/api/models
parentRename 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.py11
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'),
- ]