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

User search in MySQL store does not use full text search index #15760

Open
e-chip opened this issue Oct 3, 2020 · 2 comments
Open

User search in MySQL store does not use full text search index #15760

e-chip opened this issue Oct 3, 2020 · 2 comments

Comments

@e-chip
Copy link

e-chip commented Oct 3, 2020

Summary

Our team is using mattermost with mysql storage. And we found that we cannot find users by non-first word in the field.
For example if Firstname field is 'Maria Eleni' we cannot find her by 'Eleni' query.
SqlUserStore creates full text search index for all search modes.
But as i can see from the code user search in MySQL store does not use full text search index.

Steps to reproduce

Add user with two words in the firstname or lastname.
Try to find user (Ctrl+K or Ctrl+Shift+K) using the second word from the selected field.
User will not be found.

Expected behavior

User should be found.

Possible fixes

We have fixed this issue for us and if core team and community wants i can make a PR.
Also PR in mattermost-redux repo will be required to handle search results.

@e-chip
Copy link
Author

e-chip commented Oct 3, 2020

Also i think these issues in Jira are related to this:
https://mattermost.atlassian.net/browse/MM-5466
https://mattermost.atlassian.net/browse/MM-8423

@amyblais
Copy link
Member

amyblais commented Oct 5, 2020

@e-chip Thanks, I'm sure our team would be happy with a fix for this if you're interested to open a pull request.

e-chip pushed a commit to e-chip/mattermost-server that referenced this issue Oct 27, 2020
e-chip pushed a commit to e-chip/mattermost-server that referenced this issue Oct 27, 2020
@amyblais amyblais added Help Wanted Community help wanted PR Exists labels Sep 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants