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.
Files
Related issues
History
Updated by Thomas Noël over 11 years ago
- File wcs.cron-ok.diff added
- Status changed from Nouveau to 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
Updated by Frédéric Péters over 11 years ago
'<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.".
Updated by Thomas Noël over 11 years ago
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 !
Updated by Thomas Noël over 11 years ago
- File wcs.cron-ok.diff wcs.cron-ok.diff added
Nouvelle version du patch ...