Django视图自定义授权限制报错：未返回HttpResponse对象

核心错误：装饰器未返回视图函数的响应 #

你遇到的ValueError是因为自定义装饰器的wrapper函数在授权通过后，调用了被装饰的视图函数但没有返回它的执行结果。当授权逻辑通过时，wrapper执行完func(*args, **kwargs)后没有返回值，默认返回None，而Django要求视图必须返回HttpResponse或其子类对象。

次要问题与优化 #

除了核心错误，代码还有几个可以优化的点：

• 被装饰的edit_post视图重复获取Post实例，装饰器已经通过kwargs传入了instance，无需再次调用get_object_or_404
• 装饰器中通过args[0].user.id获取当前用户不够直观，直接从request对象（args[0]就是request）取更清晰
• 处理评论授权时，重定向的post_id应该从评论关联的文章中获取，而不是依赖kwargs中的值（编辑评论的视图可能只传入comment_id）

修复后的代码 #

修正后的装饰器 #

def authorize(func):
    def wrapper(*args, **kwargs):
        request = args[0]
        if kwargs.get('post_id'):
            instance = get_object_or_404(Post, id=kwargs.get('post_id'))
            if request.user.id != instance.author_id:
                return redirect('blog:post_detail', post_id=kwargs.get('post_id'))
            kwargs.update({'instance': instance})
        elif kwargs.get('comment_id'):
            instance = get_object_or_404(Comment, id=kwargs.get('comment_id'))
            if request.user.id != instance.author_id:
                # 从评论关联的文章中获取post_id
                return redirect('blog:post_detail', post_id=instance.post.id)
            kwargs.update({'instance': instance})
        # 关键：返回视图函数的执行结果
        return func(*args, **kwargs)
    return wrapper

修正后的编辑文章视图 #

@login_required
@authorize
def edit_post(request, post_id, instance=None):
    # 直接使用装饰器传入的instance，无需重复查询
    form = PostForm(request.POST or None, instance=instance)
    context = {'form': form}
    if form.is_valid():
        form.save()
    return render(request, 'blog/create.html', context)

内容的提问来源于stack exchange，提问作者chydik