contacts icon indicating copy to clipboard operation
contacts copied to clipboard

Recent contacts should not appear in search results

Open Jerome-Herbinet opened this issue 2 years ago • 12 comments

Describe the bug

If I search for a contact and it's also one of the recently used contacts, it appears twice in the results, creating confusion.

2023-06-21_10-37

Steps to reproduce

  1. Search for a contact which is in the recently contacted
  2. See the contact which appears twice

Expected behavior

Only the original contact should appear.

Actual behavior

The original appears with its virtual recently contacted version.

Contact version

5.3.2

Operating system

Ubuntu Server

PHP engine version

PHP 8.1

Web server

Apache (supported)

Database

MySQL

Additional info

Nextcloud 26.0.2

Jerome-Herbinet avatar Jun 21 '23 08:06 Jerome-Herbinet

@szaimen @hamza221

Jerome-Herbinet avatar Jun 21 '23 08:06 Jerome-Herbinet

I think there have been fixes to this with 27. did you already tried rhat version?

szaimen avatar Jun 21 '23 09:06 szaimen

I think there have been fixes to this with 27. did you already tried rhat version?

Not yet.

Jerome-Herbinet avatar Jun 21 '23 09:06 Jerome-Herbinet

@jancborchardt @marcoambrosini @nimishavijay for making a call on whether Recently contacted are to be treated special and excluded from search

ChristophWurst avatar Jun 22 '23 07:06 ChristophWurst

@ChristophWurst can't we just filter the list before rendering and remove dupes?
Or can we filter only the "all contacts" list?

marcoambrosini avatar Jun 23 '23 02:06 marcoambrosini

Recently contacted are part of All contacts. It is literally all contacts of all enabled address books you see there.

The duplication is tackled in https://github.com/nextcloud/server/issues/31606.

ChristophWurst avatar Jun 23 '23 06:06 ChristophWurst

But then isn't this issue a duplicate itself 👀

marcoambrosini avatar Jun 23 '23 08:06 marcoambrosini

Well, depends.

If the issue is that you see duplicate results because of a real contact and the recently contacted entry, yes.

If the issues is that no Recently contacted should ever show as search result, even if you don't have that person as real contact, then no.

ChristophWurst avatar Jun 23 '23 08:06 ChristophWurst

Ok I get it now, thanks. I think it could be handy to show deduplicated recently contacted people but I would add captions elements in the returned list to clearly differentiate between your contacts and people that you've recently contacted that are not in your contacts. Just like the conversation search in talk that displays the conversation you've joined first and then the conversations that you can join, and so forth.

Screenshot 2023-06-23 at 16 29 56

Not related but we could have an "add to contacts" action on the recently contacted

marcoambrosini avatar Jun 23 '23 08:06 marcoambrosini

Not related but we could have an "add to contacts" action on the recently contacted

We have that but it's called Clone contact

ChristophWurst avatar Jun 23 '23 09:06 ChristophWurst

I think there have been fixes to this with 27. did you already tried rhat version?

The issue is still persitent in NC 27.1.1 with Contacts App 5.4.2

gibelium avatar Sep 28 '23 13:09 gibelium

Still occurs in NC 28.0.6 and Contacts 5.5.3.

Jerome-Herbinet avatar May 24 '24 12:05 Jerome-Herbinet

The original issue you describe @Jerome-Herbinet is not so much "Recent contacts should not appear in search results" but rather "Duplicate contacts should not appear.

Since the related issue @ChristophWurst mentioned is fixed as per https://github.com/nextcloud/server/issues/31606, I would say this issue can be closed.

By design, we do want recently contacted people to show up in search.

jancborchardt avatar Aug 20 '24 10:08 jancborchardt