Project

General

Profile

Bug #1216

L'élément de worflow "changer de statut après expiration" ne fonctionne pas

Added by Victor Claudet about 12 years ago. Updated about 12 years ago.

Status:
Fermé
Priority:
Haut
Assignee:
-
Category:
-
Target version:
Start date:
13 January 2012
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Planning:

Description

J'ai fais plusieurs tests et l'expiration ne semble pas être prise en compte.


Files

wcs.cron-ok.diff (9.07 KB) wcs.cron-ok.diff Thomas Noël, 17 January 2012 10:00 AM

Related issues

Related to w.c.s. - Bug #1218: gestion des urls front/backofficeFermé15 January 2012

Actions
Related to w.c.s. - Bug #1222: crash lors d'email dans un cronFermé16 January 2012

Actions

History

#1

Updated by Thomas Noël about 12 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 :
  • 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)
Pour que les URLs front et backoffice soient configurables :
  • 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
J'en ai profité pour ajouter des variables de substitution :
  • form_url_backoffice
  • site_url et site_url_backoffice
#2

Updated by Frédéric Péters about 12 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.".

#3

Updated by Thomas Noël about 12 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 !

#4

Updated by Thomas Noël about 12 years ago

Nouvelle version du patch ...

#5

Updated by Thomas Noël about 12 years ago

  • File deleted (wcs.cron-ok.diff)
#6

Updated by Thomas Noël about 12 years ago

  • Status changed from Solution déployée to Fermé

Also available in: Atom PDF