accounts.admin中管理员创建普通博客账号时重构的save方法中user.source = 'adminsite'不生效

[JOYCAT-Q]

我确定我已经查看了 (标注[ ]为[x])

• DjangoBlog的readme
• 配置说明
• 其他 Issues

---

我要申请 (标注[ ]为[x])

• BUG 反馈
• 添加新的特性或者功能
• 请求技术支持

accounts.admin中的class BlogUserCreationForm(forms.ModelForm)中

    def save(self, commit=True):
        # Save the provided password in hashed format
        user = super().save(commit=False)
        user.set_password(self.cleaned_data["password1"])
        print("commit: ", commit)
        if commit:
            user.source = 'adminsite'
            user.save()
        return user

输出结果为 "commit: ", False
使得if判断后的语句无法正常执行，具体表现为用户的source设置失效

commit 参数为 False 的原因：
在 Django 管理员中，添加一个新用户时，ModelAdmin 会调用表单的 save_model() 方法来保存模型实例。这个方法通常不会直接调用表单的save()方法，而是接收一个已经保存（但可能未提交到数据库）的模型实例作为参数。

解决方法：在class BlogUserAdmin(UserAdmin)中重构方法

class BlogUserAdmin(UserAdmin):
    form = BlogUserChangeForm
    add_form = BlogUserCreationForm
    # 重构父类方法
    def save_model(self, request, obj, form, change):
        # 如果不是变更操作（即添加新对象）
        if not change:
            # 可以在这里设置额外的字段值
            obj.source = 'adminsite'
            # 调用父类的 save_model 方法来保存对象
        super().save_model(request, obj, form, change)
# 其他不变

同时将之前判断注释掉：

    def save(self, commit=True):
        # Save the provided password in hashed format
        user = super().save(commit=False)
        user.set_password(self.cleaned_data["password1"])
        # if commit:
        #     user.source = 'adminsite'
        #     user.save()
        return user

修改前后对比：

当前大体环境配置：
Django==4.2.14
Windows10
MySQL==8.0