Merge branch 'krille/only-use-prevcontent-fallback-on-left-users' into 'main'

fix: Do not display prevContent displayname and avatar for joined users

Closes famedly/company/product-management#754

See merge request famedly/company/frontend/famedlysdk!1306
This commit is contained in:
Nicolas Werner 2023-06-14 11:37:38 +00:00
commit ec84af679e
2 changed files with 10 additions and 11 deletions

View File

@ -70,7 +70,9 @@ class User extends Event {
/// The displayname of the user if the user has set one. /// The displayname of the user if the user has set one.
String? get displayName => String? get displayName =>
content.tryGet<String>('displayname') ?? content.tryGet<String>('displayname') ??
prevContent?.tryGet<String>('displayname'); (membership == Membership.join
? null
: prevContent?.tryGet<String>('displayname'));
/// Returns the power level of this user. /// Returns the power level of this user.
int get powerLevel => room.getPowerLevelByUserId(id); int get powerLevel => room.getPowerLevelByUserId(id);
@ -89,14 +91,11 @@ class User extends Event {
/// The avatar if the user has one. /// The avatar if the user has one.
Uri? get avatarUrl { Uri? get avatarUrl {
final prevContent = this.prevContent; final uri = content.tryGet<String>('avatar_url') ??
return content.containsKey('avatar_url') (membership == Membership.join
? (content['avatar_url'] is String ? null
? Uri.tryParse(content['avatar_url']) : prevContent?.tryGet<String>('avatar_url'));
: null) return uri == null ? null : Uri.tryParse(uri);
: (prevContent != null && prevContent['avatar_url'] is String
? Uri.tryParse(prevContent['avatar_url'])
: null);
} }
/// Returns the displayname or the local part of the Matrix ID if the user /// Returns the displayname or the local part of the Matrix ID if the user

View File

@ -5,7 +5,7 @@ homepage: https://famedly.com
repository: https://gitlab.com/famedly/company/frontend/famedlysdk.git repository: https://gitlab.com/famedly/company/frontend/famedlysdk.git
environment: environment:
sdk: ">=2.18.0 <3.0.0" sdk: ">=2.18.0 <4.0.0"
dependencies: dependencies:
async: ^2.8.0 async: ^2.8.0
@ -22,7 +22,7 @@ dependencies:
image: ^4.0.15 image: ^4.0.15
js: ^0.6.3 js: ^0.6.3
markdown: ^4.0.0 markdown: ^4.0.0
matrix_api_lite: ^1.6.1 matrix_api_lite: ">=1.6.0 <1.7.0" # Pinned until compatible with 1.7.x
mime: ^1.0.0 mime: ^1.0.0
olm: ^2.0.2 olm: ^2.0.2
random_string: ^2.3.1 random_string: ^2.3.1