entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 31 Releases Activity

misc: make workflow file options importable (#58170) #594

Merged
fpeters merged 1 commits from wip/58170-workflow-file-option-import into main 2023-08-16 19:57:46 +02:00
Conversation 2 Commits 1 Files Changed 2 +16 -13
fpeters commented 2023-08-13 11:57:45 +02:00
Owner
Copy Link
No description provided.
fpeters force-pushed wip/58170-workflow-file-option-import from 016aac1049 to 8da0d89049 2023-08-13 12:22:34 +02:00 Compare
fpeters changed title from WIP: misc: make workflow file options importable (#58170) to misc: make workflow file options importable (#58170) 2023-08-13 13:31:42 +02:00
pmarillonnet approved these changes 2023-08-16 16:47:36 +02:00
pmarillonnet left a comment
Owner
Copy Link

Bug marrant, j’arrive pas à retrouver dans le code initial le parsing bogué qui fait que c’est le contenu brut entre les deux balises ouvrantes qui est importé, mais ok, ack.

Bug marrant, j’arrive pas à retrouver dans le code initial le parsing bogué qui fait que c’est le contenu brut entre les deux balises ouvrantes qui est importé, mais ok, ack.
fpeters commented 2023-08-16 19:57:42 +02:00
Author
Owner
Copy Link

Bug marrant, j’arrive pas à retrouver dans le code initial le parsing bogué qui fait que c’est le contenu brut entre les deux balises ouvrantes qui est importé, mais ok, ack.

C'est déplacer le code gérant les fichiers au-dessus de :

            elif option.text:
                option_value = xml_node_text(option)

qui fait la correction.

À noter que dans les tests ça ne se voit pas parce qu'ils ne se font pas avec de l'XML indenté, et donc pas de texte "espace" entre les balises.

> Bug marrant, j’arrive pas à retrouver dans le code initial le parsing bogué qui fait que c’est le contenu brut entre les deux balises ouvrantes qui est importé, mais ok, ack. C'est déplacer le code gérant les fichiers au-dessus de : ``` elif option.text: option_value = xml_node_text(option) ``` qui fait la correction. À noter que dans les tests ça ne se voit pas parce qu'ils ne se font pas avec de l'XML indenté, et donc pas de texte "espace" entre les balises.
fpeters merged commit 2dc84728eb into main 2023-08-16 19:57:46 +02:00
fpeters deleted branch wip/58170-workflow-file-option-import 2023-08-16 19:57:46 +02:00
pmarillonnet commented 2023-08-16 20:19:42 +02:00
Owner
Copy Link

Bug marrant, j’arrive pas à retrouver dans le code initial le parsing bogué qui fait que c’est le contenu brut entre les deux balises ouvrantes qui est importé, mais ok, ack.
[…]
qui fait la correction.

À noter que dans les tests ça ne se voit pas parce qu'ils ne se font pas avec de l'XML indenté, et donc pas de texte "espace" entre les balises.

Ah oui ok, pigé, merci.

> > Bug marrant, j’arrive pas à retrouver dans le code initial le parsing bogué qui fait que c’est le contenu brut entre les deux balises ouvrantes qui est importé, mais ok, ack. > […] > qui fait la correction. > > À noter que dans les tests ça ne se voit pas parce qu'ils ne se font pas avec de l'XML indenté, et donc pas de texte "espace" entre les balises. Ah oui ok, pigé, merci.
Sign in to join this conversation.
Reviewers
No reviewers
pmarillonnet
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/wcs#594
No description provided.
Delete Branch "wip/58170-workflow-file-option-import"

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?