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

depreciations: don't store job on scan on import (#89213) #1370

Merged
tnoel merged 3 commits from wip/89213-fix-depreciation-scan into main 2024-04-05 16:53:23 +02:00
Conversation 2 Commits 3 Files Changed 6 +26 -1
lguerin commented 2024-04-05 15:02:08 +02:00
Owner
Copy Link
No description provided.
lguerin added 1 commit 2024-04-05 15:02:09 +02:00
ecazenave commented 2024-04-05 15:29:43 +02:00
Owner
Copy Link

Pour moi il faut remonter plus haut, faire en sorte que @job.check_deprecated_elements_in_object@ ne soit appelé quand lorsque c'est nécessaire (à l'import via API et lors des imports manuels via l'interface, j'imagine que c'est ça l'intention de https://dev.entrouvert.org/issues/72093 ?)

La ça va continuer à être appelé en permanence, même si ça pose plus de fichiers c'est pas top.

Pour moi il faut remonter plus haut, faire en sorte que @job.check_deprecated_elements_in_object@ ne soit appelé quand lorsque c'est nécessaire (à l'import via API et lors des imports manuels via l'interface, j'imagine que c'est ça l'intention de https://dev.entrouvert.org/issues/72093 ?) La ça va continuer à être appelé en permanence, même si ça pose plus de fichiers c'est pas top.
lguerin added 1 commit 2024-04-05 15:50:30 +02:00
lguerin commented 2024-04-05 15:51:46 +02:00
Author
Owner
Copy Link

0002 pour ajouter un test qui s'assure que c'est bien le cas: on ne joue pas le check des dépréciations sur un load d'objet.
c'est d'ailleurs ce code:

class XmlStorableObject(StorableObject):
    XML_NODES = []

    @classmethod
    def storage_load(cls, fd):
        first_byte = fd.read(1)
        fd.seek(0)
        if first_byte == b'<':
            return cls.import_from_xml(fd, include_id=True, check_deprecated=False)  # <- ici
        else:
            obj = StorableObject.storage_load(fd)
            obj._upgrade_must_store = True
            return obj
0002 pour ajouter un test qui s'assure que c'est bien le cas: on ne joue pas le check des dépréciations sur un load d'objet. c'est d'ailleurs ce code: ``` class XmlStorableObject(StorableObject): XML_NODES = [] @classmethod def storage_load(cls, fd): first_byte = fd.read(1) fd.seek(0) if first_byte == b'<': return cls.import_from_xml(fd, include_id=True, check_deprecated=False) # <- ici else: obj = StorableObject.storage_load(fd) obj._upgrade_must_store = True return obj ```
lguerin added 1 commit 2024-04-05 16:04:56 +02:00
gitea/wcs/pipeline/head There was a failure building this commit Details
06a1049a91
depreciations: don't check depreciations on snapshot load (#89213)
lguerin force-pushed wip/89213-fix-depreciation-scan from 06a1049a91 to e8cd2aa824 2024-04-05 16:20:21 +02:00 Compare
tnoel approved these changes 2024-04-05 16:53:13 +02:00
tnoel merged commit e8cd2aa824 into main 2024-04-05 16:53:23 +02:00
tnoel deleted branch wip/89213-fix-depreciation-scan 2024-04-05 16:53:23 +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
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/wcs#1370
No description provided.
Delete Branch "wip/89213-fix-depreciation-scan"

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?