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

sql: get new fts search value when ordering (#74972) #202

Merged
fpeters merged 1 commits from wip/74972-order-by-rank into main 2023-03-27 18:11:25 +02:00
Conversation 1 Commits 1 Files Changed 3 +69 -12
fpeters commented 2023-03-27 11:44:25 +02:00
Owner
Copy Link
No description provided.
lguerin approved these changes 2023-03-27 11:52:26 +02:00
fpeters changed title from sql: get new fts search value when ordering (#74972) to WIP: sql: get new fts search value when ordering (#74972) 2023-03-27 12:13:46 +02:00
fpeters force-pushed wip/74972-order-by-rank from cd6fc9a863 to 9b9869817d 2023-03-27 12:15:46 +02:00 Compare
fpeters force-pushed wip/74972-order-by-rank from 9b9869817d to 2dfd84ed35 2023-03-27 14:27:17 +02:00 Compare
fpeters force-pushed wip/74972-order-by-rank from 2dfd84ed35 to 54b19753e8 2023-03-27 15:08:40 +02:00 Compare
fpeters force-pushed wip/74972-order-by-rank from 54b19753e8 to 09cf2e30c1 2023-03-27 15:09:56 +02:00 Compare
fpeters force-pushed wip/74972-order-by-rank from 09cf2e30c1 to 9eceae046a 2023-03-27 15:48:22 +02:00 Compare
fpeters changed title from WIP: sql: get new fts search value when ordering (#74972) to sql: get new fts search value when ordering (#74972) 2023-03-27 16:31:50 +02:00
fpeters commented 2023-03-27 16:36:23 +02:00
Author
Owner
Copy Link

J'ai du refaire parce que

  1. il y a des arguments parfois supplémentaires, donc j'ai ajouté des **kwargs sur les critères où il manquait,
  2. il ne fallait pas recréer un objet Critère quand l'objet initial était déjà SQL parce que ça pouvait modifier une fois de trop les données (pour l'échappement des données sur l'ilike)
  3. il y a un attribut supplémentaire _label moche utilisé pour l'affichage sur la page de statistiques, qu'il ne faut pas perdre.
J'ai du refaire parce que 1. il y a des arguments parfois supplémentaires, donc j'ai ajouté des `**kwargs` sur les critères où il manquait, 2. il ne fallait pas recréer un objet Critère quand l'objet initial était déjà SQL parce que ça pouvait modifier une fois de trop les données (pour l'échappement des données sur l'ilike) 3. il y a un attribut supplémentaire `_label` moche utilisé pour l'affichage sur la page de statistiques, qu'il ne faut pas perdre.
fpeters requested review from lguerin 2023-03-27 16:36:28 +02:00
lguerin approved these changes 2023-03-27 18:05:19 +02:00
fpeters merged commit 63d4c99c9d into main 2023-03-27 18:11:25 +02:00
fpeters deleted branch wip/74972-order-by-rank 2023-03-27 18:11:25 +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#202
No description provided.
Delete Branch "wip/74972-order-by-rank"

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?