История изменений
Исправление hippi90, (текущая версия) :
0. Подход вообще говно, я бы полностью переделал на нормальную ролевую модель, потому что текущий подход не расширяем. По факту сейчас есть только админ и обычный юзер, если нам завтра захочется добавить например модератора, или юзера с доп. правами, то этот код только выкинуть.
1. Оставаясь в рамках данного подхода, «роль» должна хранится в виде кода, который не меняется никогда, например 'ADMIN';
2. Выше уже правильно заметили, результат может быть пустым списком или None, это надо учитывать.
is_admin = perm is not None and 'ADMIN' in perm
Исходная версия hippi90, :
0. Подход вообще говно, я бы полностью переделал на нормальную ролевую модель, потому что текущий подход не расширяем. По факту сейчас есть только админ и обычный юзер, если нам завтра захочется добавить например модератора, или юзера с доп. правами, то этот код только выкинуть.
1. Оставаясь в рамках данного подхода, «роль» должна хранится в виде кода, который не меняется никогда, например 'ADMIN';
2. Выше уже правильно заметили, результат может быть пустым списком, это надо учитывать.
is_admin = 'ADMIN' in perm