Rédaction d'un gestionnaire pour filtrer ensemble de résultats de requête

Question

Je le code suivant:

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() 

... et il fonctionne très bien. La seule chose est que je dois remplacer group='1' avec group=(the group specified by group = models.ForeignKey(Group)). Je passe un certain temps à essayer de déterminer si cette clé étrangère doit être transmise dans la classe, ou dans la fonction get_query_set, ou quoi. Je sais que vous pouvez accomplir ceci avec group.department_set.filter(group=desired group), mais j'écris ce modèle pour le site d'admin, ainsi j'ai besoin d'employer une variable et pas une constante après le signe =.

Answer 1

J'ai le pressentiment que le remplacement par défaut Manager sur objects de cette manière peut-être pas une bonne idée, surtout si vous prévoyez d'utiliser le site admin ... Même si elle vous aide avec vos employés, il a gagné » t vous aider du tout lors de la manipulation des départements. Que diriez-vous d'une deuxième propriété offrant une vue restreinte sur les départements à côté de l'habituel objects? Ou déplacez la norme Manager de objects vers _objects et renommez from_same_group en objects si vous préférez vraiment votre approche originale pour votre application.

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) 

Aussi, je comprends que vous savez comment configurer le site d'administration pour tirer profit du Manager drôle; si ce n'est pas le cas (ou si j'ai mal compris votre question), laissez un commentaire, j'essaierai de faire un suivi très bientôt.

---

EDIT: OK, pour le rendre plus clair: si vous tenez absolument à remplacer objects, vous auriez probablement envie de faire ceci:

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

Vous pourriez vouloir reconsidérer la relation entre les groupes et les départements si vous trouvez que tenter de créer un gestionnaire personnalisé est trop complexe. Les gestionnaires excellent dans la simplification des requêtes courantes, mais se trompent dans l'affichage des relations complexes entre les modèles et les instances de modèles.

Cependant, je pense que cet article à propos de filtering model objects with a custom manager vous pointera dans la bonne direction. L'auteur propose une technique pour effectuer un appel de fonction qui renvoie une classe de gestionnaire personnalisée dont les paramètres de filtre que vous spécifiez sont enregistrés dans la classe afin qu'ils ne soient pas transmis à l'instance. Fais le!