Prevent empty WorkerConfiguration names
Closes #1808 (closed)
I first tried adding a MinLengthValidator
, but its validation does not apply when creating instances with the ORM directly; this only impacts the API (which already enforced non-empty config names) and the Django admin (same, since it does not have blank=True
). I left it there because it doesn't cause any trouble, and added a CHECK
constraint instead, which is the only way to really ensure names cannot be empty.
I spent some time analyzing every other text field and found many other cases where they could be empty. For example, ImageServer.objects.create()
works just fine, and creates a server with no name and no URL. I will file issues to deal with that later.
Merge request reports
Activity
changed milestone to %Arkindex 1.6.2
requested review from @babadie
assigned to @erouchet
added 6 commits
-
1585f47f...ce9a04a3 - 5 commits from branch
master
- 3bdb1a1f - Prevent empty WorkerConfiguration names
-
1585f47f...ce9a04a3 - 5 commits from branch
mentioned in merge request !2402 (merged)