DRY: affiche uniquement les enregistrements appartenant à l'utilisateur. Possible avec models.Manager?

Question

Je n'ai besoin de montrer à un utilisateur que les objets qu'il possède. Comme je dois le faire sur plus de 80% de mes vues, le codage en dur tue le DRY. Plus encore, il est absolument impératif qu'un utilisateur ne voit jamais les enregistrements appartenant à d'autres. Et le faire à la main (dans toutes les vues) semble également sujet à erreur.

J'ai regardé des décorateurs (comme login_required), mais cela semble au début du processus de gestion des demandes. Est-il possible d'obtenir la valeur request.user dans un gestionnaire écrit personnalisé et faire quelque chose comme ceci:

class CustomerManager(models.Manager): 
    def get_query_set(self): 
     return super(CustomerManager, self).get_query_set().filter(created_by=request.user) 

Ou est l'objet manager comme «no go! comme la définition du modèle en ce qui concerne les informations de demande?

Merci beaucoup.

---

Ceci est la partie d'enregistrement des objets, mais purement ici en tant qu'élaboration et pas une lecture nécessaire.

La partie de sauvegarde est relativement sécurisée. J'ai changé la fonction object.save() sur le modèle de sorte qu'il prend l'ID utilisateur en tant que parm.

def save(self, userid): 
    self.created_by = userid  
    super(Customer, self).save(userid) 

De l'avis:

if form.is_valid(): 
    customer = form.save(commit=False) 
    customer.save(request.user) 

De cette façon, je ne besoin d'avoir la ligne ci-dessous à mon avis avant customer.save...

customer.created_by = request.user 

Et ce qui rend moins sujette aux erreurs.

Answer 1

Je pense que le meilleur endroit pour cela peut être dans votre point de vue - même si je peux me tromper. Ajout de self.created_by = userid dans la méthode save() ne fait rien pour protéger les gens de en affichant l'information. Juste de le sauver.

Si vous utilisez un gestionnaire personnalisé pour un ensemble de requêtes par défaut, les 20% de vos vues n'auront pas de chance.

Je pense que vous avez besoin d'une vue - pourquoi ne pas créer votre propre vue générique une fois et ensuite la réutiliser?

def object_detail_user_only(request. queryset, **kwargs): 
    queryset = queryset.filter(created_by=request.user) 

    return list_detail.object_detail(request,queryset,**kwargs) 

Quelque chose comme ça (non testé)?

Answer 2

Pourquoi ne pas définir une méthode qui va chercher des documents pour l'utilisateur dans le CustomerManager:

class CustomerManager(models.Manager): 
    def get_user_records(self, user): 
     return self.filter(created_by=user) 

Je laisserais le get_query_set unoverridden que je vois aucun avantage pour l'emporter dans votre cas.