utiliser return upper dans admin.py

Version imprimable

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

Code:

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:

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

simplifier le problème?

Bonjour,

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

Le modèle suivant:
Code:

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:

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

me renvoie (None) ....

Que faire???

Merci pour votre aide!

réponse partielle

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

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:

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.

03/02/2011, 15h31
sjrd

Euh tu te rends compte que

Code:

("%s" % (oses_installed))

c'est la même chose que

Code:

str(oses_installed)

et que tu ferais beaucoup mieux de faire simplement :

Code:

return oses_installed

Puis tu devrais utiliser ça comme list_display :

Code:

list_display = ('ordi_id', renvoi_os)

apprendre l'orthographe!

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

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

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:

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:

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

04/02/2011, 12h48
sjrd
Ah oui tiens... Il ne veut prendre que les vrais champs. Bon ben ajoute une fonction en plus alors :
Code:

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

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:

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:

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.

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

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!

08/02/2011, 11h53
sjrd
Et si tu fais :
Code:

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.
08/02/2011, 13h50
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!

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:

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:

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:

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

,
la console me renvoie:

Code:

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:

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!

Citation:

Envoyé par makayabou

Voici mon nouveau models.py:

Code:

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:

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:

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

Salut,
quand je met dans models.py:

Code:

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

la console me répond

Code:

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:

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:

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É!!!!!! :ccool::lol::ccool::lol::ccool::lol:

Un grand grand merci à toi!

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

10/02/2011, 13h36
sjrd

Citation:

Envoyé par makayabou

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.
21/02/2011, 19h11
makayabou

1 pièce(s) jointe(s)

résultat de ton aide: une belle première appli sous Django!

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