Merge pull request #807 from nextcloud/bugfix/noid/display-name-should-be-a-string

Make sure the displayname is always a string
This commit is contained in:
Joas Schilling 2018-04-24 16:49:04 +02:00 коммит произвёл GitHub
Родитель 16bc8888a0 e924958052
Коммит 09a7bd51c7
Не найден ключ, соответствующий данной подписи
Идентификатор ключа GPG: 4AEE18F83AFDEB23
1 изменённых файлов: 2 добавлений и 2 удалений

Просмотреть файл

@ -240,10 +240,10 @@ class ChatController extends OCSController {
$guestNames = !empty($guestSessions) ? $this->guestManager->getNamesBySessionHashes($guestSessions) : [];
$response = new DataResponse(array_map(function (IComment $comment) use ($token, $guestNames) {
$displayName = null;
$displayName = '';
if ($comment->getActorType() === 'users') {
$user = $this->userManager->get($comment->getActorId());
$displayName = $user instanceof IUser ? $user->getDisplayName() : null;
$displayName = $user instanceof IUser ? $user->getDisplayName() : '';
} else if ($comment->getActorType() === 'guests' && isset($guestNames[$comment->getActorId()])) {
$displayName = $guestNames[$comment->getActorId()];
}