Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Recherche d'apprenants lors de l'inscription d'utilisateurs! #3045

Closed
ticfp opened this issue Dec 9, 2019 · 5 comments
Closed

Recherche d'apprenants lors de l'inscription d'utilisateurs! #3045

ticfp opened this issue Dec 9, 2019 · 5 comments
Assignees
Milestone

Comments

@ticfp
Copy link

ticfp commented Dec 9, 2019

Bonjour!

Sur notre plateforme Chamilo 1.11.10 adaptée (www.ticfp.qc.ca) nous vivons un problème avec la recherche d'utilisateurs sur notre plateforme.

Au moment d'ajouter un élève ou un enseignant à un cours, si nous mettons un espace entre les 2 noms dans la zone de saisie pour la recherche, aucun résultats de recherche n’apparaît. Si je recherche un mot seulement, les résultats apparaissent normalement.

De plus, pourquoi ne pourrais t'on pas avoir accès à un champ recherche avancée dans cette section? En tant qu'admin, j'ai accès à ce type de recherche (avancées) dans la section recherche d'utilisateurs, il serait bien de la rendre accessible dans la section d'ajout d'élèves ou d’enseignants aux différents cours!

Système d'exploitation : Windows 10
Navigateur: peu importe: Chrome, Firefox, etc...
**Serveur
Système d'exploitation : Ubuntu 18.04 / Windows Server
Version de Chamilo: 1.11.10 et 1.11x latest
Version de PHP: 7.3

MERCI!

@ticfp ticfp closed this as completed Dec 16, 2019
@ticfp
Copy link
Author

ticfp commented Dec 17, 2019

D'autres détails me permettent de prouver que le problème n'est pas avec notre version, mais dans le code de base! (à venir dans quelque minutes)

@ticfp ticfp reopened this Dec 17, 2019
@ticfp ticfp changed the title Recherche d'élèves lors de l'inscription d'utilisateurs! Recherche d'apprenants lors de l'inscription d'utilisateurs! Dec 17, 2019
@ticfp
Copy link
Author

ticfp commented Dec 17, 2019

Dans le code trouvé ici: https://github.com/chamilo/chamilo-lms/blob/1.11.x/main/user/subscribe_user.php

Il faudrait apporter les modification suivantes:

Blocs de code (lignes 371-375 et lignes 636-640)
Remplacer:
firstname LIKE '%".$keyword."%' OR
lastname LIKE '%".$keyword."%' OR
email LIKE '%".$keyword."%' OR
username LIKE '%".$keyword."%' OR
official_code LIKE '%".$keyword."%'

Par:

firstname LIKE '%".$keyword."%' OR
lastname LIKE '%".$keyword."%' OR
concat(firstname, ' ', lastname) LIKE '%".$keyword."%' OR
concat(lastname, ' ', firstname) LIKE '%".$keyword."%' OR
email LIKE '%".$keyword."%' OR
username LIKE '%".$keyword."%' OR
official_code LIKE '%".$keyword."%'

@ywarnier
Copy link
Member

Malheureusement ce changement de requête aurait un effet affreux sur les performances (il me semble que nous l'utilisions par le passé et que nous l'avons supprimé), en particulier avec un grand nombre d'utilisateurs.
Peut être qu'à l'avenir nous pourrons compter sur une fonctionnalité automatique d'indexation full text des noms d'utilisateurs, mais en dehors de ça, implémenter ce changement serait trop coûteux pour certaines plateformes.

@ywarnier ywarnier added this to the Undefined milestone Jun 20, 2020
@ywarnier
Copy link
Member

Je reviens sur cette tâche: implémenter un formulaire de recherche avancée, comme suggéré par @ticfp permettrait en effet de chercher à la fois nom + prénom, avec une implémentation relativement simple.

@ywarnier
Copy link
Member

Ajouté au travers de 098626e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants