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

misc: use transient data objects for autocomplete contexts (#76943) #277

Merged
fpeters merged 1 commits from wip/76943-use-transient-data-for-autocomplete-tokens into main 2023-05-02 08:16:37 +02:00
Conversation 1 Commits 1 Files Changed 4 +31 -28
fpeters commented 2023-05-01 09:43:30 +02:00
Owner
Copy Link

Vraiment wip, essai d'utilisation de TransientData pour stocker les jetons d'autocomplétion (plutôt que Token).

Vraiment wip, essai d'utilisation de TransientData pour stocker les jetons d'autocomplétion (plutôt que Token).
fpeters added 1 commit 2023-05-01 09:43:30 +02:00
fpeters commented 2023-05-01 13:38:35 +02:00
Author
Owner
Copy Link

Le sujet de base est la requête SQL sur les tokens, avec comparaison sur la colonne json; dans un premier temps j'ai imaginé sortir du json l'id de session pour en faire une colonne à part mais cette table liée au session existe déjà, c'est TransientData, donc je bascule pour l'utiliser.

Ça y met comme id le hash du contexte,

hashlib.sha1(repr(autocomplete_context).encode()).hexdigest()

c'est détourner l'id qui dans les autres utilisations de cette table est une valeur aléatoire mais ça passe, avec une unicité "garantie" par la présence de l'id de session dans le contexte.

Le sujet de base est la requête SQL sur les tokens, avec comparaison sur la colonne json; dans un premier temps j'ai imaginé sortir du json l'id de session pour en faire une colonne à part mais cette table liée au session existe déjà, c'est TransientData, donc je bascule pour l'utiliser. Ça y met comme id le hash du contexte, ``` hashlib.sha1(repr(autocomplete_context).encode()).hexdigest() ``` c'est détourner l'id qui dans les autres utilisations de cette table est une valeur aléatoire mais ça passe, avec une unicité "garantie" par la présence de l'id de session dans le contexte.
fpeters changed title from WIP: misc: use transient data objects for autocomplete contexts (#76943) to misc: use transient data objects for autocomplete contexts (#76943) 2023-05-01 13:38:40 +02:00
lguerin approved these changes 2023-05-01 19:21:52 +02:00
fpeters merged commit e988caa563 into main 2023-05-02 08:16:37 +02:00
fpeters deleted branch wip/76943-use-transient-data-for-autocomplete-tokens 2023-05-02 08:16:37 +02: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#277
No description provided.
Delete Branch "wip/76943-use-transient-data-for-autocomplete-tokens"

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?