Đặt giá trị trường theo phương thức clean() Django Form, nếu trường này không được truyền trong hàm tạo

Question

Tôi cần đặt giá trị trường, không được truyền cho hàm tạo của Django Form.

tôi có mô hình và hình thức như thế này:

class Message(models.Model): 
    created  = models.DateTimeField() 
    text  = models.CharField(max_length=200, blank=True, null=True) 
    active  = models.BooleanField(default=False) 

class MessageForm(forms.ModelForm): 
    class Meta: 
     model = Message 
     exclude = ('created', 'active') 

    def clean(self): 
     # check if user is blocked 
     if user.is_admin(): 
      self.cleaned_data['active'] = True 
     return self.cleaned_data 

Dự kiến ​​: nếu người dùng hiện nay là admin - Tôi không cần tự động thiết lập thông điệp như hoạt động. Người dùng không được chuyển thông số này theo biểu mẫu.

Thực tế: Tôi thấy thông báo đã lưu luôn có cờ "Sai" (Tôi có thể xóa điều kiện và trong trường hợp này tôi cũng thấy thông báo đó không hoạt động).

Hãy giúp tôi hiểu, làm cách nào tôi có thể đặt phương thức "hoạt động" này ở clean().

Answer 1

Câu trả lời trước sẽ hoạt động, nhưng tôi thích đóng gói tất cả các hoạt động nội bộ của biểu mẫu như những gì hiển thị và những gì không có trong biểu mẫu. Tôi biết bạn đã đề cập bạn không muốn gửi giá trị trường tới hàm tạo, nhưng nếu bạn không nhớ gửi người dùng, giải pháp của bạn sẽ hoạt động. ví dụ: nhà xây dựng của bạn:

def __init__(self, user): 
    self.user = user 
    super(BaseForm, self).__init__() 

sau đó trong sạch của bạn, bạn chỉ cần thay đổi user-self.user. Có một lợi ích bổ sung cho điều này. Giả sử ngày mai bạn muốn chỉ định nhiều trường hơn dựa trên người dùng của mình, bạn không cần thêm bất kỳ thứ gì vào chế độ xem, bạn có thể chỉ cần thêm trường vào biểu mẫu.

EDIT: Khi bạn thêm trường để loại trừ, trường này không có sẵn trong dữ liệu được làm sạch. Thay vào đó, hãy đặt tiện ích con của nó là ẩn.

active = forms.BooleanField(widget=forms.HiddenInput) 

EDIT 2: Nếu bạn thực sự không muốn các lĩnh vực theo hình thức Trong trường hợp này, thay vì trọng sạch sẽ, tại sao bạn không ghi đè tiết kiệm?

def save (self): 
    super(BaseForm, self).save() 
    if user.is_admin(): 
     self.instance.active=True 
    super(BaseForm, self).save() 

Answer 2

Không làm điều này trong phương thức clean() của biểu mẫu, thực hiện việc này trong chế độ xem.

def your_view(request): 
    if request.method == 'POST': 
     form = MessageForm(data=request.POST) 
     if form.is_valid(): 
      new_message = form.save(commit=False) 
      if user.is_admin(): 
       new_message.active = True 

Tuy nhiên, nếu bạn cũng muốn để xử lý các trường hợp sử dụng của bạn là không quản trị sử dụng các hình thức tương tự, bạn có thể nhìn vào kết hợp logic tương tự trong init của hình thức() thay vì quan điểm, có lẽ bởi chuyển thông tin về người dùng từ chế độ xem đến biểu mẫu init()

Answer 3

Sử dụng này:

def message_form_factory(user): 
    class MessageForm(forms.ModelForm): 
     def clean(self): 
      # check if user is blocked 
      if user.is_admin(): 
       self.cleaned_data['active'] = True 
      return self.cleaned_data 
    return MessageForm 

Và theo quan điểm sử dụng của bạn:

form = message_form_factory(request.user)() 
form = message_form_factory(request.user)(request.POST)