Skip to content

Commit 0633d95

Browse files
nfebeskjnldsv
authored andcommitted
fix(unifiedSearch): Prevent broken avatars for federated users
Signed-off-by: fenn-cs <[email protected]>
1 parent bf4d2ec commit 0633d95

File tree

4 files changed

+9
-4
lines changed

4 files changed

+9
-4
lines changed

core/src/components/UnifiedSearch/SearchableList.vue

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,11 @@
4646
:wide="true"
4747
@click="itemSelected(element)">
4848
<template #icon>
49-
<NcAvatar :user="element.user" :show-user-status="false" :hide-favorite="false" />
49+
<NcAvatar v-if="element.isUser" :user="element.user" :show-user-status="false" />
50+
<NcAvatar v-else
51+
:is-no-user="true"
52+
:display-name="element.displayName"
53+
:show-user-status="false" />
5054
</template>
5155
{{ element.displayName }}
5256
</NcButton>

core/src/views/UnifiedSearchModal.vue

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -380,6 +380,7 @@ export default {
380380
subname: contact.emailAddresses[0] ? contact.emailAddresses[0] : '',
381381
icon: '',
382382
user: contact.id,
383+
isUser: contact.isUser,
383384
}
384385
})
385386
},

dist/core-unified-search.js

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/core-unified-search.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)