trier par pertinence les résultats d'une recherche full text (#16253) #1063

Merged
fpeters merged 2 commits from wip/16253-order-by-rank-on-query into main 2024-01-26 08:47:27 +01:00
Owner
No description provided.
fpeters force-pushed wip/16253-order-by-rank-on-query from 9bf037ddfa to 95dd302b33 2024-01-23 14:04:18 +01:00 Compare
fpeters force-pushed wip/16253-order-by-rank-on-query from 95dd302b33 to 8be90cff0e 2024-01-23 15:40:04 +01:00 Compare
fpeters force-pushed wip/16253-order-by-rank-on-query from 8be90cff0e to 150521e337 2024-01-23 15:46:28 +01:00 Compare
fpeters force-pushed wip/16253-order-by-rank-on-query from 150521e337 to 330ea30524 2024-01-23 16:12:50 +01:00 Compare
fpeters force-pushed wip/16253-order-by-rank-on-query from 330ea30524 to 139ea9e70c 2024-01-23 16:40:24 +01:00 Compare
fpeters force-pushed wip/16253-order-by-rank-on-query from 139ea9e70c to f66f33cb5c 2024-01-24 15:36:47 +01:00 Compare
fpeters reviewed 2024-01-24 15:48:37 +01:00
@ -1488,0 +1489,4 @@
default_order_by = get_publisher().get_site_option('default-sort-order') or system_default_order_by
if self.view:
default_order_by = self.view.order_by or default_order_by
return default_order_by
Author
Owner

Le tri par défaut, qui va être -receipt_time sur les tableaux de traitement (mais -id sur l'export), aussi ça peut être overridé dans la configuration système (paramétrage qui serait à regarder un jour, pas l'impression que ça soit encore utilisé).

Et s'il y a une vue elle peut définir un tri, qui sera donc le tri par défaut de cette vue.

Le tri par défaut, qui va être -receipt_time sur les tableaux de traitement (mais -id sur l'export), aussi ça peut être overridé dans la configuration système (paramétrage qui serait à regarder un jour, pas l'impression que ça soit encore utilisé). Et s'il y a une vue elle peut définir un tri, qui sera donc le tri par défaut de cette vue.
@ -1488,0 +1498,4 @@
if get_request().form.get('q') and not order_by:
order_by = 'rank'
if not order_by:
order_by = default_order_by
Author
Owner

S'il n'y a pas de order_by explicite dans la query string, s'il y a une recherche full text on trie par pertinence, sinon on prend le tri par défaut.

Ces deux méthodes vont permettre un comportement cohérent plus bas, où il y avait du code qui n'était pas toujours pareil selon les circonstances.

S'il n'y a pas de order_by explicite dans la query string, s'il y a une recherche full text on trie par pertinence, sinon on prend le tri par défaut. Ces deux méthodes vont permettre un comportement cohérent plus bas, où il y avait du code qui n'était pas toujours pareil selon les circonstances.
@ -1134,3 +1134,3 @@
def get_order_by_or_400(value):
if value is None:
if value in (None, ''):
Author
Owner

On peut désormais avoir un order_by en chaine vide, on considère ça comme valide et le tri par défaut s'appliquera.

On peut désormais avoir un order_by en chaine vide, on considère ça comme valide et le tri par défaut s'appliquera.
@ -109,4 +109,0 @@
if ($('input[name="order_by"]').length == 0) {
/* if we don't have an order_by field, that means we do not support server
* side sorting, so we abort here */
return;
Author
Owner

Obsolète, on est en SQL on a tout le temps le tri serveur.

Obsolète, on est en SQL on a tout le temps le tri serveur.
@ -132,1 +127,3 @@
if (reversed === false) {
var new_key = $(this).data('field-sort-key');
if (sort_key === new_key) { // same column, reverse on second click, reset on third click
if (! reversed) {
Author
Owner

Cf le commentaire, un clic sur une colonne trie sur celle-ci, un second clic inverse le tri, un troisième retire le tri particulier; ça permet ainsi de revenir au tri par défaut.

Cf le commentaire, un clic sur une colonne trie sur celle-ci, un second clic inverse le tri, un troisième retire le tri particulier; ça permet ainsi de revenir au tri par défaut.
fpeters force-pushed wip/16253-order-by-rank-on-query from f66f33cb5c to 55e524b903 2024-01-24 15:48:47 +01:00 Compare
fpeters changed title from WIP: trier par pertinence les résultats d'une recherche full text (#16253) to trier par pertinence les résultats d'une recherche full text (#16253) 2024-01-24 15:58:59 +01:00
lguerin approved these changes 2024-01-25 09:01:15 +01:00
fpeters merged commit 2b2d59baed into main 2024-01-26 08:47:27 +01:00
fpeters deleted branch wip/16253-order-by-rank-on-query 2024-01-26 08:47:28 +01:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
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#1063
No description provided.