Как автоматически указать автора записи?

Question

[zkweb]

Есть модель:

class Request(models.Model):
    **        
    usercreated = models.ForeignKey(User, blank=True, null=True)

Как мне указать чтобы поле usercreated заполнялось в БД автоматом, после(перед) сохранением записи. ID текущего пользователя. Делал с помощью сигнала, но не вышло, т.к request не может найти.

Проект только в административном интерфейсе, без cbv

Comments

[Сергей Горностаев]

Никак. Юзер определён только в области видимости представления, вместе с request. Соответственно, в представлении и устанавливать значение поля.

[zkweb]

Сергей Горностаев: что можно придумать? переход на cbv?

[Сергей Горностаев]

Pavel Denisov: Когда я писал ответ, последнего абзаца не было.

Answer 1

[zkweb]

Вот решение, которое помогло мне. Всем спасибо!

def save_formset(self, request, form, formset, change):
        instances = formset.save(commit=False)
        for instance in instances:
            if isinstance(instance, Request):
                if(not instance.usercreated):
                    instance.usercreated = request.user          
                instance.save()

Answer 2

[iegor]

Можете поймать юзера в миддлваре и потом достать его от туда при сейве

Comments

[zkweb]

можно пример? интересно

[iegor]

https://habrahabr.ru/post/130566/

[iegor]

Pavel Denisov: а что не так с threading.local? В чем костыль то?

Answer 3

[Alexander Lebedev]

У себя в проектах долгое время делал так:
1. Модель со всеми общими полями для аудита:

models.py
class Audit(models.Model):
    """
    A common class for auditing the models' instances.
    """
    created_by = models.ForeignKey(
        User,
        related_name='%(app_label)s_%(class)s_created_related',
        verbose_name=_(u'Создал'),
        on_delete=models.SET_NULL,
        blank=True, null=True)
    changed_by = models.ForeignKey(
        User,
        related_name='%(app_label)s_%(class)s_changed_related',
        verbose_name=_(u'Изменил'),
        on_delete=models.SET_NULL,
        blank=True, null=True)
    created = models.DateTimeField(
        _(u'Дата создания'), auto_now_add=True)
    changed = models.DateTimeField(
        _(u'Дата изменения'), auto_now=True)
    class Meta:
        abstract = True

2. Для админки:

admin.py
class AuditAdmin(admin.ModelAdmin):
    """
    Admin class for audit fields
    """
    readonly_fields = (
        'created_by',
        'changed_by',
        'created',
        'changed')

    fieldsets_preset = (
        _(u'Прочее'), {
            'fields': (
                'created_by',
                'created',
                'changed_by',
                'changed'
            )
        })
    actions_on_top = True

    def save_model(self, request, obj, form, change):
        if not change:
            obj.created_by = request.user
        obj.changed_by = request.user
        obj.save()

Наследуем все модели и админ-классы, для которых необходимо следить за пользователями, и радуемся.

Это решение перестало меня удовлетворять, как только моделей стало слишком много -- список related отношений для класса User стал просто невыносим. Работаю над переходом к хранению логина пользователя в текстовом виде вместо внешнего ключа на пользователя.

Comments

[Илья]

А разве не следует возвращать super(AuditAdmin, self).save_model(*args, **kwargs)?

[Alexander Lebedev]

FireGM: Pavel Denisov: формально - да, так корректней. Однако, если глянуть в исходники, то ModelAdmin save_model(...) содержит в себе только obj.save().
Если бы понадобилось внедрить более сложную логику (наследовать, например, еще от одного админ-класса с какой-то своей дополнительной логикой сохранения), то да, конечно super()

[Илья]

Alexander Lebedev: у него это примись.