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

toulouse-maelis: do not crash on cron when family is removed (#78929) #294

Merged
nroche merged 1 commits from wip/78929-parsifal-basket-on-removed-family into main 2023-06-27 18:13:26 +02:00
Conversation 0 Commits 1 Files Changed 3 +81 -9
nroche commented 2023-06-23 10:51:14 +02:00
Owner
Copy Link

Quand la famille n'existe plus, renvoyer un panier vide.

Je n'ai pas ajouté de test qui reproduit le bug sentry en démarrant du cron,
parce que le correctif renvoie un panier vide si la famille n'existe plus,
et que le test test_trigger_wcs_on_removed_subscriptions_cron traite déjà le cas d'un panier vide.

Quand la famille n'existe plus, renvoyer un panier vide. Je n'ai pas ajouté de test qui reproduit le bug sentry en démarrant du cron, parce que le correctif renvoie un panier vide si la famille n'existe plus, et que le test test_trigger_wcs_on_removed_subscriptions_cron traite déjà le cas d'un panier vide.
tnoel requested changes 2023-06-26 11:46:22 +02:00
passerelle/contrib/toulouse_maelis/models.py Outdated
@ -807,2 +810,2 @@
or []
)
except SOAPFault as e:
if 'E02' in e.data['soap_fault'].get('message'):
tnoel commented 2023-06-26 11:46:17 +02:00
Owner
Copy Link

Le get('message') pourrait renvoyer un None, et boum, le "in" va planter. Aussi, le « E02 » doit être au début du message (on ne sait jamais).

J'écrirais plutôt :

fault_message = e.data['soap_fault'].get('message') or ''
if fault_message.startswith(''E02 '):
....

Le get('message') pourrait renvoyer un None, et boum, le "in" va planter. Aussi, le « E02 » doit être au début du message (on ne sait jamais). J'écrirais plutôt : fault_message = e.data['soap_fault'].get('message') or '' if fault_message.startswith(''E02 '): ....
nroche commented 2023-06-26 12:05:29 +02:00
Author
Owner
Copy Link

Oui, j'intègre ça, merci.

Oui, j'intègre ça, merci.
nroche requested review from tnoel 2023-06-26 15:33:10 +02:00
tnoel approved these changes 2023-06-27 15:46:06 +02:00
passerelle/contrib/toulouse_maelis/models.py Outdated
@ -808,1 +810,3 @@
)
except SOAPFault as e:
fault_message = e.data['soap_fault'].get('message') or ''
if fault_message.startswith('E02 : '):
tnoel commented 2023-06-27 15:46:01 +02:00
Owner
Copy Link

à rerelire on aurait pu faire un

if 'ne correspond à aucune famille' in fault_message:

mais c'est très bien aussi avec E02, on verra si "ça tient" la distance.

à rerelire on aurait pu faire un if 'ne correspond à aucune famille' in fault_message: mais c'est très bien aussi avec E02, on verra si "ça tient" la distance.
nroche force-pushed wip/78929-parsifal-basket-on-removed-family from bc9d0a8bdc to 1625527646 2023-06-27 16:56:31 +02:00 Compare
nroche force-pushed wip/78929-parsifal-basket-on-removed-family from 1625527646 to 262340b6a4 2023-06-27 17:30:36 +02:00 Compare
nroche force-pushed wip/78929-parsifal-basket-on-removed-family from 262340b6a4 to 0b8730b9ba 2023-06-27 17:58:07 +02:00 Compare
nroche merged commit 0b8730b9ba into main 2023-06-27 18:13:26 +02:00
nroche deleted branch wip/78929-parsifal-basket-on-removed-family 2023-06-27 18:13:26 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
tnoel
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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/passerelle#294
No description provided.
Delete Branch "wip/78929-parsifal-basket-on-removed-family"

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?