Почему не добавляется файл в админку Django через форму из html?

Question

[leo9797]

Хочу создать запись в модель, но при создании добавляются текста о имени и описании файла но не сам файл. Почему? Кто сталкивался?

#views
class AddServ(FormView):
    ....
    def post(self, request, *args, **kwargs):
        form = AddServiceForm(request.POST, request.FILES)
        model = Service.objects.all()
        to_pk = model.count() + 1
        name_service = form['name_service'].data
        about_service = form['about_service'].data
        file_service = form['file_service'].data
        user = request.user
        if user.is_authenticated:
            user.loads_services.create(
                pk=to_pk,
                name_service=name_service,
                about_service=about_service,
                file_service=file_service
            )
            user.save()
        return render(request, 'elitemservice/main.html')

#model

class Service(models.Model):
    name_service = models.CharField(max_length=255, verbose_name='Назание сервиса')
    about_service = models.TextField(verbose_name='Описание микросервиса')
    file_service = models.FileField(upload_to='upload_microservice', verbose_name='Файл микросервиса')

#forms

class AddServiceForm(forms.ModelForm):
    name_service = forms.CharField(max_length=255, label="Имя сервиса")
    about_service = forms.CharField(widget=forms.Textarea, label='Описание сервиса')
    file_service = forms.FileField(label='Файл сервиса')

    class Meta:
        model = Service
        fields = '__all__'

{% load bootstrap4 %}
{% if user.is_authenticated %}
    <form method="post" enctype="multipart/form-data">
        {% csrf_token %}
        {% for field in form %}
            {% bootstrap_field field layout='horizontal' %}
        {% endfor %}
        <label>
            <input type="button" value="Добавить сервис" id="addService">
        </label>
    </form>
{% else %}
    <p>Что бы добавить сервис требуется авторизация!</p>
{% endif %}

Comments

[Владимир]

Здесь "прекрасно" всё. Возникают вопросы (надеюсь не риторические):

• Зачем вы в классе отображения переопределяете метод post, когда FormView, от которого вы наследуетесь, предполагает переопределение методов is_valid и is_invalid?
  
• Зачем вы всё пытаетесь сделать вручную, но при этом используете FormView, который автоматизирует работу с формами?
  
• Зачем вы рассчитываете первичный ключ?
  
• Зачем вы делаете класс формы, если можно использовать CreateView, который сам по модели сделает класс формы?
  
• Зачем вы в своём классе МОДЕЛЬНОЙ формы дублируете поля модели?
  

[leo9797]

1) Изменил на CreateView
2) Расчитываю ключ из за этого:

user.loads_services.create(
                pk=to_pk,
                name_service=name_service,
                about_service=about_service,
                file_service=file_service
            )

у user есть поле loads_service с ManyToManyField и для записи в саму модель и в поле loads_service надо указать первичный ключ созданной записи
3) Незнал что он по моделе делает класс формы....

[leo9797]

Владимир, 4) Я указываю класс формы для того что бы вызвать get функцию и передать туда этот класс....

Answer 1

[Владимир]

class ServiceCreateView(CreateView):
    model = Service
    template_name = 'elitemservice/main.html'

    def form_valid(self, form):
        resp = super(ServiceCreateView, self).form_valid(form)
        # после вызова суперметода в self.object находится созданный по форме объект
        user = self.request.user
        if user.is_authenticated:
            user.loads_services.create(
                pk=self.object.pk,
                name_service=self.object.name_service,
                about_service=self.object.about_service,
                file_service=self.object.file_service,
            )
            user.save()
        return resp

Comments

[leo9797]

У меня был другой вопрос, почему при передачи файла request.FILE пуст? Я указывал в форме

enctype="multipart/form-data"

. Но всеравно передав значение поля request.FILE пуст

[Владимир]

leo9797, потому что request.FILE != request.FILES.

[Владимир]

Дополнил ответ логикой, связанной с user.loads_services.

[leo9797]

Владимир, Выводит следующую ошибку

django.core.exceptions.ImproperlyConfigured: Using ModelFormMixin (base class of ServiceCreateView) without the 'fields' attribute is prohibited.

Я же правильно вас понял, надо передать форму в данный view? Я использую для этого AJAX

$.ajax({
                type: "POST",
                url: "{% url 'doneAddService' %}",
                data: {
                    'form': $('#addServiceForm').serialize()
                },
                success: function () {
                   window.location = '/'
                }
            })

[leo9797]

Владимир, извеняюсь, не так написал request.FILES пуст

[Владимир]

leo9797, так бы и сказали, что ajax-ом отправляете. Вы неправильно это делаете. Вернее этот способ годится, если в форме нет файловых полей. Лучше всегда это делать через FormData:

var $form = $('#addServiceForm'),
    form_action = $form.attr('action'),
    form_data = new FormData($form[0]);
$.ajax({
    type: "POST",
    url: form_action,
    data: form_data,
    cache: false,
    contentType: false,
    processData: false,
    success: function () {
        window.location = '/'
    }
});

Это будет максимально близко к тому, что вы просто отправили форму безо всякого javascript.

[leo9797]

Владимир, Воспользовался вашим кодом:

class ServiceCreateView(CreateView):
    model = Service
    template_name = 'elitemservice/main.html'

    def form_valid(self, form):
        resp = super(ServiceCreateView, self).form_valid(form)
        # после вызова суперметода в self.object находится созданный по форме объект
        user = self.request.user
        if user.is_authenticated:
            user.loads_services.create(
                pk=self.object.pk,
                name_service=self.object.name_service,
                about_service=self.object.about_service,
                file_service=self.object.file_service,
            )
            user.save()
        return resp

Мне выдает ошибка о дубликате ключа....если кстати я пытаюсь использовать вместо create get то мне пишет о том что ключ несуществует....

[leo9797]

Владимир, Все решил проблемы немного исправив функцию form_valid

method_decorator(csrf_protect)

    def form_valid(self, form):
        resp = super(DoneAdd, self).form_valid(form)
        # после вызова суперметода в self.object находится созданный по форме объект
        user = self.request.user
        model = Service.objects.all()
        if user.is_authenticated:
            user.loads_services.add(model.get(pk=self.object.pk))
            user.save()
        return resp

После чего ошибки о дубликате id нету...и запись созданная добавляется автоматически и пользователю в поле.

[Владимир]

leo9797, рад, что смог помочь! Только связь можно проще установить:
user.loads_services.add(self.object)
В вашем варианте результат тот же, только выполняется лишний sql-запрос.

[leo9797]

Владимир, учту, спасибо за подсказки)