Development #12758
pouvoir préciser une régie par défaut
0%
Description
if request.GET.get('regie_id'):
item.regie = Regie.objects.get(id=request.GET.get('regie_id'))
else:
# if there's no regie specified, use the first one we get from the
# database...
item.regie = Regie.objects.all()[0]
Pour le moment sortir au petit bonheur la chance le premier résultat envoyé par la db, ce n'est vraiment pas terrible.
On pourrait ajouter .order_by('id')[0] pour au moins avoir un truc stable mais le mieux ce serait d'ajouter la possibilité de définir quelle est la régie par défaut (en ajoutant un attribut booléen supplémentaire au modèle).
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 7 ans
- Fichier 0001-lingo-make-it-possible-to-mark-a-regie-as-default-12.patch 0001-lingo-make-it-possible-to-mark-a-regie-as-default-12.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
J'avais aussi une version avec en plus un RunSQL UPDATE lingo_regie SET is_default=TRUE WHERE id = 1 un peu naïf mais j'ai au final préféré garder le fallback actuel, aussi peu terrible soit-il.
Mis à jour par Thomas Noël il y a plus de 7 ans
sans le « .order_by('id') » ? histoire d'avoir moins peur des aléas
Mis à jour par Frédéric Péters il y a plus de 7 ans
sans le « .order_by('id') » ? histoire d'avoir moins peur des aléas
C'est de toute façon une mauvaise pratique mais le résultat est désormais stable via le ordering = ['label'] posé sur la classe.
Mis à jour par Frédéric Péters il y a plus de 7 ans
- Statut changé de En cours à Résolu (à déployer)
commit 699e24f1f4149bf05e1db3ab42f0318789f2390b
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Sep 27 17:06:17 2016 +0200
lingo: make it possible to mark a regie as default (#12758)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
lingo: make it possible to mark a regie as default (#12758)