Skip to content

Commit 9e1914c

Browse files
ArtificialOwlArtificialOwl
committed
displayname on federated shares
Signed-off-by: Maxence Lange <[email protected]>
1 parent 3eb73d0 commit 9e1914c

5 files changed

Lines changed: 162 additions & 35 deletions

File tree

‎apps/federatedfilesharing/lib/Notifier.php‎

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -102,8 +102,9 @@ public function prepare(INotification $notification, string $languageCode): INot
102102
$notification->setIcon($this->url->getAbsoluteURL($this->url->imagePath('core', 'actions/share.svg')));
103103

104104
$params = $notification->getSubjectParameters();
105+
$displayName = (count($params) > 3) ? $params[3] : '';
105106
if ($params[0] !== $params[1] && $params[1] !== null) {
106-
$remoteInitiator = $this->createRemoteUser($params[0]);
107+
$remoteInitiator = $this->createRemoteUser($params[0], $displayName);
107108
$remoteOwner = $this->createRemoteUser($params[1]);
108109
$params[3] = $remoteInitiator['name'] . '@' . $remoteInitiator['server'];
109110
$params[4] = $remoteOwner['name'] . '@' . $remoteOwner['server'];
@@ -121,7 +122,7 @@ public function prepare(INotification $notification, string $languageCode): INot
121122
]
122123
);
123124
} else {
124-
$remoteOwner = $this->createRemoteUser($params[0]);
125+
$remoteOwner = $this->createRemoteUser($params[0], $displayName);
125126
$params[3] = $remoteOwner['name'] . '@' . $remoteOwner['server'];
126127

127128
$notification->setRichSubject(
@@ -166,19 +167,21 @@ public function prepare(INotification $notification, string $languageCode): INot
166167

167168
/**
168169
* @param string $cloudId
170+
* @param string $displayName - overwrite display name
171+
*
169172
* @return array
170173
*/
171-
protected function createRemoteUser($cloudId, $displayName = null) {
174+
protected function createRemoteUser(string $cloudId, string $displayName = '') {
172175
try {
173176
$resolvedId = $this->cloudIdManager->resolveCloudId($cloudId);
174-
if ($displayName === null) {
177+
if ($displayName === '') {
175178
$displayName = $this->getDisplayName($resolvedId);
176179
}
177180
$user = $resolvedId->getUser();
178181
$server = $resolvedId->getRemote();
179182
} catch (HintException $e) {
180183
$user = $cloudId;
181-
$displayName = $cloudId;
184+
$displayName = ($displayName !== '') ? $displayName : $cloudId;
182185
$server = '';
183186
}
184187

‎apps/federatedfilesharing/lib/OCM/CloudFederationProviderFiles.php‎

Lines changed: 32 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -55,10 +55,12 @@
5555
use OCP\IURLGenerator;
5656
use OCP\IUserManager;
5757
use OCP\Notification\IManager as INotificationManager;
58+
use OCP\Server;
5859
use OCP\Share\Exceptions\ShareNotFound;
5960
use OCP\Share\IManager;
6061
use OCP\Share\IShare;
6162
use OCP\Util;
63+
use Psr\Container\ContainerExceptionInterface;
6264

6365
class CloudFederationProviderFiles implements ICloudFederationProvider {
6466

@@ -250,26 +252,29 @@ public function shareReceived(ICloudFederationShare $share) {
250252
$this->externalShareManager->addShare($remote, $token, '', $name, $owner, $shareType,false, $shareWith, $remoteId);
251253
$shareId = \OC::$server->getDatabaseConnection()->lastInsertId('*PREFIX*share_external');
252254

255+
// get DisplayName about the owner of the share
256+
$ownerDisplayName = $this->getUserDisplayName($ownerFederatedId);
257+
253258
if ($shareType === IShare::TYPE_USER) {
254259
$event = $this->activityManager->generateEvent();
255260
$event->setApp('files_sharing')
256261
->setType('remote_share')
257-
->setSubject(RemoteShares::SUBJECT_REMOTE_SHARE_RECEIVED, [$ownerFederatedId, trim($name, '/')])
262+
->setSubject(RemoteShares::SUBJECT_REMOTE_SHARE_RECEIVED, [$ownerFederatedId, trim($name, '/'), $ownerDisplayName])
258263
->setAffectedUser($shareWith)
259264
->setObject('remote_share', $shareId, $name);
260265
\OC::$server->getActivityManager()->publish($event);
261-
$this->notifyAboutNewShare($shareWith, $shareId, $ownerFederatedId, $sharedByFederatedId, $name);
266+
$this->notifyAboutNewShare($shareWith, $shareId, $ownerFederatedId, $sharedByFederatedId, $name, $ownerDisplayName);
262267
} else {
263268
$groupMembers = $this->groupManager->get($shareWith)->getUsers();
264269
foreach ($groupMembers as $user) {
265270
$event = $this->activityManager->generateEvent();
266271
$event->setApp('files_sharing')
267272
->setType('remote_share')
268-
->setSubject(RemoteShares::SUBJECT_REMOTE_SHARE_RECEIVED, [$ownerFederatedId, trim($name, '/')])
273+
->setSubject(RemoteShares::SUBJECT_REMOTE_SHARE_RECEIVED, [$ownerFederatedId, trim($name, '/'), $ownerDisplayName])
269274
->setAffectedUser($user->getUID())
270275
->setObject('remote_share', $shareId, $name);
271276
\OC::$server->getActivityManager()->publish($event);
272-
$this->notifyAboutNewShare($user->getUID(), $shareId, $ownerFederatedId, $sharedByFederatedId, $name);
277+
$this->notifyAboutNewShare($user->getUID(), $shareId, $ownerFederatedId, $sharedByFederatedId, $name, $ownerDisplayName);
273278
}
274279
}
275280
return $shareId;
@@ -335,13 +340,13 @@ private function mapShareTypeToNextcloud($shareType) {
335340
return $result;
336341
}
337342

338-
private function notifyAboutNewShare($shareWith, $shareId, $ownerFederatedId, $sharedByFederatedId, $name): void {
343+
private function notifyAboutNewShare($shareWith, $shareId, $ownerFederatedId, $sharedByFederatedId, $name, $displayName): void {
339344
$notification = $this->notificationManager->createNotification();
340345
$notification->setApp('files_sharing')
341346
->setUser($shareWith)
342347
->setDateTime(new \DateTime())
343348
->setObject('remote_share', $shareId)
344-
->setSubject('remote_share', [$ownerFederatedId, $sharedByFederatedId, trim($name, '/')]);
349+
->setSubject('remote_share', [$ownerFederatedId, $sharedByFederatedId, trim($name, '/'), $displayName]);
345350

346351
$declineAction = $notification->createAction();
347352
$declineAction->setLabel('decline')
@@ -579,6 +584,8 @@ private function unshare($id, array $notification) {
579584
->where($qb->expr()->eq('parent', $qb->createNamedParameter((int)$share['id'])));
580585
$qb->execute();
581586

587+
$ownerDisplayName = $this->getUserDisplayName($owner->getId());
588+
582589
if ((int)$share['share_type'] === IShare::TYPE_USER) {
583590
if ($share['accepted']) {
584591
$path = trim($mountpoint, '/');
@@ -594,7 +601,7 @@ private function unshare($id, array $notification) {
594601
$event = $this->activityManager->generateEvent();
595602
$event->setApp('files_sharing')
596603
->setType('remote_share')
597-
->setSubject(RemoteShares::SUBJECT_REMOTE_SHARE_UNSHARED, [$owner->getId(), $path])
604+
->setSubject(RemoteShares::SUBJECT_REMOTE_SHARE_UNSHARED, [$owner->getId(), $path, $ownerDisplayName])
598605
->setAffectedUser($user)
599606
->setObject('remote_share', (int)$share['id'], $path);
600607
\OC::$server->getActivityManager()->publish($event);
@@ -824,4 +831,22 @@ private function isS2SEnabled($incoming = false) {
824831
public function getSupportedShareTypes() {
825832
return ['user', 'group'];
826833
}
834+
835+
836+
private function getUserDisplayName(string $userId): string {
837+
// check if gss is enabled and available
838+
if (!$this->appManager->isInstalled('globalsiteselector')
839+
|| !class_exists('\OCA\GlobalSiteSelector\Service\SlaveService')) {
840+
return '';
841+
}
842+
843+
try {
844+
$slaveService = Server::get(\OCA\GlobalSiteSelector\Service\SlaveService::class);
845+
} catch (ContainerExceptionInterface $e) {
846+
\OC::$server->getLogger()->logException($e);
847+
return '';
848+
}
849+
850+
return $slaveService->getUserDisplayName($userId, false);
851+
}
827852
}

‎apps/files_sharing/lib/Activity/Providers/Base.php‎

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -157,9 +157,11 @@ protected function getFile($parameter, IEvent $event = null) {
157157

158158
/**
159159
* @param string $uid
160+
* @param string $overwriteDisplayName - overwrite display name, only if user is not local
161+
*
160162
* @return array
161163
*/
162-
protected function getUser($uid) {
164+
protected function getUser(string $uid, string $overwriteDisplayName = '') {
163165
// First try local user
164166
$displayName = $this->userManager->getDisplayName($uid);
165167
if ($displayName !== null) {
@@ -176,7 +178,7 @@ protected function getUser($uid) {
176178
return [
177179
'type' => 'user',
178180
'id' => $cloudId->getUser(),
179-
'name' => $this->getDisplayNameFromAddressBook($cloudId->getDisplayId()),
181+
'name' => (($overwriteDisplayName !== '') ? $overwriteDisplayName : $this->getDisplayNameFromAddressBook($cloudId->getDisplayId())),
180182
'server' => $cloudId->getRemote(),
181183
];
182184
}
@@ -185,7 +187,7 @@ protected function getUser($uid) {
185187
return [
186188
'type' => 'user',
187189
'id' => $uid,
188-
'name' => $uid,
190+
'name' => (($overwriteDisplayName !== '') ? $overwriteDisplayName : $uid),
189191
];
190192
}
191193

‎apps/files_sharing/lib/Activity/Providers/RemoteShares.php‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -115,13 +115,14 @@ protected function getParsedParameters(IEvent $event) {
115115
switch ($subject) {
116116
case self::SUBJECT_REMOTE_SHARE_RECEIVED:
117117
case self::SUBJECT_REMOTE_SHARE_UNSHARED:
118+
$displayName = (count($parameters) > 2) ? $parameters[2] : '';
118119
return [
119120
'file' => [
120121
'type' => 'pending-federated-share',
121122
'id' => $parameters[1],
122123
'name' => $parameters[1],
123124
],
124-
'user' => $this->getUser($parameters[0]),
125+
'user' => $this->getUser($parameters[0], $displayName)
125126
];
126127
case self::SUBJECT_REMOTE_SHARE_ACCEPTED:
127128
case self::SUBJECT_REMOTE_SHARE_DECLINED:

0 commit comments

Comments
 (0)