Bug #52793
plus d'exécution du job d'expiration de saut
Début:
06 avril 2021
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Il n'est plus enregistré dans les jobs; ça vient d'une modification de l'ordre des imports (bissecté sur 04f68506dc56e7370b75416cc78f5f478999719d).
C'est compliqué à suivre mais je dirais que ça vient de la partie sur wcs/publisher.py, qui déplace des imports qui avaient été mochement placés derrière le StubWcsPublisher.
# this is terribly ugly but import RootDirectory will import a bunch of things, # and some of them need a publisher to be set
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 3 ans
- Fichier 0001-general-add-explicit-registration-of-cron-jobs-52793.patch 0001-general-add-explicit-registration-of-cron-jobs-52793.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
--- a/wcs/publisher.py
+++ b/wcs/publisher.py
@@ -34,7 +34,6 @@ from .Defaults import * # noqa pylint: disable=wildcard-import
from .qommon.cron import CronJob
from .qommon.publisher import QommonPublisher, get_request, set_publisher_class
from .roles import Role
-from .root import RootDirectory
from .tracking_code import TrackingCode
from .users import User
@@ -53,6 +52,9 @@ class StubWcsPublisher(QommonPublisher):
set_publisher_class(StubWcsPublisher)
+from .root import RootDirectory # noqa pylint: disable=wrong-import-position
+
+
class UnpicklerClass(pickle.Unpickler):
def find_class(self, module, name):
if module == 'qommon.form':
assure le job mais c'est juste remettre le truc moche.
Sans aller jusqu'au nettoyage ultime qui serait le merge wcs/qommon/publisher.py et wcs/publisher.py, voici un patch qui ajouter des méthodes explicites aux différents endroits concernés, et ajoute les appels à celles-ci. Et supprime donc le truc du StubWcsPublisher.
Mis à jour par Frédéric Péters il y a environ 3 ans
- Priorité changé de Normal à Haut
(relecture critique avant la mise en prod)
Mis à jour par Thomas Noël il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit a407b21008439b74c805c902fd9fd9bef92b0047
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Tue Apr 6 19:03:44 2021 +0200
general: add explicit registration of cron jobs (#52793)
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
general: add explicit registration of cron jobs (#52793)