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

misc: make {% temporary_access_url %} work during draft (#83091) #811

Merged
fpeters merged 1 commits from wip/83091-draft-temporary-access into main 2023-11-03 12:05:12 +01:00
Conversation 0 Commits 1 Files Changed 3 +31
fpeters commented 2023-11-03 10:34:17 +01:00
Owner
Copy Link
No description provided.
fpeters force-pushed wip/83091-draft-temporary-access from e657d1150f to 78105c1d5d 2023-11-03 10:57:29 +01:00 Compare
fpeters force-pushed wip/83091-draft-temporary-access from 78105c1d5d to e0012b7aee 2023-11-03 11:29:21 +01:00 Compare
fpeters changed title from WIP: misc: make {% temporary_access_url %} work during draft (#83091) to misc: make {% temporary_access_url %} work during draft (#83091) 2023-11-03 11:43:55 +01:00
fpeters reviewed 2023-11-03 11:47:08 +01:00
fpeters left a comment
Author
Owner
Copy Link

Ça fonctionne mais ça laisse comme grosse limitation la nécessité d'avoir activé le code de suivi sur la démarche concernée.

Pour le moment les brouillons ne sont pas enregistrés quand ça n'est pas le cas,

    def has_draft_support(self):
        if self.edit_mode:
            return False
        if self.formdef.enable_tracking_codes:
            return True
        session = get_session()
        return session.has_user()

mais remplacer ça pour enregistrer systématiquement amène un tas de tests en échec, qui demandent une analyse plus longue.

Il y aurait https://dev.entrouvert.org/issues/24635 avec le gros nettoyage pour permettre ça de manière propre.

Ça fonctionne mais ça laisse comme grosse limitation la nécessité d'avoir activé le code de suivi sur la démarche concernée. Pour le moment les brouillons ne sont pas enregistrés quand ça n'est pas le cas, ``` def has_draft_support(self): if self.edit_mode: return False if self.formdef.enable_tracking_codes: return True session = get_session() return session.has_user() ``` mais remplacer ça pour enregistrer systématiquement amène un tas de tests en échec, qui demandent une analyse plus longue. Il y aurait https://dev.entrouvert.org/issues/24635 avec le gros nettoyage pour permettre ça de manière propre.
wcs/forms/root.py
@ -904,6 +904,7 @@ class FormPage(Directory, TempfileDirectoryMixin, FormTemplateMixin):
pass
formdata = FormData()
formdata._draft_id = session_data.get('draft_formdata_id')
fpeters commented 2023-11-03 11:44:18 +01:00
Author
Owner
Copy Link

Stockage dans le formdata temporaire de l'id du brouillon.

Stockage dans le formdata temporaire de l'id du brouillon.
wcs/qommon/templatetags/qommon.py
@ -1205,0 +1205,4 @@
if not formdata_id:
# try to get id of draft formdata
try:
formdata_id = context.get('form')._formdata._draft_id
fpeters commented 2023-11-03 11:44:48 +01:00
Author
Owner
Copy Link

Et utilisation de celui-ci dans temporary_access_url.

Et utilisation de celui-ci dans temporary_access_url.
lguerin approved these changes 2023-11-03 11:56:26 +01:00
fpeters merged commit e0012b7aee into main 2023-11-03 12:05:12 +01:00
fpeters deleted branch wip/83091-draft-temporary-access 2023-11-03 12:05:12 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
lguerin
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#811
No description provided.
Delete Branch "wip/83091-draft-temporary-access"

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?