Skip to content
This repository was archived by the owner on Mar 28, 2026. It is now read-only.

restrict post search "filtering on e-mail addresses matching search string" when there are multiple e-mail adresses per result - #197

Open
prigaux wants to merge 1 commit into
horde:masterfrom
prigaux:fix12480
Open

prigaux wants to merge 1 commit into
horde:masterfrom
prigaux:fix12480

Conversation

@prigaux

@prigaux prigaux commented Jul 4, 2016

Copy link
Copy Markdown
Contributor

This limits the effects of request #12480.

Rationale: there is no reason to completely discard some search results.
Example: result for "Rigaux Pascal" (that matches our LDAP "cn") are discarded because my e-mail address is "Pascal.Rigaux@univ-paris1.fr".

Limitation: what should be done if 2 e-mail addresses are returned, none matching the search string?
Better fix would be to include all e-mails, but sort them?

…tring" when there are multiple e-mail adresses per result

This limits the effects of request #12480.

Rationale: there is no reason to completely discard some search results.
Example: result for "Rigaux Pascal" (that matches our LDAP "cn") are discarded because my e-mail address is "Pascal.Rigaux@univ-paris1.fr".

Limitation: what should be done if 2 e-mail addresses are returned, none matching the search string?
Better fix would be to include all e-mails, but sort them?
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant