Discussion :

[makayabou]

Bonjour,
je cherche à modifier mon fichier admin.py de mon application ordis:

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
from ordis.models import Ordi, Maintenance, OperatingSystem
from django.contrib import admin
 
#class MaintenanceAdmin(admin.ModelAdmin): 
    #list_display = (???) here I would like to see my Computer id, and the OS installed on it
 
def renvoi_os(Ordi):
    #return ("%d" % (Ordi.id)).upper()
    return ("%d %d" % (Ordi.id, Ordi.operatingsystemused)).upper()
class MaintenanceAdmin(admin.ModelAdmin):
    list_display = (renvoi_os,)
#class OrdiAdmin(admin.ModelAdmin):
 
 
admin.site.register(Ordi)
admin.site.register(Maintenance,MaintenanceAdmin)
admin.site.register(OperatingSystem)

Ce admin.py est associé à mon fichier models.py que voici:

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
from django.db import models
 
from django.db.models import (Model, BooleanField,
    CharField, DateTimeField, TextField, URLField,
    EmailField, ManyToManyField, ForeignKey, IntegerField,
    FileField, ImageField)
 
# Create your models here.
 
class OperatingSystem (models.Model):
    operatingsystem = CharField (max_length=30, blank=True, null=True)
class Ordi(models.Model):
    architecture = CharField (max_length=30, blank=True, null=True)
    operatingsystemused = ManyToManyField(OperatingSystem, null=True, blank=True)
class Maintenance(models.Model):
    ordi=ForeignKey(Ordi, blank=True, null=True)
    action = TextField(null=True, blank=True)

QUand je n'appelle que l'Ordi.id dans le admin.py (ligne commentée), j'obtiens bien le numéro de l'ordi correspondant dans Maintenance. En revanche, la ligne avec Ordi.id + Ordi.operatingsystemused me renvoie simplement (None).

Il y a quelquechose que je fais de travers??

Merci de votre aide.

Florian

[makayabou]

Bonjour,

Pour avancer sur mon appli, je commence par simplifier le problème...
Comment faire pour retourner un ManyToMany ,???

Le modèle suivant:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
class OperatingSystem (models.Model):
	operatingsystem = CharField (max_length=30, blank=True, null=True)
	def __unicode__(self):
		return "%s" % (self.operatingsystem)
class Ordi(models.Model):
	architecture = CharField (max_length=30, blank=True, null=True)
	operatingsystemused = ManyToManyField(OperatingSystem, null=True, blank=True)
 
	def __unicode__(self):
		oses_installed = u','.join(self.operationsystemused_set.all().values('operatingsystem',flat=True))
		return ("%s" % (oses_installed)).upper()

ou avec la dernière ligne return comme ça:

Code :

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

return models.join(list(self.operatingsystemused))

me renvoie (None) ....

Que faire???

Merci pour votre aide!

[makayabou]

Bonjour,
j'ai trouvé la solution à la version simplifiée du problème
Dans le models.py de Ordis, je sors:

Code :

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

1
2
3
	def __unicode__(self):
		oses_installed = u','.join(self.operatingsystemused.all().values_list('operatingsystem', flat=True))
		return ("%s" % (oses_installed))

Par contre ça ne fonctionne toujours pas dans le admin.py.
Le résultat des lignes suivantes est None:

Code :

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

1
2
3
4
5
6
def renvoi_os(maintenance):
    oses_installed = u','.join(maintenance.ordi.operationsystemused_set.all().values_list('operatingsystem',flat=True))
    return ("%s" % (oses_installed))
 
class MaintenanceAdmin(admin.ModelAdmin):
   list_display = (renvoi_os,)

Merci pour l'aide apportée.

[sjrd]

Euh tu te rends compte que

Code :

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

("%s" % (oses_installed))

c'est la même chose que

Code :

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

str(oses_installed)

et que tu ferais beaucoup mieux de faire simplement :

Code :

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

return oses_installed

Puis tu devrais utiliser ça comme list_display :

Code :

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

list_display = ('ordi_id', renvoi_os)

[makayabou]

merci pour les coups de main à l'écriture.

Par contre, le admin.py de l'appli ordis:

Code :

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

1
2
3
4
5
6
7
from ordis.models import Ordi, Maintenance, OperatingSystem
from django.contrib import admin
def renvoi_os(maintenance):
    oses_installed = u','.join(maintenance.ordi.operationsystemused_set.all().values_list('operatingsystem',flat=True))
    return oses_installed
class MaintenanceAdmin(admin.ModelAdmin):
   list_display = ('ordi_id', renvoi_os)

me renvoie toujours un beau:

Code :

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

MaintenanceAdmin.list_display[0], 'ordi_id' is not a callable or an attribute of 'MaintenanceAdmin' or found in the model 'Maintenance'.

avec une classe Mainteance dans l'appli ordis définie comme ça:

Code :

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

1
2
3
class Maintenance(models.Model):
	ordi=ForeignKey(Ordi, blank=True, null=True)
	action = TextField(null=True, blank=True)

[sjrd]

Ah oui tiens... Il ne veut prendre que les vrais champs. Bon ben ajoute une fonction en plus alors :

Code :

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

1
2
3
4
5
def get_ordi_id(maintenance):
    return maintenance.ordi_id
 
...
    list_display = (get_ordi_id, renvoi_os)

[makayabou]

Salut,
merci pour ta réponse.

L'ajout d'une fonction get_ordi_id me permet bien d'avoir l'id correspondant à l'ordi lié à la maintenance. Par contre, il ne m'affiche toujours pas les OS liés à l'ordi en question. Dans la colonne renvoi os il m'affiche (None).
Voici mon admin.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
from ordis.models import Ordi, Maintenance, OperatingSystem
from django.contrib import admin
 
def get_ordi_id(maintenance):
    return maintenance.ordi_id
 
def renvoi_os(maintenance):
    oses_installed = u','.join(maintenance.ordi.operationsystemused_set.all().values_list('operatingsystem',flat=True))
    return ("%s" % (oses_installed))
 
 
class MaintenanceAdmin(admin.ModelAdmin):
   list_display = (get_ordi_id, renvoi_os,) 
 
admin.site.register(Ordi)
admin.site.register(Maintenance,MaintenanceAdmin)
admin.site.register(OperatingSystem)

et 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
from django.db import models
 
from django.db.models import (Model, BooleanField,
    CharField, DateTimeField, TextField, URLField,
    EmailField, ManyToManyField, ForeignKey, IntegerField,
    FileField, ImageField)
 
# Create your models here.
 
class OperatingSystem (models.Model):
	operatingsystem = CharField (max_length=30, blank=True, null=True)
	def __unicode__(self):
		return "%s" % (self.operatingsystem)
 
class Ordi(models.Model):
	architecture = CharField (max_length=30, blank=True, null=True)
	operatingsystemused = models.ManyToManyField(OperatingSystem, null=True, blank=True)
 
 
class Maintenance(models.Model):
	ordi=ForeignKey(Ordi, blank=True, null=True)
	action = TextField(null=True, blank=True)

merci de ton aide.

[makayabou]

Si j'essaie d'avoir l'architecture du PC associé à la maintenance, j'ai aussi (None) qui est affiché:

Code :

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

1
2
3
4
5
def get_ordi_architecture(maintenance):
	return maintenance.ordi_architecture
 
class MaintenanceAdmin(admin.ModelAdmin):
   list_display = (get_ordi_id, renvoi_os, get_ordi_architecture)

Si cela peut t'aider à m'aider... merci!

[sjrd]

Et si tu fais :

Code :

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

1
2
def get_ordi_architecture(maintenance):
    return maintenance.ordi.architecture

(donc avec un . et non un _)
?

Tu dois aussi arrêter d'écrire "%s" % une_chaine. Tu dois juste mettre une une_chaine à ce moment. Et ce partout, dans tous tes programmes Python.

[makayabou]

Salut,

ça marche impec pour architecture.
Par contre j'ai toujours None pour le champ renvoi_os.....

J'ai enlevé mes %s inutiles. Ne t'inquiètes pas, je l'avais laissé dans ce fichier, mais sur l'application en production j'avais écouté tes conseils!

[makayabou]

J'ai tenté de définir la variable os_installed dans mon model.py pour l'appeler plus facilement à partir de admin.py.

Voici mon nouveau 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
24
25
from django.db import models
 
from django.db.models import (Model, BooleanField,
    CharField, DateTimeField, TextField, URLField,
    EmailField, ManyToManyField, ForeignKey, IntegerField,
    FileField, ImageField)
 
# Create your models here.
 
class OperatingSystem (models.Model):
	operatingsystem = CharField (max_length=30, blank=True, null=True)
	def __unicode__(self):
		return self.operatingsystem
 
class Ordi(models.Model):
	architecture = CharField (max_length=30, blank=True, null=True)
	operatingsystemused = models.ManyToManyField(OperatingSystem, null=True, blank=True)
	oses_installed = u','.join(operatingsystemused().values_list('operatingsystem',flat=True))
	def __unicode__(self):
		return "%d" % (self.id)
 
 
class Maintenance(models.Model):
	ordi=ForeignKey(Ordi, blank=True, null=True)
	action = TextField(null=True, blank=True)

la console me renvoie:

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
makayabou@makayabou-testing:~/dev/essai3$ python manage.py runserver
Validating models...
Unhandled exception in thread started by <function inner_run at 0x91bd684>
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/django/core/management/commands/runserver.py", line 48, in inner_run
    self.validate(display_num_errors=True)
  File "/usr/lib/pymodules/python2.6/django/core/management/base.py", line 249, in validate
    num_errors = get_validation_errors(s, app)
  File "/usr/lib/pymodules/python2.6/django/core/management/validation.py", line 28, in get_validation_errors
    for (app_name, error) in get_app_errors().items():
  File "/usr/lib/pymodules/python2.6/django/db/models/loading.py", line 146, in get_app_errors
    self._populate()
  File "/usr/lib/pymodules/python2.6/django/db/models/loading.py", line 61, in _populate
    self.load_app(app_name, True)
  File "/usr/lib/pymodules/python2.6/django/db/models/loading.py", line 78, in load_app
    models = import_module('.models', app_name)
  File "/usr/lib/pymodules/python2.6/django/utils/importlib.py", line 35, in import_module
    __import__(name)
  File "/home/makayabou/dev/essai3/ordis/models.py", line 15, in <module>
    class Ordi(models.Model):
  File "/home/makayabou/dev/essai3/ordis/models.py", line 18, in Ordi
    oses_installed = u','.join(operatingsystemused().values_list('operatingsystem',flat=True))
TypeError: 'ManyToManyField' object is not callable

Si je met:

Code :

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

	oses_installed = u','.join(operatingsystemused_set.all().values_list('operatingsystem',flat=True))

,
la console me renvoie:

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
 
makayabou@makayabou-testing:~/dev/essai3$ python manage.py runserver
Validating models...
Unhandled exception in thread started by <function inner_run at 0x9e2b684>
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/django/core/management/commands/runserver.py", line 48, in inner_run
    self.validate(display_num_errors=True)
  File "/usr/lib/pymodules/python2.6/django/core/management/base.py", line 249, in validate
    num_errors = get_validation_errors(s, app)
  File "/usr/lib/pymodules/python2.6/django/core/management/validation.py", line 28, in get_validation_errors
    for (app_name, error) in get_app_errors().items():
  File "/usr/lib/pymodules/python2.6/django/db/models/loading.py", line 146, in get_app_errors
    self._populate()
  File "/usr/lib/pymodules/python2.6/django/db/models/loading.py", line 61, in _populate
    self.load_app(app_name, True)
  File "/usr/lib/pymodules/python2.6/django/db/models/loading.py", line 78, in load_app
    models = import_module('.models', app_name)
  File "/usr/lib/pymodules/python2.6/django/utils/importlib.py", line 35, in import_module
    __import__(name)
  File "/home/makayabou/dev/essai3/ordis/models.py", line 15, in <module>
    class Ordi(models.Model):
  File "/home/makayabou/dev/essai3/ordis/models.py", line 18, in Ordi
    oses_installed = u','.join(operatingsystemused_set.all().values_list('operatingsystem',flat=True))
NameError: name 'operatingsystemused_set' is not defined

Au passage, comment écrire le

Code :

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

1
2
	def __unicode__(self):
		return "%d" % (self.id)

plus proprement?

merci beaucoup pour ton aide, j'ai mis le fil aussi sur django.fr et la liste de diffusion django et t'es le seul à me répondre!

[sjrd]

Voici mon nouveau models.py:

Code :

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

1
2
3
4
5
6
class Ordi(models.Model):
    architecture = CharField (max_length=30, blank=True, null=True)
    operatingsystemused = models.ManyToManyField(OperatingSystem, null=True, blank=True)
    oses_installed = u','.join(operatingsystemused().values_list('operatingsystem',flat=True))
    def __unicode__(self):
        return "%d" % (self.id)

Nan mais ça ça peut juste pas marcher, hein Tu dois en faire une méthode, dans ce cas :

Code :

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

1
2
3
4
5
6
7
class Ordi(models.Model):
    architecture = CharField (max_length=30, blank=True, null=True)
    operatingsystemused = models.ManyToManyField(OperatingSystem, null=True, blank=True)
 
    def oses_installed(self):
        return u','.join(self.operatingsystemused.values_list(
            'operatingsystem', flat=True))

Et pour ton autre question :

Code :

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

1
2
    def __unicode__(self):
        return unicode(self.id)

[makayabou]

Salut,
quand je met dans models.py:

Code :

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

1
2
    def oses_installed(self):
		return u','.join(self.operatingsystemused.values_list('operatingsystem', flat=True))

la console me répond

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
Django version 1.2.3, using settings 'essai3.settings'
Development server is running at http://127.0.0.1:8000/
Quit the server with CONTROL-C.
Validating models...
Unhandled exception in thread started by <function inner_run at 0x8b5f684>
Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/django/core/management/commands/runserver.py", line 48, in inner_run
    self.validate(display_num_errors=True)
  File "/usr/lib/pymodules/python2.6/django/core/management/base.py", line 249, in validate
    num_errors = get_validation_errors(s, app)
  File "/usr/lib/pymodules/python2.6/django/core/management/validation.py", line 28, in get_validation_errors
    for (app_name, error) in get_app_errors().items():
  File "/usr/lib/pymodules/python2.6/django/db/models/loading.py", line 146, in get_app_errors
    self._populate()
  File "/usr/lib/pymodules/python2.6/django/db/models/loading.py", line 61, in _populate
    self.load_app(app_name, True)
  File "/usr/lib/pymodules/python2.6/django/db/models/loading.py", line 78, in load_app
    models = import_module('.models', app_name)
  File "/usr/lib/pymodules/python2.6/django/utils/importlib.py", line 35, in import_module
    __import__(name)
  File "/home/makayabou/dev/essai3/ordis/models.py", line 32
    def oses_installed(self):
                            ^
IndentationError: unindent does not match any outer indentation level

Par contre, si je l'écris de cette manière:

Code :

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

1
2
3
	def renvoios(self):
		oses_installed = u','.join(self.operatingsystemused.values_list('operatingsystem',flat=True))
		return ("%s" % (oses_installed))

pas de problème.

Reste ensuite à l'appeler dans admin.py..
Je met:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
def get_ordi_id(maintenance):
    return maintenance.ordi_id
 
def oses(maintenance):
	return maintenance.ordi.renvoios()
 
def get_ordi_architecture(maintenance):
	return maintenance.ordi.architecture
 
class MaintenanceAdmin(admin.ModelAdmin):
   list_display = (get_ordi_id, oses, get_ordi_architecture, 'action')

ET C'EST GAGNÉ!!!!!!

Un grand grand merci à toi!

J'attends ta réponse et peut-être quelques dernières retouches syntaxiques pour mettre Résolu!!

[sjrd]

la console me répond

c'est parce que tu mixes les tab et les 4 espaces n'importe comment. En Python c'est un gros problème de faire ça.

Tu dois configurer ton éditeur, quel qu'il soit, pour transformer Tab en 4 espaces ou inversement.

[makayabou]

Salut,

tu m'avais bien dépanné pour ma découverte de django.
Grâce à toi j'ai pu finaliser une première version de mon application, qui me permet de gérer un stock d'ordis avec les utilisateurs, pour l'association dans laquelle je suis bénévole.
Si t'as un peu de temps pour jeter un oeil à ça, je me permet de t'envoyer l'application dans une archive, pour que tu puisse me corriger, notamment sur les problèmes de syntaxe.
Par ailleurs, deux problèmes se posent encore à moi:
- comment prédéfinir la taille des colonnes de ma vue admin.
- comment définir des 'choix liés'. ex: si le boolean SATA est true, alors seulement j'ai la possibilité de remplir les champs Sata1,2,3,4...


Si t'as pas le temps pour ça, pas de problème, je comprendrai...

En tout cas merci beaucoup à toi.

Florian