Discussion :

[GrimaudClement]

Bonjour.

En tant que débutant, je vous demande un peu de compréhension si ma question et mes erreurs sont simplistes...

Je suis confronté à un problème. Je souhaiterai tester si un champs d'un models est Vrai ou Faux dans le views.py

Voici les class concernés dans mon models.py:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
class Communaute(models.Model):
    nom=models.CharField(max_length=50, verbose_name="Nom de la communauté", unique=True)   #nom de la communauté
    description=models.TextField(blank=True, verbose_name="Description")
    TYPE_COMMUNAUTE=(
        ('PUBLIQUE','Publique'),
        ('PRIVEE','Privée'),
    )
    type_communaute=models.CharField(max_length=50, verbose_name="Type de la communauté", choices=TYPE_COMMUNAUTE)
    inscriptionauto=models.BooleanField(default=False, verbose_name="Inscription automatique")
    membres=models.ManyToManyField(User, through='Rejoindre', verbose_name="Communaute", null=True, blank=True)
    def __unicode__(self):
        return self.nom
 
 
class Rejoindre(models.Model):
    createur = models.BooleanField(default=False, verbose_name="Créateur de la communaute")
    date = models.DateTimeField(auto_now_add=True, auto_now=False,verbose_name="Rejoins depuis le:")
    user = models.ForeignKey(User)
    communaute=models.ForeignKey(Communaute)
    postuler=models.BooleanField(default=False, verbose_name="La personne a t'elle postulée")
    refuser=models.BooleanField(default=False, verbose_name="La personne est elle refusée")
    def __unicode__(self):
        return "Communauté: {0}".format(self.communaute.nom)

Et voici mon fichier views.py (Attention, il n'est pas complet !)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
def details_communaute(request, id):
 
    try:
        communaute=Communaute.objects.get(id=id)
    except Communaute.DoesNotExist:
        raise Http404
 
    messages=MessagePublic.objects.filter(communaute=communaute).order_by('-date')
 
 
    masquer=True
    inscrire=True
 
    if communaute.type_communaute=="PUBLIQUE":
        masquer=False
        for membre in communaute.membres.all():
            if membre==request.user:
                inscrire=False
    else:
         for membre in communaute.membres.all():
            if membre==request.user:
                rejoindre=Rejoindre.objects.filter(user=membre,communaute=communaute)
                if rejoindre.refuser==True :
                    inscrire=False
                    masquer=True
                else:
                    masquer=False
                    inscire=False

J'ai donc le message d'erreur suivant:

AttributeError at /communaute/num-4/
'QuerySet' object has no attribute 'refuser'
Request Method: GET
Request URL: http://localhost:8000/communaute/num-4/
Django Version: 1.5.4
Exception Type: AttributeError
Exception Value:
'QuerySet' object has no attribute 'refuser'
Python Version: 3.3.2

Après plusieurs tentatives en bidouillant le code, je n'arrive pas à régler le problème.

Ma question est donc plutôt simple, comment tester si le champs Rejoindre.refuser pour l'utilisateur connecté et la communauté sélectionnée est "vrai"?

Merci d'avance

[Drawingrom]

Bonjour,

Rejoindre.objects.filter renvoie un QuerySet (grosso modo, une liste) et pas une instance de Rejoindre.

Essayez de prendre le premier élément du QuerySet :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
rejoindre=Rejoindre.objects.filter(user=membre,communaute=communaute)[0]
    if rejoindre.refuser==True :
        inscrire=False
        masquer=True

Remarque : appeler une classe de modèle en utilisant un verbe, c'est un peu bizarre. C'est plutôt un journal d'inscription non ? Si c'est le cas pourquoi ne pas l'appeler JournalInscription ?

[GrimaudClement]

Merci pour cette solution, cela fonctionne parfaitement.
Si mon champs JournalInscription.createur avait été en seconde position dans mon models, j'aurais du écrire cela ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
rejoindre=Rejoindre.objects.filter(user=membre,communaute=communaute)[1]
    if rejoindre.refuser==True :
        inscrire=False
        masquer=True

Comme tu le vois, j'ai renommé ma class "Rejoindre", j'ai des difficultés à trouver des nominatifs pour mes bases de données

Merci encore pour ta réponse

[Drawingrom]

Bonjour,

Je crois que vous n'avez pas compris. On attend de la méthode filter qu'elle renvoie plusieurs enregistrements (ou ligne si vous préférez). Elle renverra donc toujours une liste.

Chaque enregistrement contient les champs définis dans votre modèle.

Si vous voulez accéder au champ créateur du premier enregistrement renvoyé, vous pouvez faire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Rejoindre.objects.filter(user=membre,communaute=communaute)[0].createur

[GrimaudClement]

Bonjour.

Il est vraie que je n'avais pas complètement compris le fonctionnement de filter, maintenant c'est beaucoup plus clair.

Je te remercie de ta réponse.

Cependant, sur un autre forum une personne m'a expliqué que cela était plus du bricolage... on m'a donné une autre méthode utilisant le get et cela fonctionne pour le moment. Je vais peut-être trouver un cas particulier qui va faire que j'aurais un bug...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

rejoindre=Rejoindre.objects.get(user=membre,communaute=communaute)

[wiztricks]

Cependant, sur un autre forum une personne m'a expliqué que cela était plus du bricolage...

Et cette personne a raison car .filter dans ce cas "fonctionne" mais n'est pas cohérent avec ce que vous avez écrit par ailleurs.
Rejoindre est associe a une "association" qui est une table dans laquelle les colonnes (communaute, membre) seront la clef qui identifie les entrées.
Si c'est une "cle" - on dira "candidate" - il y aura 0 ou 1 enregistrement associes. Si c'est autre chose, il pourra y en avoir 0 ou N.
Subtil ? En fait, ce sera le même select qui sera expédie a la BDD.
Mais ça change tout cote API: le .get retourne quelque chose ou levera une exception alors que .filter pourra retourner une liste de... éventuellement vide.

Le .get sera cohérent avec la nature de la table.
Utiliser .filter "fonctionne" mais montre que vous n’êtes pas très clair...
Normal: vous débutez.

- W