Как исправить ошибку в Django — add_view?

Question

[zkweb]

Есть 2 группы пользователей: продавцы, директор магазина
Ну и суперадмин, который видит всё и может редактировать всё.

Для продавца при добавлении записи скрыты определенные поля, тоже самое для директора.

Вот код:

def add_view(self, request, form_url='', extra_context=None):
        if not request.user.is_superuser:
            if request.user.groups.all()[0].id == 2:
                self.exclude=('date_job', 'text_job', 'vendors_response', 'solution', 'response_buyer', 'status', 'user', 'shop')
            elif request.user.groups.all()[0].id == 4:
                self.exclude=('date_job', 'text_job', 'vendors_response', 'solution', 'response_buyer', 'status', 'user') 
            else:    
                self.exclude=('date_job')             
        return super(ClaimAdmin, self).add_view(request, form_url='', extra_context=None)

Проблема: открыты два браузера. В одном под суперадмином, в другом под продавцом.
Добавляю запись под продавцом, всё в порядке, нужные поля скрыты.
Добавляю под суперадмином, ТАКЖЕ все поля скрыты, обновляю страницу (раз 5) вдруг доступны все остальные поля, снова обновляю опять скрыто (

Как такое может быть?

Answer 1

[Roman K]

exclude - поле класса, а не объекта. В __init__ вьюхи пропиши self.exclude = None

Comments

[zkweb]

простите, не совсем понял куда прописать (новичок еще в джанго). Этот код у меня прописан в admin.py

[Roman K]

zkweb: изменил ответ. Так понятнее?

[zkweb]

Roman Kitaev:
вроде бы. А если так, не правильно будет ?

def add_view(self, request, form_url='', extra_context=None):
self.exclude = None

[Roman K]

zkweb: Так ты перезапишешь поле класса. Я бы не назвал это правильным, в общем-то, как и сравнивание id с 2 и 4.

[zkweb]

Roman Kitaev: согласен. Т.е прописать так:

admin.py

def __init__(self, *args, **kwargs):
self.exclude = None

?

А как бы лучше сделали по этому: "как и сравнивание id с 2 и 4" ? решение профи хочу услышать. Понятно, что у меня это костыль какой-то

[Roman K]

zkweb:

def __init__(self, *args, **kwargs):
    super(ClaimAdmin, self).__init__(*args, **kwargs)
    self.exclude = None

Никаких сравнений по id делать не надо. Есть отличный класс Permission "из коробки". Эти пермишены можно как юзеру натолкать, так и группе.

[zkweb]

Roman Kitaev: спасибо больше, Роман. Может небольшой пример кода приведёте про Permission, именно по моей задаче с проверкой группы?

[Roman K]

zkweb: Для начала нужно создать те самые права, затем просто проверять:

if request.user.has_perm('director-permission'):
    self.exclude=('date_job', 'text_job', 'vendors_response', 'solution', 'response_buyer', 'status', 'user', 'shop')
elif request.user.has_perm('manager-permission'):
    self.exclude=('...')
else:
    ...

А уж Django разберётся, где искать права. К слову, права конкретного пользователя - это совокупность множеств прав всех его групп и его собственных прав.

[zkweb]

Roman Kitaev: спасибо, буду разбираться! Роман, по основному вопросу всё таки:

def __init__(self, *args, **kwargs):
super(ClaimAdmin, self).__init__(*args, **kwargs)
self.exclude = None

- это у меня не срабатывает

а если так:

def add_view(self, request, form_url='', extra_context=None):
self.exclude = None

то работает, странно конечно. но факт

В любом случае вам спасибо, помогли

[Roman K]

zkweb: Что именно не работает? Оно не может просто "не работать".

[Roman K]

zkweb: Ну и скинь весь код класса. Может, куда-то не туда вставил :)

[zkweb]

Roman Kitaev:
"не работает", значит мой вопрос снова актуален.

Весь код admin.py

# -*- coding: utf-8 -*-
import sys
reload(sys)
sys.setdefaultencoding('utf8')
from django.contrib import admin
from claim.models import Claim
from claim.models import EntryImage
from django.db.models.query import QuerySet
from django.contrib.auth.models import User
import datetime

class ImageInline(admin.StackedInline):
    model = EntryImage
    
class ClaimAdmin(admin.ModelAdmin):
    list_display = ('client', 'item_product', 'date_add', 'shop', 'prodavec_show', 'claim', 'date_job', 'text_job', 'status')
    inlines = (ImageInline,)
        
    def add_view(self, request, form_url='', extra_context=None):
        #self.exclude = None
        if not request.user.is_superuser:
            if request.user.groups.all()[0].id == 2:
                self.exclude=('date_job', 'text_job', 'vendors_response', 'solution', 'response_buyer', 'status', 'user', 'shop')
            elif request.user.groups.all()[0].id == 4:
                self.exclude=('date_job', 'text_job', 'vendors_response', 'solution', 'response_buyer', 'status', 'user')   
        return super(ClaimAdmin, self).add_view(request, form_url='', extra_context=None)             
    
    def prodavec_show(self, obj):
        records = User.objects.get(pk=obj.user_id)
        return u"%s %s" % (records.last_name,records.first_name)
    prodavec_show.short_description = u'Опубликовал'
    prodavec_show.allow_tags = True
    
    # сохранение
    def save_model(self, request, obj, form, change):
        if not obj.id: 
            if not request.user.is_superuser:        
                if request.user.groups.all()[0].id == 2:
                    obj.shop_id = request.user.usershops.shops.all()[0].id
                    obj.user_id = request.user.id
                if request.user.groups.all()[0].id == 4:
                    obj.user_id = request.user.id
        obj.save()
    
    def __init__(self, *args, **kwargs):
        super(ClaimAdmin, self).__init__(*args, **kwargs)
        self.exclude = None
    
# связываем эту модель с моделью MkAdmin
admin.site.register(Claim, ClaimAdmin)

[zkweb]

Roman Kitaev: Роман, что скажите? есть-ли ошибка в коде?

[zkweb]

Roman Kitaev: Роман, что скажите? есть-ли ошибка в коде?

[Roman K]

zkweb: Я не знаю, я же его не запускал и не тестировал.