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

data sources: export detailed roles infos (#84889) #948

Merged
fpeters merged 1 commits from wip/84889-ds-roles-export-import into main 2024-03-19 15:35:22 +01:00
Conversation 0 Commits 1 Files Changed 3 +85 -3
fpeters commented 2023-12-19 11:41:06 +01:00
Owner
Copy Link
No description provided.
fpeters added 1 commit 2023-12-19 11:41:07 +01:00
gitea/wcs/pipeline/head This commit looks good Details
6317ffc705 data sources: export detailed roles infos (#84889)
fpeters force-pushed wip/84889-ds-roles-export-import from 6317ffc705 to 9c2b41c792 2023-12-19 12:01:25 +01:00 Compare
fpeters force-pushed wip/84889-ds-roles-export-import from 9c2b41c792 to eb1ccb489e 2023-12-19 12:31:25 +01:00 Compare
fpeters changed title from WIP: data sources: export detailed roles infos (#84889) to data sources: export detailed roles infos (#84889) 2023-12-21 09:59:00 +01:00
pmarillonnet reviewed 2024-01-02 11:00:59 +01:00
pmarillonnet left a comment
Owner
Copy Link

Quelques petits trucs vus au passage, plus pour ma compréhension de l’affaire, pas une demande de modifications.

Quelques petits trucs vus au passage, plus pour ma compréhension de l’affaire, pas une demande de modifications.
wcs/qommon/xml_storage.py Outdated
@ -163,0 +165,4 @@
def import_ds_roles_from_xml(self, element, include_id=False, **kwargs):
imported_roles = self.import_roles_from_xml(element, include_id=include_id, **kwargs)
if callable(imported_roles):
pmarillonnet commented 2024-01-02 11:00:22 +01:00
Owner
Copy Link

Par simple curiosité de ma part, qu’est-ce qui fait qu’on a besoin de gérer explicitement le laziness ici ? Ce n’est pas automatiquement résolu lors du listcomp juste en dessous ?

Par simple curiosité de ma part, qu’est-ce qui fait qu’on a besoin de gérer explicitement le laziness ici ? Ce n’est pas automatiquement résolu lors du listcomp juste en dessous ?
fpeters commented 2024-03-01 10:31:35 +01:00
Author
Owner
Copy Link

le import_roles_from_xml peut retourner une liste vide [] ou une fonction (lazy_roles) qu'il faut appeler.

le import_roles_from_xml peut retourner une liste vide [] ou une fonction (lazy_roles) qu'il faut appeler.
pmarillonnet commented 2024-03-19 15:30:46 +01:00
Owner
Copy Link

Ok, pigé, merci.

Ok, pigé, merci.
pmarillonnet marked this conversation as resolved
wcs/qommon/xml_storage.py Outdated
@ -163,0 +169,4 @@
imported_roles = imported_roles()
role_ids = [x.id for x in imported_roles]
for sub in element:
if sub.tag == 'item': # legacy support for <item>{id}</item>
pmarillonnet commented 2024-01-02 10:58:43 +01:00
Owner
Copy Link

Est-ce qu’on veut quand même inclure le format legacy <item>{id}</item> lorsque include_id=False ?

Est-ce qu’on veut quand même inclure le format legacy `<item>{id}</item>` lorsque `include_id=False` ?
fpeters commented 2024-03-01 10:35:05 +01:00
Author
Owner
Copy Link

Oui, en fait le include_id a une définition un peu large et mouvante, sur de l'import c'est plutôt "remettre les id trouvés dans le fichier importé" que "ne pas utiliser les id du fichier importé pour trouver un objet".

Oui, en fait le include_id a une définition un peu large et mouvante, sur de l'import c'est plutôt "remettre les id trouvés dans le fichier importé" que "ne pas utiliser les id du fichier importé pour trouver un objet".
pmarillonnet commented 2024-03-19 15:31:42 +01:00
Owner
Copy Link

Arf ok, je n’avais pas capté cette subtilité.

Arf ok, je n’avais pas capté cette subtilité.
pmarillonnet marked this conversation as resolved
wcs/qommon/xml_storage.py Outdated
@ -163,0 +178,4 @@
[Contains('id', getattr(self, attribute_name, None) or [])]
):
sub = ET.SubElement(element, 'role')
sub.attrib['role-id'] = role.id # always einclude id
pmarillonnet commented 2024-01-02 10:58:59 +01:00
Owner
Copy Link

Coquille dans le commentaire ici.

Coquille dans le commentaire ici.
fpeters commented 2024-03-01 10:29:06 +01:00
Author
Owner
Copy Link

corrigé.

corrigé.
pmarillonnet marked this conversation as resolved
fpeters force-pushed wip/84889-ds-roles-export-import from eb1ccb489e to b268bdb154 2024-03-01 10:29:00 +01:00 Compare
fpeters requested review from pmarillonnet 2024-03-01 10:35:30 +01:00
pmarillonnet approved these changes 2024-03-19 15:31:58 +01:00
fpeters merged commit 76b94d7ee8 into main 2024-03-19 15:35:22 +01:00
fpeters deleted branch wip/84889-ds-roles-export-import 2024-03-19 15:35:23 +01:00
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#948
No description provided.
Delete Branch "wip/84889-ds-roles-export-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?