application avec workflow avec données de traitement (#76909) #787

Merged
fpeters merged 2 commits from wip/76909-applification-backoffice-field into main 2023-10-27 08:34:54 +02:00
Owner
No description provided.
fpeters added 2 commits 2023-10-26 17:32:45 +02:00
fpeters reviewed 2023-10-26 18:08:54 +02:00
@ -381,1 +381,4 @@
# process pre-import after jobs earlier, so there are no multiple jobs for
# the same object afterwards.
get_response().process_after_jobs()
Author
Owner

Le problème principal étant résolu (premier commit, voir commentaire ci-dessous sur wcs/middleware.py), il y avait un autre problème, on se trouvait avec deux afterjobs liés à la mise à jour du workflow (une fois pour le préimport, une seconde pour l'import réel), ça faisait qu'à l'exécution des afterjobs, le premier était exécuté sur base d'un workflow qui avait depuis été modifié. C'est contourné en faisant une passe sur les after jobs après la phase de pré-import.

Le problème principal étant résolu (premier commit, voir commentaire ci-dessous sur wcs/middleware.py), il y avait un autre problème, on se trouvait avec deux afterjobs liés à la mise à jour du workflow (une fois pour le préimport, une seconde pour l'import réel), ça faisait qu'à l'exécution des afterjobs, le premier était exécuté sur base d'un workflow qui avait depuis été modifié. C'est contourné en faisant une passe sur les after jobs après la phase de pré-import.
@ -120,3 +120,3 @@
def process_response(self, request, response):
if hasattr(response, 'after_jobs') and response.after_jobs:
if hasattr(request, 'quixote_request') and getattr(request.quixote_request.response, 'after_jobs'):
http_response = HTTPResponse()
Author
Owner

La principale part du problème était que les afterjobs n'étaient pas exécutés, parce qu'on était dans une vue native django, qu'on ne se trouvait du coup sans after_jobs ici (parce qu'ils y étaient posés via le code dans compat.py, supprimé plus haut). C'est donc modifié ici pour récupérer différemment les after_jobs.

La principale part du problème était que les afterjobs n'étaient pas exécutés, parce qu'on était dans une vue native django, qu'on ne se trouvait du coup sans after_jobs ici (parce qu'ils y étaient posés via le code dans compat.py, supprimé plus haut). C'est donc modifié ici pour récupérer différemment les after_jobs.
fpeters changed title from WIP: application avec workflow avec données de traitement (#76909) to application avec workflow avec données de traitement (#76909) 2023-10-26 18:09:00 +02:00
lguerin approved these changes 2023-10-26 21:02:26 +02:00
fpeters merged commit 92f9007506 into main 2023-10-27 08:34:54 +02:00
fpeters deleted branch wip/76909-applification-backoffice-field 2023-10-27 08:34:54 +02:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#787
No description provided.