entrouvert
/
passerelle
15
0
Fork 0
Code Issues Pull Requests 15 Releases Activity

toulouse_maelis: mettre à jour notification_date (#90074) #519

Open
nroche wants to merge 2 commits from wip/90074-updating-maelis-notification-date into main
pull from: wip/90074-updating-maelis-notification-date
merge into: entrouvert:main
Conversation 1 Commits 2 Files Changed 2 +73 -25
nroche commented 2024-04-26 12:23:29 +02:00
Owner
Copy Link

Patch un peu désespéré.
J'ajoute un test, mais qui ne permet pas de reproduire le bug.
Je tente d'enregistrer au plus tôt en BD,
ce qui sera sans effet s'il s'agit ici d'un problème de transaction
(cf https://dev.entrouvert.org/issues/85758#note-9 : peut-être un coup de defer qui est appliqué sur BaseCommand.execute).

Patch un peu désespéré. J'ajoute un test, mais qui ne permet pas de reproduire le bug. Je tente d'enregistrer au plus tôt en BD, ce qui sera sans effet s'il s'agit ici d'un problème de transaction (cf https://dev.entrouvert.org/issues/85758#note-9 : peut-être un coup de defer qui est appliqué sur BaseCommand.execute).
nroche added 2 commits 2024-04-26 12:23:29 +02:00
smihai reviewed 2024-04-26 12:38:43 +02:00
passerelle/contrib/toulouse_maelis/models.py Outdated
@ -4670,6 +4670,8 @@ class Invoice(models.Model):
)
return False
except SOAPError as e:
obj.maelis_notification_date = now()
smihai commented 2024-04-26 12:38:38 +02:00
Owner
Copy Link

C'est déjà fait juste après dans le code:

obj.maelis_notification_date = now()
obj.maelis_notification_data = result
obj.save()

C'est déjà fait juste après dans le code: `obj.maelis_notification_date = now()` `obj.maelis_notification_data = result` `obj.save()`
nroche commented 2024-05-02 10:22:15 +02:00
Author
Owner
Copy Link

Oui.
On reçoit le message dans les logs, mais la date n'est pas mise à jour.
C'est l'objet du ticket (redmine) : la date n'est pas positionnée bien qu'on passe dessus après dans le code.
J'avoue que ce patch est stupide, mais en enregistrant avant de loguer, je serais certain que le code est vraiment passé dessus.
Après oui, en vrai, on se doute déjà que c'est un problème de transaction ou associé.

Oui. On reçoit le message dans les logs, mais la date n'est pas mise à jour. C'est l'objet du ticket (redmine) : la date n'est pas positionnée bien qu'on passe dessus après dans le code. J'avoue que ce patch est stupide, mais en enregistrant avant de loguer, je serais certain que le code est vraiment passé dessus. Après oui, en vrai, on se doute déjà que c'est un problème de transaction ou associé.
bdauvergne changed title from wip/90074-updating-maelis-notification-date to toulouse_maelis: mettre à jour notification_date (#90074) 2024-04-30 04:06:52 +02:00
bdauvergne requested changes 2024-04-30 10:14:00 +02:00
bdauvergne left a comment
Owner
Copy Link

Ce n'est pas la bonne piste je pense, voir ticket redmine.

Ce n'est pas la bonne piste je pense, voir ticket redmine.
nroche force-pushed wip/90074-updating-maelis-notification-date from 21e4d8ad9d to 103ea773ea 2024-05-06 15:34:24 +02:00 Compare
nroche commented 2024-05-06 15:43:25 +02:00
Author
Owner
Copy Link

Il y a 4 appels à save() sur les factures et effectivement (indication donnée dans le ticket redmine) ceux de get_invoices ne sont pas protégés.
J'ai ajouté des verrous.

Il y a 4 appels à save() sur les factures et effectivement (indication donnée dans le ticket redmine) ceux de get_invoices ne sont pas protégés. J'ai ajouté des verrous.
nroche requested review from bdauvergne 2024-05-06 15:44:58 +02:00
All checks were successful
gitea/passerelle/pipeline/head This commit looks good
Details
This pull request can be merged automatically.
You are not authorized to merge this pull request.
Sign in to join this conversation.
Reviewers
No reviewers
bdauvergne
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 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/passerelle#519
No description provided.
Delete Branch "wip/90074-updating-maelis-notification-date"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?