From f42c29ebaceafc75f387d60b1f5e3a0cce15b1db Mon Sep 17 00:00:00 2001
From: Arthur Schiwon <blizzz@arthur-schiwon.de>
Date: Wed, 14 Mar 2018 14:30:45 +0100
Subject: [PATCH] the FN is optional, carrying the displayname if present

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
---
 .../Collaboration/Collaborators/MailPlugin.php   | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

diff --git a/lib/private/Collaboration/Collaborators/MailPlugin.php b/lib/private/Collaboration/Collaborators/MailPlugin.php
index ee5fcd60bd..101d6845ec 100644
--- a/lib/private/Collaboration/Collaborators/MailPlugin.php
+++ b/lib/private/Collaboration/Collaborators/MailPlugin.php
@@ -88,6 +88,10 @@ class MailPlugin implements ISearchPlugin {
 					$emailAddresses = [$emailAddresses];
 				}
 				foreach ($emailAddresses as $emailAddress) {
+					$displayName = $emailAddress;
+					if (isset($contact['FN'])) 	{
+						$displayName = $contact['FN'] . ' (' . $emailAddress . ')';
+					}
 					$exactEmailMatch = strtolower($emailAddress) === $lowerSearch;
 
 					if (isset($contact['isLocalSystemBook'])) {
@@ -116,7 +120,7 @@ class MailPlugin implements ISearchPlugin {
 
 							if (!$this->isCurrentUser($cloud) && !$searchResult->hasResult($userType, $cloud->getUser())) {
 								$singleResult = [[
-									'label' => $contact['FN'] . " ($emailAddress)",
+									'label' => $displayName,
 									'value' => [
 										'shareType' => Share::SHARE_TYPE_USER,
 										'shareWith' => $cloud->getUser(),
@@ -137,7 +141,7 @@ class MailPlugin implements ISearchPlugin {
 
 							if (!$this->isCurrentUser($cloud) && !$searchResult->hasResult($userType, $cloud->getUser())) {
 								$userResults['wide'][] = [
-									'label' => $contact['FN'] . " ($emailAddress)",
+									'label' => $displayName,
 									'value' => [
 										'shareType' => Share::SHARE_TYPE_USER,
 										'shareWith' => $cloud->getUser(),
@@ -148,12 +152,14 @@ class MailPlugin implements ISearchPlugin {
 						continue;
 					}
 
-					if ($exactEmailMatch || strtolower($contact['FN']) === $lowerSearch) {
+					if ($exactEmailMatch
+						|| isset($contact['FN']) && strtolower($contact['FN']) === $lowerSearch)
+					{
 						if ($exactEmailMatch) {
 							$searchResult->markExactIdMatch($emailType);
 						}
 						$result['exact'][] = [
-							'label' => $contact['FN'] . " ($emailAddress)",
+							'label' => $displayName,
 							'value' => [
 								'shareType' => Share::SHARE_TYPE_EMAIL,
 								'shareWith' => $emailAddress,
@@ -161,7 +167,7 @@ class MailPlugin implements ISearchPlugin {
 						];
 					} else {
 						$result['wide'][] = [
-							'label' => $contact['FN'] . " ($emailAddress)",
+							'label' => $displayName,
 							'value' => [
 								'shareType' => Share::SHARE_TYPE_EMAIL,
 								'shareWith' => $emailAddress,