Написание менеджера для фильтрации набора результатов запроса

Question

У меня есть следующий код:

class GroupDepartmentManager(models.Manager): 
    def get_query_set(self): 
    return super(GroupDepartmentManager, self).get_query_set().filter(group='1') 

class Department(models.Model): 
name = models.CharField(max_length=128) 
group = models.ForeignKey(Group) 
def __str__(self): 
    return self.name 
objects = GroupDepartmentManager() 

... и она отлично работает. Единственное, что мне нужно заменить group='1' на group=(the group specified by group = models.ForeignKey(Group)). У меня довольно много времени, пытаясь определить, должен ли этот внешний ключ быть передан в класс, или в функцию get_query_set, или что. Я знаю, что вы можете выполнить это с помощью group.department_set.filter(group=desired group), но я пишу эту модель для сайта администратора, поэтому мне нужно использовать переменную, а не константу после знака =.

Answer 1

У меня есть догадка, что замена по умолчанию Manager на objects таким образом может быть неправильной идеей, особенно если вы планируете использовать админ-сайт ... Даже если это поможет вам с вашими сотрудниками, t помочь вам при работе с отделами. Как насчет второго имущества, предоставляющего ограниченное представление о департаментах наряду с обычным objects? Или переместите стандарт Manager с objects в _objects и переименуйте from_same_group в objects, если вы действительно любите свой оригинальный подход для своего приложения.

class Department(models.Model): 
    name = models.CharField(max_length=128) 
    group = models.ForeignKey(Group) 
    def __str__(self): 
     return self.name 
    objects = models.Manager() 

    @property 
    def from_same_group(self): 
     return Department.objects.filter(group__exact=self.group) 

Кроме того, я понимаю, что вы знаете, как настроить сайт администратора, чтобы воспользоваться смешным Manager; если нет (или если я неправильно понял ваш вопрос как-то), оставьте комментарий, я постараюсь последовать за ним в ближайшее время.

---

EDIT: OK, чтобы сделать это более ясно: если вы абсолютно настаивают на замене objects, вы, вероятно, хотите сделать это:

class Department(models.Model): 
    name = models.CharField(max_length=128) 
    group = models.ForeignKey(Group) 
    def __str__(self): 
     return self.name 

    _objects = models.Manager() 

    @property 
    def objects(self): 
     # note the _objects in the next line 
     return Department._objects.filter(group__exact=self.group) 

Answer 2

Вы можете пересмотреть отношения между группами и отделами, если вы обнаружите, что попытка создания настраиваемого менеджера слишком сложна. Менеджеры преуспевают в упрощении общих запросов, но на флопе отображаются сложные отношения между моделями и экземплярами моделей.

Однако, думаю, эта статья о filtering model objects with a custom manager укажет вам в правильном направлении. Автор предлагает метод выполнения вызова функции, который возвращает настраиваемый класс менеджера, который имеет параметры фильтра, указанные вами в классе, поэтому они не передаются экземпляру. Сделай это!