workflow: guarantee jump identifiers unicity on new workflows (#74132) #303
Loading…
Reference in New Issue
No description provided.
Delete Branch "wip/74132-jump-identifier-unicity"
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?
And on existing workflow with no existing identifier duplication.
WIP: workflow: guarantee jump identifiers unicity on new workflows (#74132)to workflow: guarantee jump identifiers unicity on new workflows (#74132)@ -1355,0 +1359,4 @@
identifier = getattr(item, 'identifier', None)
if identifier:
action_ids.append((item, identifier))
return action_ids
Comme ça permet de sortir d'une boucle sans passer sur toutes les valeurs, ça gagnerait à être un itérateur.
Voilà.
@ -2725,0 +2739,4 @@
if value == self.identifier:
# we don't want to block if a duplication of identifier already exists
return False
for jump, jump_identifier in self.parent.parent.get_identified_jumps():
Je voulais commenter le self.parent.parent que je trouve toujours peu lisible mais il n'y avait pas vraiemnt d'autre possibilité, j'ai fait #312 comme ça il pourra être écrit self.get_workflow().get_identified_jumps() et ça sera plus clair.
Voilà.
(j'aurais attendu le prochain cycle pour merger mais ok, je vais voir pour tagguer/déployer tout de suite).
J'ai hésité à pousser, merci pour le déploiement rapide.