Bug #1216
L'élément de worflow "changer de statut après expiration" ne fonctionne pas
0%
Description
J'ai fais plusieurs tests et l'expiration ne semble pas être prise en compte.
Fichiers
Demandes liées
Historique
Mis à jour par Thomas Noël il y a plus de 12 ans
- Fichier wcs.cron-ok.diff ajouté
- Statut changé de Nouveau à Solution déployée
Voici une résolution de ce problème... assez large !
- qommon/cron.py gère mieux le publisher crée : il en charge la config, la langue, initialise les var de subst
- en mode cron, on n'a ni request ni response, donc :
Pour que les URLs front et backoffice soient configurables :
- les URL sont calculées à partir du nom du répertoire du publisher (voir aussi ci-dessous)
- les emails envoyés le sont en mode direct (sans fire_and_forget)
- la langue est celle du site ou None (puisque pas de négociation possible)
- ajout de deux variables de config misc/frontoffice-url et misc/backoffice-url
- utilisation de ces variables si elles existent à la place de tout autre algo
- form_url_backoffice
- site_url et site_url_backoffice
Mis à jour par Frédéric Péters il y a plus de 12 ans
'<dt><a href="urls">%s</a></dt> <dd>%s</dd>' % (
_('URLs'), _('Configure base URLs'))
Ça pourrait pas être casé dans un autre écran ?; par exemple l'écran "Nom du site" n"a pour le moment qu'une variable, on pourrait le renommer en "nom et adresses du site" et y avoir plusieurs options.
if accepted_languages:
Je suis plutôt favorable à une sortie expresse des fonctions, quand la condition n'est pas remplie, plutôt qu'ajouter un niveau d'indendation.
if fire_and_forget and get_request() is None:
fire_and_forget = False
Cette partie je la mettrais dans un patch séparé, et j'y ajouterais un commentaire, genre: "we are not processing a request, no sense to delay the handling.".
Mis à jour par Thomas Noël il y a plus de 12 ans
Frédéric Péters a écrit :
[...]
Ça pourrait pas être casé dans un autre écran ?; par exemple l'écran "Nom du site" n"a pour le moment qu'une variable, on pourrait le renommer en "nom et adresses du site" et y avoir plusieurs options.
J'avais fait ça au début puis je me suis dit "Fred il voudra jamais mettre cet aspect technique dans la colonne Personnalisation", comme quoi, je suis trop torturé. Je refais le patch avec "Nom et adresse du site" illico.
[...]
Je suis plutôt favorable à une sortie expresse des fonctions, quand la condition n'est pas remplie, plutôt qu'ajouter un niveau d'indendation.
Ack.
[...]
Cette partie je la mettrais dans un patch séparé, et j'y ajouterais un commentaire, genre: "we are not processing a request, no sense to delay the handling.".
Ack.
Merci Fred !
Mis à jour par Thomas Noël il y a plus de 12 ans
- Fichier wcs.cron-ok.diff wcs.cron-ok.diff ajouté
Nouvelle version du patch ...