Django UpdateView 创建新对象

Django UpdateView create new object

我的问题: UpdateView 创建新对象而不是更新以前的对象,我认为它发生是因为在我的视图的 class 定义中我重写了 get_object 方法,如下所示:

def get_object(self, queryset=None):
    try:
        object_get = self.model.objects.get(pk=self.kwargs['pk'])
    except ObjectDoesNotExist:
        raise Http404("No object found matching this query")

    if self.request.user.is_authenticated():
        if object_get.owner == self.request.user:
            return object_get

因此,如果当前用户不是该对象的所有者 - 此方法 return 没什么 - 这是我想要的,但我的表单 class 而是创建新对象:

class ClientCreation(forms.ModelForm):

    class Meta:
        model = Client
        fields = ('name', 'loyal')

我认为这是因为表单没有收到 self.instance 而是创建新的 - 在这种情况下我应该怎么做?我不想创建新对象,以防万一对象的所有者不是当前用户我不想发生任何事情然后发送这样的 post 请求。我应该如何正确实施它?

更新views.py:

class Distinct(generic.UpdateView):
    def get_object(self, queryset=None):
        try:
            object_get = self.model.objects.get(pk=self.kwargs['pk'])
        except ObjectDoesNotExist:
            raise Http404("No object found matching this query")

        if self.request.user.is_authenticated():
            if object_get.owner == self.request.user:
                return object_get

    def get_form_kwargs(self):
        kwargs = super(Distinct, self).get_form_kwargs()
        if self.request.user.is_authenticated():
            kwargs.update({'user': self.request.user})
        return kwargs

    def post(self, request, *args, **kwargs):
        if request.POST.get('action', '') == 'Delete':
            object_get = self.get_object()
            request.session['deleted_data'] = str(object_get)
            object_get.delete()
            return redirect(reverse('crm:main'))
        else:
            return super(Distinct, self).post(request, *args, **kwargs)

    def get_success_url(self):
        return reverse('crm:{}'.format(self.distinct_template), kwargs={'pk': self.kwargs['pk']})

class DistinctClient(Distinct):

    form_class = ClientCreation
    model = Client
    template_name = 'crm/client_detail.html'
    all_template = 'clients'
    distinct_template = 'client'

    def get_form_kwargs(self):

        return generic.UpdateView.get_form_kwargs(self)

UpdateView中,如果get_object returns None django会创建一个新的object.So而不是return None 随心所欲

def get_object(self, queryset=None):
    try:
        object_get = self.model.objects.get(pk=self.kwargs['pk'])
    except ObjectDoesNotExist:
        raise Http404("No object found matching this query")

    if self.request.user.is_authenticated():
        if object_get.owner == self.request.user:
            return object_get
    raise My #do something here.

更新

class My(Exception):
    pass
class DistinctClient(Distinct):

    form_class = ClientCreation
    model = Client
    template_name = 'crm/client_detail.html'
    all_template = 'clients'
    distinct_template = 'client'

    def dispatch(self, *args, **kwargs):
        try:
            return super(DistinctClient, self).dispatch(*args, **kwargs)
        except My:
            return redirect #to do or (return render(self.request, 'mytemplate.html', {}))