Allow creating of objects only with own mnt
This commit is contained in:
parent
dbcb963e3c
commit
80d4b1ad50
|
@ -55,18 +55,34 @@ class MntFormMixin(object):
|
||||||
for key in self.protectedFields:
|
for key in self.protectedFields:
|
||||||
self.fields[key].disabled = True
|
self.fields[key].disabled = True
|
||||||
|
|
||||||
#self.fields["mnt_by"].widget.attrs["data-role"] = "tagsinput"
|
|
||||||
instance = getattr(self, "instance", None)
|
instance = getattr(self, "instance", None)
|
||||||
|
if not hasattr(self, "_create"):
|
||||||
|
self._create = not (instance and instance.pk)
|
||||||
|
|
||||||
mntWhens = [When(auth=self._user, then=2)]
|
mntWhens = [When(auth=self._user, then=2)]
|
||||||
if instance and instance.pk:
|
if not self._create:
|
||||||
mntWhens.append(When(handle__in=instance.mnt_by.all().values_list("handle"), then=1))
|
mntWhens.append(When(handle__in=instance.mnt_by.all().values_list("handle"), then=1))
|
||||||
mntQs = Maintainer.objects.annotate(card=Case(*mntWhens, default=0, output_field=IntegerField())).order_by("-card")
|
mntQs = Maintainer.objects.annotate(card=Case(*mntWhens, default=0, output_field=IntegerField())).order_by("-card")
|
||||||
|
# NOTE: We cannot use distinct on a field as some db backends don't support it
|
||||||
|
#self.fields["mnt_by"].queryset = mntQs.distinct("handle")
|
||||||
self.fields["mnt_by"].queryset = mntQs
|
self.fields["mnt_by"].queryset = mntQs
|
||||||
|
|
||||||
if "mnt_lower" in self.fields:
|
if "mnt_lower" in self.fields:
|
||||||
self.fields["mnt_lower"].queryset = mntQs
|
self.fields["mnt_lower"].queryset = mntQs
|
||||||
|
|
||||||
|
|
||||||
|
def clean(self):
|
||||||
|
cleaned_data = super(MntFormMixin, self).clean()
|
||||||
|
|
||||||
|
if not self.errors and self._create:
|
||||||
|
mnts = self._user.maintainer_set.all()
|
||||||
|
|
||||||
|
for mnt in cleaned_data['mnt_by']:
|
||||||
|
if mnt in mnts:
|
||||||
|
break
|
||||||
|
else:
|
||||||
|
raise forms.ValidationError("On object creation at least one maintainer needs to be under your control")
|
||||||
|
|
||||||
class MntForm(WhoisObjectFormMixin, forms.ModelForm):
|
class MntForm(WhoisObjectFormMixin, forms.ModelForm):
|
||||||
class Meta:
|
class Meta:
|
||||||
model = Maintainer
|
model = Maintainer
|
||||||
|
|
Loading…
Reference in New Issue